Description
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 products
Version statuses are supplied by the assigning organization. Consult the original record for detailed ranges and platform conditions.
Linux Linux
Default version status: unaffected
e5c8e62ae551e0ad2e15412aa0c57c2856d59677 to < 8b998c171ed875e402e1fd1c5cd619e4ca05e9c1affected8b451a9a46f2bfc510e6d5c2492df91647586184 to < 3de14b4f681a1b3c6d375d98d9d37a55ebb85349affectedb6be1f5633eae200af2527e9eeb7f51be5739b0f to < 8027f82c0f5474ed72448e160ee9778051fa1f49affectedae7b143e05b36fc69d6571751855946cc45064c6 to < 2ac009a65d2385e9f0799eaf9d9694a29153a161affected2429bb9fad88c8fa84c4956b0a21cf5afe5e92b7 to < 712ca1cf756de2557a116e0df55791bac0c9f99eaffected2a934fdb01db6458288fc9386d3d8ceba6dd551a to < aad08b5f67d2a8116e1a00bce2611c1513b10bceaffected2a934fdb01db6458288fc9386d3d8ceba6dd551a to < ce792b94e03882108019ba996c1a7c4d4e09be2caffected2a934fdb01db6458288fc9386d3d8ceba6dd551a to < e7600f5cee5de14065f950807931d6e6d40fb2d7affectedee141706e701356dda41c6fed9ee18bf427c28e3affected4451412739ed33a49b34624299394ee575116d0caffected5.10.239 to < 5.10.270affected5.15.186 to < 5.15.221affected6.1.142 to < 6.1.188affected6.6.95 to < 6.6.157affected6.12.35 to < 6.12.109affected5.4.295 to < 5.5affected6.15.4 to < 6.16affected
Linux Linux
Default version status: affected
6.16affected0 to < 6.16unaffected5.10.270 to ≤ 5.10.*unaffected5.15.221 to ≤ 5.15.*unaffected6.1.188 to ≤ 6.1.*unaffected6.6.157 to ≤ 6.6.*unaffected6.12.109 to ≤ 6.12.*unaffected6.18.50 to ≤ 6.18.*unaffected7.2.4 to ≤ 7.2.*unaffected7.3 to ≤ *unaffected
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 ↗