diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2022-03-01 11:11:04 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2022-03-02 13:45:27 +0000 |
| commit | d5dd781bcc81aa31b62310927f25cfa2574450f1 (patch) | |
| tree | 55c7ec8c2625ef1020681ad146b39ee241e126f6 /sound/soc/qcom | |
| parent | de2c6f98817fa5decb9b7d3b3a8a3ab864c10588 (diff) | |
| download | linux-d5dd781bcc81aa31b62310927f25cfa2574450f1.tar.gz linux-d5dd781bcc81aa31b62310927f25cfa2574450f1.tar.bz2 linux-d5dd781bcc81aa31b62310927f25cfa2574450f1.zip | |
ASoC: qcom: Fix error code in lpass_platform_copy()
The copy_to/from_user() functions return the number of bytes remaining
to be copied. This function needs to return negative error codes
because snd_soc_pcm_component_copy_user() treats positive returns as
success in soc_component_ret().
Fixes: 7d7209557b67 ("ASoC: qcom: Add support for codec dma driver")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/20220301081104.GB17375@kili
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/qcom')
| -rw-r--r-- | sound/soc/qcom/lpass-platform.c | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/sound/soc/qcom/lpass-platform.c b/sound/soc/qcom/lpass-platform.c index 1ce0878665ca..6f58f246476e 100644 --- a/sound/soc/qcom/lpass-platform.c +++ b/sound/soc/qcom/lpass-platform.c @@ -1229,15 +1229,19 @@ static int lpass_platform_copy(struct snd_soc_component *component, channel * (rt->dma_bytes / rt->channels)); if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) { - if (is_cdc_dma_port(dai_id)) + if (is_cdc_dma_port(dai_id)) { ret = copy_from_user_toio(dma_buf, buf, bytes); - else - ret = copy_from_user((void __force *)dma_buf, buf, bytes); + } else { + if (copy_from_user((void __force *)dma_buf, buf, bytes)) + ret = -EFAULT; + } } else if (substream->stream == SNDRV_PCM_STREAM_CAPTURE) { - if (is_cdc_dma_port(dai_id)) + if (is_cdc_dma_port(dai_id)) { ret = copy_to_user_fromio(buf, dma_buf, bytes); - else - ret = copy_to_user(buf, (void __force *)dma_buf, bytes); + } else { + if (copy_to_user(buf, (void __force *)dma_buf, bytes)) + ret = -EFAULT; + } } return ret; |