diff options
| author | Pavel Skripkin <[email protected]> | 2021-08-19 12:42:21 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2021-09-30 10:08:01 +0200 |
| commit | 44870a9e7a3c24acbb3f888b2a7cc22c9bdf7e7f (patch) | |
| tree | a8d660561327c989efd9d8d41143360aedc80e50 /tools/perf/scripts/python/bin | |
| parent | dccdd92b7b08ddbb7a8dbc1d6daba1f4fbd6bba2 (diff) | |
media: mxl111sf: change mutex_init() location
Syzbot reported, that mxl111sf_ctrl_msg() uses uninitialized
mutex. The problem was in wrong mutex_init() location.
Previous mutex_init(&state->msg_lock) call was in ->init() function, but
dvb_usbv2_init() has this order of calls:
dvb_usbv2_init()
dvb_usbv2_adapter_init()
dvb_usbv2_adapter_frontend_init()
props->frontend_attach()
props->init()
Since mxl111sf_* devices call mxl111sf_ctrl_msg() in ->frontend_attach()
internally we need to initialize state->msg_lock before
frontend_attach(). To achieve it, ->probe() call added to all mxl111sf_*
devices, which will simply initiaize mutex.
Reported-and-tested-by: [email protected]
Fixes: 8572211842af ("[media] mxl111sf: convert to new DVB USB")
Signed-off-by: Pavel Skripkin <[email protected]>
Signed-off-by: Sean Young <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/bin')
0 files changed, 0 insertions, 0 deletions