Unverified Commit 00566ad4 authored by Cezary Rojewski's avatar Cezary Rojewski Committed by Mark Brown
Browse files

ASoC: Intel: avs: Shield LARGE_CONFIG_GETs against zero payload_size 



Some LARGE_CONFIG_GETs are never expected to return payload of size 0.
Check for such situation and collapse if met.

Signed-off-by: default avatarCezary Rojewski <cezary.rojewski@intel.com>
Link: https://lore.kernel.org/r/20220707124153.1858249-6-cezary.rojewski@intel.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 3c1923a1

sound/soc/intel/avs/messages.c

+9 −0
Original line number Diff line number Diff line
@@ -474,6 +474,9 @@ int avs_ipc_get_fw_config(struct avs_dev *adev, struct avs_fw_cfg *cfg) 
				       &payload, &payload_size);

	if (ret)

		return ret;

	/* Non-zero payload expected for FIRMWARE_CONFIG. */

	if (!payload_size)

		return -EREMOTEIO;



	while (offset < payload_size) {

		tlv = (struct avs_tlv *)(payload + offset);
@@ -587,6 +590,9 @@ int avs_ipc_get_hw_config(struct avs_dev *adev, struct avs_hw_cfg *cfg) 
				       &payload, &payload_size);

	if (ret)

		return ret;

	/* Non-zero payload expected for HARDWARE_CONFIG. */

	if (!payload_size)

		return -EREMOTEIO;



	while (offset < payload_size) {

		tlv = (struct avs_tlv *)(payload + offset);
@@ -670,6 +676,9 @@ int avs_ipc_get_modules_info(struct avs_dev *adev, struct avs_mods_info **info) 
				       &payload, &payload_size);

	if (ret)

		return ret;

	/* Non-zero payload expected for MODULES_INFO. */

	if (!payload_size)

		return -EREMOTEIO;



	*info = (struct avs_mods_info *)payload;

	return 0;