diff options
author | Nicolas Iooss <nicolas.iooss_linux@m4x.org> | 2016-08-28 21:10:04 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-10-22 12:26:55 +0200 |
commit | a3a9b33e49ff91f6ceaeb1d5855046d50cefa3ef (patch) | |
tree | 1f8ee122ebe601fbb9315a3607edc1f141e04b9d /fs | |
parent | 85e27fe87acd059aa1c615184ec2c9ac9bfe29d2 (diff) |
ASoC: Intel: Atom: add a missing star in a memcpy call
commit 61ab0d403bbd9d5f6e000e3b5734049141b91f6f upstream.
In sst_prepare_and_post_msg(), when a response is received in "block",
the following code gets executed:
*data = kzalloc(block->size, GFP_KERNEL);
memcpy(data, (void *) block->data, block->size);
The memcpy() call overwrites the content of the *data pointer instead of
filling the newly-allocated memory (which pointer is hold by *data).
Fix this by merging kzalloc+memcpy into a single kmemdup() call.
Thanks Joe Perches for suggesting using kmemdup()
Fixes: 60dc8dbacb00 ("ASoC: Intel: sst: Add some helper functions")
Signed-off-by: Nicolas Iooss <nicolas.iooss_linux@m4x.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'fs')
0 files changed, 0 insertions, 0 deletions