Tools: Testbench: Fix bytes control data validation in tb_send_bytes_data

The change in removing bytes control blob in module init() triggered
an issue in sof-testbench4. It resulted in valgrind fail with
scripts/host-testbench.sh run with error "Invalid read of size 1" in
"memcpy(msg + sizeof(config), (char *)abi->data + offset, chunk_size)".
The invalid read happens when abi->data doesn't have chunk_size bytes
available.

The fix is to skip bytes controls with no private data to avoid
reading garbage abi->size from adjacent topology buffer data, which
causes invalid memory reads in tb_send_bytes_data().

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>

Author: singalsu

tools/testbench/topology_ipc4.c

@@ -1358,7 +1358,9 @@ static int tb_kcontrol_cb_new(struct snd_soc_tplg_ctl_hdr *tplg_ctl,
 				tplg_bytes->priv.size);
 			return -EINVAL;
 		}
-		ctl->data = tplg_bytes->priv.data;
+		if (tplg_bytes->priv.size >= sizeof(struct sof_abi_hdr))
+			ctl->data = tplg_bytes->priv.data;
+
 		ctl->comp_info = comp_info;
 		strncpy(ctl->name, tplg_ctl->name, TB_MAX_CTL_NAME_CHARS);
 		break;

tools/testbench/utils_ipc4.c

@@ -290,7 +290,8 @@ static int tb_set_up_widget(struct testbench_prm *tp, struct tplg_comp_info *com
 		/* send the bytes data from kcontrols associated with current widget */
 		if (ctl->module_id != comp_info->module_id ||
 		    ctl->instance_id != comp_info->instance_id ||
-		    ctl->type != SND_SOC_TPLG_TYPE_BYTES)
+		    ctl->type != SND_SOC_TPLG_TYPE_BYTES ||
+		    !ctl->data)
 			continue;
 
 		abi = (struct sof_abi_hdr *)ctl->data;