tplg: result of the free function propagated

Marcin Maka · Marcin Maka · commit f42c6573ea62 · 2018-10-08T14:21:18.000+02:00
Signed-off-by: Marcin Maka &lt;marcin.maka@linux.intel.com&gt;
diff --git a/src/ipc/handler.c b/src/ipc/handler.c
@@ -935,6 +935,7 @@ static int ipc_glb_tplg_free(uint32_t header,
 		int (*free_func)(struct ipc *ipc, uint32_t id))
 {
 	struct sof_ipc_free *ipc_free = _ipc->comp_data;
+	int ret;
 
 	trace_ipc("Tcf");
 
@@ -945,9 +946,14 @@ static int ipc_glb_tplg_free(uint32_t header,
 	}
 
 	/* free the object */
-	free_func(_ipc, ipc_free->id);
+	ret = free_func(_ipc, ipc_free->id);
 
-	return 0;
+	if (ret < 0) {
+		trace_error(TRACE_CLASS_IPC,
+			    "ipc-glb-tplg-free free_func failed %d", ret);
+	}
+
+	return ret;
 }
 
 static int ipc_glb_tplg_message(uint32_t header)

Original file line number	Diff line number	Diff line change
`@@ -935,6 +935,7 @@ static int ipc_glb_tplg_free(uint32_t header,`
`935`	`935`	`int (free_func)(struct ipc ipc, uint32_t id))`
`936`	`936`	`{`
`937`	`937`	`struct sof_ipc_free *ipc_free = _ipc->comp_data;`
	`938`	`+ int ret;`
`938`	`939`
`939`	`940`	`trace_ipc("Tcf");`
`940`	`941`
`@@ -945,9 +946,14 @@ static int ipc_glb_tplg_free(uint32_t header,`
`945`	`946`	`}`
`946`	`947`
`947`	`948`	`/* free the object */`
`948`		`- free_func(_ipc, ipc_free->id);`
	`949`	`+ ret = free_func(_ipc, ipc_free->id);`
`949`	`950`
`950`		`- return 0;`
	`951`	`+ if (ret < 0) {`
	`952`	`+ trace_error(TRACE_CLASS_IPC,`
	`953`	`+ "ipc-glb-tplg-free free_func failed %d", ret);`
	`954`	`+ }`
	`955`	`+`
	`956`	`+ return ret;`
`951`	`957`	`}`
`952`	`958`
`953`	`959`	`static int ipc_glb_tplg_message(uint32_t header)`