module_adapter: generic: Fix use after free

Remove any containers from the free container list so that
we don't keep pointers to containers that are no longer used
and will be freed when container chunks are released below.

Leaving those nodes in the free container list would cause
use-after-free on subsequent allocations.

While at it, make sure all resource lists are reset.

Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>

Author: Daniel Baluta

src/audio/module_adapter/module/generic.c

@@ -594,13 +594,26 @@ void mod_free_all(struct processing_module *mod)
 		list_item_del(&container->list);
 	}
 
+	list_for_item_safe(list, _list, &res->free_cont_list) {
+		struct module_resource *container =
+			container_of(list, struct module_resource, list);
+
+		list_item_del(&container->list);
+	}
+
 	list_for_item_safe(list, _list, &res->cont_chunk_list) {
 		struct container_chunk *chunk =
 			container_of(list, struct container_chunk, chunk_list);
 
 		list_item_del(&chunk->chunk_list);
 		rfree(chunk);
 	}
+
+	/* Make sure resource lists and accounting are reset */
+	list_init(&res->res_list);
+	list_init(&res->free_cont_list);
+	list_init(&res->cont_chunk_list);
+	res->heap_usage = 0;
 }
 EXPORT_SYMBOL(mod_free_all);