[PATCH] firmware: arm_scmi: fix missing destroy_workqueue() on error in scmi_notification_init

From: Qinglang Miao
Date: Mon Nov 09 2020 - 04:10:14 EST


Add the missing destroy_workqueue() before return from
scmi_notification_init in the error handling case when
fails to do devm_kcalloc().

Fixes: bd31b249692e ("firmware: arm_scmi: Add notification dispatch and delivery")
Signed-off-by: Qinglang Miao <miaoqinglang@xxxxxxxxxx>
---
drivers/firmware/arm_scmi/notify.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/firmware/arm_scmi/notify.c b/drivers/firmware/arm_scmi/notify.c
index 2754f9d01636..3048e57d9731 100644
--- a/drivers/firmware/arm_scmi/notify.c
+++ b/drivers/firmware/arm_scmi/notify.c
@@ -1476,8 +1476,10 @@ int scmi_notification_init(struct scmi_handle *handle)

ni->registered_protocols = devm_kcalloc(handle->dev, SCMI_MAX_PROTO,
sizeof(char *), GFP_KERNEL);
- if (!ni->registered_protocols)
+ if (!ni->registered_protocols) {
+ destroy_workqueue(ni->notify_wq);
goto err;
+ }

mutex_init(&ni->pending_mtx);
hash_init(ni->pending_events_handlers);
--
2.23.0