Skip to content

Commit 9b9132e

Browse files
committed
module-adapter: fix wrong memory freeing
Memory, allocated using the module_driver_heap_*() API, should be freed using module_driver_heap_free(), not rfree(). Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
1 parent 1df93bc commit 9b9132e

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/audio/module_adapter/module_adapter.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -181,10 +181,10 @@ struct comp_dev *module_adapter_new_ext(const struct comp_driver *drv,
181181
err:
182182
#if CONFIG_IPC_MAJOR_4
183183
if (mod)
184-
rfree(mod->priv.cfg.input_pins);
184+
module_driver_heap_free(drv->user_heap, mod->priv.cfg.input_pins);
185185
#endif
186-
rfree(mod);
187-
rfree(dev);
186+
module_driver_heap_free(drv->user_heap, mod);
187+
module_driver_heap_free(drv->user_heap, dev);
188188
return NULL;
189189
}
190190
EXPORT_SYMBOL(module_adapter_new);
@@ -1283,12 +1283,12 @@ void module_adapter_free(struct comp_dev *dev)
12831283
mod_free_all(mod);
12841284

12851285
#if CONFIG_IPC_MAJOR_4
1286-
rfree(mod->priv.cfg.input_pins);
1286+
module_driver_heap_free(dev->drv->user_heap, mod->priv.cfg.input_pins);
12871287
#endif
12881288

12891289
rfree(mod->stream_params);
1290-
rfree(mod);
1291-
rfree(dev);
1290+
module_driver_heap_free(dev->drv->user_heap, mod);
1291+
module_driver_heap_free(dev->drv->user_heap, dev);
12921292
}
12931293
EXPORT_SYMBOL(module_adapter_free);
12941294

0 commit comments

Comments
 (0)