Patch "ASoC: Intel: Skylake: fix invalid memory access due to wrong reference of pointer" has been added to the 4.10-stable tree

This is a note to let you know that I've just added the patch titled

ASoC: Intel: Skylake: fix invalid memory access due to wrong reference of pointer

to the 4.10-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
asoc-intel-skylake-fix-invalid-memory-access-due-to-wrong-reference-of-pointer.patch
and it can be found in the queue-4.10 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.

From: Takashi Sakamoto <[email protected]>
Date: Fri, 24 Feb 2017 11:48:41 +0900
Subject: ASoC: Intel: Skylake: fix invalid memory access due to wrong reference of pointer

From: Takashi Sakamoto <[email protected]>

commit d1a6fe41d3c4ff0d26f0b186d774493555ca5282 upstream.

In 'skl_tplg_set_module_init_data()', a pointer to 'params' member of 'struct skl_algo_data' is calculated, then casted to (u32 *) and assigned to a member of configuration data. The configuration data is passed to the other functions and used to process intel IPC. In this processing, the value of member is used to get message data, however this can bring invalid memory access in 'skl_set_module_params()' as a result of calculation of a pointer for actual message data.

(sound/soc/intel/skylake/skl-topology.c)
skl_tplg_init_pipe_modules()
->skl_tplg_set_module_init_data() (has this bug)
->skl_tplg_set_module_params()
(sound/soc/intel/skylake/skl-messages.c)
->skl_set_module_params()
((char *)param) + data_offset

This commit fixes the bug.

---
 sound/soc/intel/skylake/skl-topology.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

@@ -486,7 +486,7 @@ static int skl_tplg_set_module_init_data
 			if (bc->set_params != SKL_PARAM_INIT)
 				continue;
 
-			mconfig->formats_config.caps = (u32 *)&bc->params;
+			mconfig->formats_config.caps = (u32 *)bc->params;
 			mconfig->formats_config.caps_size = bc->size;
 
 			break;

Patches currently in stable-queue which might be from [email protected] are

queue-4.10/asoc-intel-skylake-fix-invalid-memory-access-due-to-wrong-reference-of-pointer.patch