lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 21 Dec 2022 11:07:11 +0530 From: Jishnu Prakash <quic_jprakash@...cinc.com> To: <gregkh@...uxfoundation.org>, <sboyd@...nel.org>, <agross@...nel.org>, <devicetree@...r.kernel.org>, <robh+dt@...nel.org>, <linus.walleij@...aro.org>, <quic_kamalw@...cinc.com>, <quic_subbaram@...cinc.com>, <quic_collinsd@...cinc.com> CC: <linux-arm-msm@...r.kernel.org>, <linux-kernel@...r.kernel.org>, <linux-arm-msm-owner@...r.kernel.org>, Jishnu Prakash <quic_jprakash@...cinc.com> Subject: [PATCH] Add a check for remove callback in spmi driver removal API When removing a SPMI driver, there can be a crash due to NULL pointer dereference if it does not have a remove callback defined, as the remove callback gets called directly in spmi_drv_remove(). This is one such call trace observed when removing the QCOM SPMI PMIC driver: dump_backtrace.cfi_jt+0x0/0x8 dump_stack_lvl+0xd8/0x16c panic+0x188/0x498 __cfi_slowpath+0x0/0x214 __cfi_slowpath+0x1dc/0x214 spmi_drv_remove+0x16c/0x1e0 device_release_driver_internal+0x468/0x79c driver_detach+0x11c/0x1a0 bus_remove_driver+0xc4/0x124 driver_unregister+0x58/0x84 cleanup_module+0x1c/0xc24 [qcom_spmi_pmic] __do_sys_delete_module+0x3ec/0x53c __arm64_sys_delete_module+0x18/0x28 el0_svc_common+0xdc/0x294 el0_svc+0x38/0x9c el0_sync_handler+0x8c/0xf0 el0_sync+0x1b4/0x1c0 If a driver has all its resources allocated through devm_() APIs and does not need any other explicit cleanup, it would not require a remove callback to be defined. The SPMI framework also does not enforce the presence of a remove callback when a client driver registers with it. Hence, add a check for remove callback presence before calling it when removing a SPMI driver. Jishnu Prakash (1): spmi: Add a check for remove callback when removing a SPMI driver drivers/spmi/spmi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) -- 2.7.4
Powered by blists - more mailing lists