Description (CNA)
In the Linux kernel, the following vulnerability has been resolved: Revert "media: v4l2-dev: fix error handling in __video_register_device()" This reverts commit 2a934fdb01db6458288fc9386d3d8ceba6dd551a. The intentions of that patch were good, but it doesn't work. The idea is that if device_register fails, you have to do a put_device to let the ref counter release resources. However, the V4L2 API says that if video_register_device() fails, then you have to call video_device_release(), which kfree()s the video_device struct. But the put_device() will already have freed the struct, so you end up in a double-free scenario. There is not really a good way of fixing this without breaking video_register_device() into two parts, one that initializes everything, and one that does the actual device_register, and then converting all V4L2 drivers to this new model. That is a massive job, and it is very unlikely that device_register will fail. So rather than ending up in a double-free scenario, just revert this patch, and in that case we'll have a small memory leak. Which is a lot more robust.
Affected (CNA)
| Vendor / product | Versions |
|---|---|
| Linux / Linux | affected e5c8e62ae551e0ad2e15412aa0c57c2856d59677 before 8b998c171ed875e402e1fd1c5cd619e4ca05e9c1; affected 8b451a9a46f2bfc510e6d5c2492df91647586184 before 3de14b4f681a1b3c6d375d98d9d37a55ebb85349; affected b6be1f5633eae200af2527e9eeb7f51be5739b0f before 8027f82c0f5474ed72448e160ee9778051fa1f49; affected ae7b143e05b36fc69d6571751855946cc45064c6 before 2ac009a65d2385e9f0799eaf9d9694a29153a161; affected 2429bb9fad88c8fa84c4956b0a21cf5afe5e92b7 before 712ca1cf756de2557a116e0df55791bac0c9f99e; affected 2a934fdb01db6458288fc9386d3d8ceba6dd551a before aad08b5f67d2a8116e1a00bce2611c1513b10bce; affected 2a934fdb01db6458288fc9386d3d8ceba6dd551a before ce792b94e03882108019ba996c1a7c4d4e09be2c; affected 2a934fdb01db6458288fc9386d3d8ceba6dd551a before e7600f5cee5de14065f950807931d6e6d40fb2d7; affected ee141706e701356dda41c6fed9ee18bf427c28e3; affected 4451412739ed33a49b34624299394ee575116d0c; affected 5.10.239 before 5.10.270; affected 5.15.186 before 5.15.221; affected 6.1.142 before 6.1.188; affected 6.6.95 before 6.6.157; affected 6.12.35 before 6.12.109; affected 5.4.295 before 5.5; affected 6.15.4 before 6.16 |
| Linux / Linux | affected 6.16; unaffected 0 before 6.16; unaffected 5.10.270 through 5.10.*; unaffected 5.15.221 through 5.15.*; unaffected 6.1.188 through 6.1.*; unaffected 6.6.157 through 6.6.*; unaffected 6.12.109 through 6.12.*; unaffected 6.18.50 through 6.18.*; unaffected 7.2.4 through 7.2.*; unaffected 7.3 through * |
CWE
No CWE identifiers supplied.
References
- https://git.kernel.org/stable/c/8b998c171ed875e402e1fd1c5cd619e4ca05e9c1
- https://git.kernel.org/stable/c/3de14b4f681a1b3c6d375d98d9d37a55ebb85349
- https://git.kernel.org/stable/c/8027f82c0f5474ed72448e160ee9778051fa1f49
- https://git.kernel.org/stable/c/2ac009a65d2385e9f0799eaf9d9694a29153a161
- https://git.kernel.org/stable/c/712ca1cf756de2557a116e0df55791bac0c9f99e
- https://git.kernel.org/stable/c/aad08b5f67d2a8116e1a00bce2611c1513b10bce
- https://git.kernel.org/stable/c/ce792b94e03882108019ba996c1a7c4d4e09be2c
- https://git.kernel.org/stable/c/e7600f5cee5de14065f950807931d6e6d40fb2d7