Bug 2297467 - kernel: greybus: Fix use-after-free bug in gb_interface_release due to race condition.
Summary: kernel: greybus: Fix use-after-free bug in gb_interface_release due to race c...
Keywords:
Status: CLOSED DUPLICATE of bug 2297612
Alias: None
Product: Security Response
Classification: Other
Component: vulnerability
Version: unspecified
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Product Security DevOps Team
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2024-07-12 13:20 UTC by OSIDB Bzimport
Modified: 2024-07-16 23:53 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2024-07-12 15:03:10 UTC
Embargoed:


Attachments (Terms of Use)

Description OSIDB Bzimport 2024-07-12 13:20:57 UTC
In the Linux kernel, the following vulnerability has been resolved:

greybus: Fix use-after-free bug in gb_interface_release due to race condition.

In gb_interface_create, &intf->mode_switch_completion is bound with
gb_interface_mode_switch_work. Then it will be started by
gb_interface_request_mode_switch. Here is the relevant code.
if (!queue_work(system_long_wq, &intf->mode_switch_work)) {
	...
}

If we call gb_interface_release to make cleanup, there may be an
unfinished work. This function will call kfree to free the object
"intf". However, if gb_interface_mode_switch_work is scheduled to
run after kfree, it may cause use-after-free error as
gb_interface_mode_switch_work will use the object "intf".
The possible execution flow that may lead to the issue is as follows:

CPU0                            CPU1

                            |   gb_interface_create
                            |   gb_interface_request_mode_switch
gb_interface_release        |
kfree(intf) (free)          |
                            |   gb_interface_mode_switch_work
                            |   mutex_lock(&intf->mutex) (use)

Fix it by canceling the work before kfree.

Comment 1 Mauro Matteo Cascella 2024-07-12 15:03:10 UTC

*** This bug has been marked as a duplicate of bug 2297612 ***


Note You need to log in before you can comment on or make changes to this bug.