diff options
| author | Gaosheng Cui <cuigaosheng1@huawei.com> | 2024-07-06 08:51:55 +0200 |
|---|---|---|
| committer | Michal Simek <michal.simek@amd.com> | 2024-10-02 09:15:29 +0200 |
| commit | 44ed4f90a97ff6f339e50ac01db71544e0990efc (patch) | |
| tree | ed258f52b47b13013596c19776f8210a6da2556b /drivers/soc | |
| parent | Linux 6.12-rc1 (diff) | |
| download | linux-44ed4f90a97ff6f339e50ac01db71544e0990efc.tar.xz linux-44ed4f90a97ff6f339e50ac01db71544e0990efc.zip | |
drivers: soc: xilinx: add the missing kfree in xlnx_add_cb_for_suspend()
If we fail to allocate memory for cb_data by kmalloc, the memory
allocation for eve_data is never freed, add the missing kfree()
in the error handling path.
Fixes: 05e5ba40ea7a ("driver: soc: xilinx: Add support of multiple callbacks for same event in event management driver")
Signed-off-by: Gaosheng Cui <cuigaosheng1@huawei.com>
Link: https://lore.kernel.org/r/20240706065155.452764-1-cuigaosheng1@huawei.com
Signed-off-by: Michal Simek <michal.simek@amd.com>
Diffstat (limited to 'drivers/soc')
| -rw-r--r-- | drivers/soc/xilinx/xlnx_event_manager.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/soc/xilinx/xlnx_event_manager.c b/drivers/soc/xilinx/xlnx_event_manager.c index f529e1346247..85df6b9c04ee 100644 --- a/drivers/soc/xilinx/xlnx_event_manager.c +++ b/drivers/soc/xilinx/xlnx_event_manager.c @@ -188,8 +188,10 @@ static int xlnx_add_cb_for_suspend(event_cb_func_t cb_fun, void *data) INIT_LIST_HEAD(&eve_data->cb_list_head); cb_data = kmalloc(sizeof(*cb_data), GFP_KERNEL); - if (!cb_data) + if (!cb_data) { + kfree(eve_data); return -ENOMEM; + } cb_data->eve_cb = cb_fun; cb_data->agent_data = data; |