summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMengdong Lin <mengdong.lin@linux.intel.com>2016-07-22 13:12:54 +0800
committerTakashi Iwai <tiwai@suse.de>2016-07-25 11:55:35 +0200
commit98a0a2a98ffca4ed929c34d322c8389816a5e8a8 (patch)
treebcd2fb5350b9f1f9b90f6844acbd1421c9ed186b
parent776eb7fa5e705d46ac2cac517cdb69dc6c7d8f56 (diff)
downloadalsa-lib-98a0a2a98ffca4ed929c34d322c8389816a5e8a8.tar.gz
topology: Fix the missing referenced elem ptr when merging private data
tplg_copy_data() should set the valid referenced data element pointer on success. The caller will double check this pointer for all kinds of references, including controls and data. Signed-off-by: Mengdong Lin <mengdong.lin@linux.intel.com> Signed-off-by: Takashi Iwai <tiwai@suse.de>
-rw-r--r--src/topology/data.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/src/topology/data.c b/src/topology/data.c
index e81b7f12..59bc9700 100644
--- a/src/topology/data.c
+++ b/src/topology/data.c
@@ -1049,8 +1049,10 @@ int tplg_copy_data(snd_tplg_t *tplg, struct tplg_elem *elem,
tplg_dbg("Data '%s' used by '%s'\n", ref->id, elem->id);
/* overlook empty private data */
- if (!ref_elem->data || !ref_elem->data->size)
+ if (!ref_elem->data || !ref_elem->data->size) {
+ ref->elem = ref_elem;
return 0;
+ }
old_priv = get_priv_data(elem);
if (!old_priv)
@@ -1074,6 +1076,8 @@ int tplg_copy_data(snd_tplg_t *tplg, struct tplg_elem *elem,
ref_elem->compound_elem = 1;
memcpy(priv->data + old_priv_data_size,
ref_elem->data->data, priv_data_size);
+
+ ref->elem = ref_elem;
return 0;
}