Skip to content
This repository was archived by the owner on Nov 8, 2023. It is now read-only.

Commit d885c46

Browse files
zijun-hugregkh
authored andcommitted
driver core: bus: Fix double free in driver API bus_register()
[ Upstream commit bfa54a7 ] For bus_register(), any error which happens after kset_register() will cause that @priv are freed twice, fixed by setting @priv with NULL after the first free. Signed-off-by: Zijun Hu <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Greg Kroah-Hartman <[email protected]> Signed-off-by: Sasha Levin <[email protected]>
1 parent a84ae28 commit d885c46

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

drivers/base/bus.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -920,6 +920,8 @@ int bus_register(const struct bus_type *bus)
920920
bus_remove_file(bus, &bus_attr_uevent);
921921
bus_uevent_fail:
922922
kset_unregister(&priv->subsys);
923+
/* Above kset_unregister() will kfree @priv */
924+
priv = NULL;
923925
out:
924926
kfree(priv);
925927
return retval;

0 commit comments

Comments
 (0)