[alsa-devel] [PATCH] topology: Fix the missing referenced elem ptr when merging private data

mengdong.lin at linux.intel.com mengdong.lin at linux.intel.com
Thu Jul 21 09:00:47 CEST 2016


From: Mengdong Lin <mengdong.lin at linux.intel.com>

tplg_copy_data() should set the valid referenced data element pointer.
The caller will double check this pointer for all kinds of references,
including TLV, controls, text and data.

Also fix an inaccurate dmesg.

Signed-off-by: Mengdong Lin <mengdong.lin at linux.intel.com>

diff --git a/src/topology/dapm.c b/src/topology/dapm.c
index 4d343b2..e3c90d8 100644
--- a/src/topology/dapm.c
+++ b/src/topology/dapm.c
@@ -201,7 +201,7 @@ static int tplg_build_widget(snd_tplg_t *tplg,
 		}
 
 		if (!ref->elem) {
-			SNDERR("error: cannot find control '%s'"
+			SNDERR("error: cannot find '%s'"
 				" referenced by widget '%s'\n",
 				ref->id, elem->id);
 			return -EINVAL;
diff --git a/src/topology/data.c b/src/topology/data.c
index 768fc27..397f6a5 100644
--- a/src/topology/data.c
+++ b/src/topology/data.c
@@ -1050,6 +1050,7 @@ int tplg_copy_data(snd_tplg_t *tplg, struct tplg_elem *elem,
 		" element '%s'\n", ref->id, elem->id);
 		return -EINVAL;
 	}
+	ref->elem = ref_elem;
 
 	tplg_dbg("Data '%s' used by '%s'\n", ref->id, elem->id);
 	/* overlook empty private data */
-- 
2.5.0



More information about the Alsa-devel mailing list