summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVinod Koul <vinod.koul@intel.com>2013-07-29 15:10:22 +0530
committerTakashi Iwai <tiwai@suse.de>2013-07-29 13:58:08 +0200
commita8d30608eaed6cc759b8e2e8a8bbbb42591f797f (patch)
tree579ffca8fb888783e6632482b11d003668b3565e
parent43cbd286fbba4bfb0c2ef05dc62522dfd5a2c000 (diff)
downloadop-kernel-dev-a8d30608eaed6cc759b8e2e8a8bbbb42591f797f.zip
op-kernel-dev-a8d30608eaed6cc759b8e2e8a8bbbb42591f797f.tar.gz
ALSA: compress: fix the return value for SNDRV_COMPRESS_VERSION
the return value of SNDRV_COMPRESS_VERSION always return default -ENOTTY as the return value was never updated for this call assign return value from put_user() Reported-by: Haynes <hgeorge@codeaurora.org> CC: stable@vger.kernel.org Signed-off-by: Vinod Koul <vinod.koul@intel.com> Signed-off-by: Takashi Iwai <tiwai@suse.de>
-rw-r--r--sound/core/compress_offload.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/core/compress_offload.c b/sound/core/compress_offload.c
index 99db892..9896954 100644
--- a/sound/core/compress_offload.c
+++ b/sound/core/compress_offload.c
@@ -743,7 +743,7 @@ static long snd_compr_ioctl(struct file *f, unsigned int cmd, unsigned long arg)
mutex_lock(&stream->device->lock);
switch (_IOC_NR(cmd)) {
case _IOC_NR(SNDRV_COMPRESS_IOCTL_VERSION):
- put_user(SNDRV_COMPRESS_VERSION,
+ retval = put_user(SNDRV_COMPRESS_VERSION,
(int __user *)arg) ? -EFAULT : 0;
break;
case _IOC_NR(SNDRV_COMPRESS_GET_CAPS):
OpenPOWER on IntegriCloud