[patch 2/3] Staging: sst: more dereferencing user pointers
Koul, Vinod
vinod.koul at intel.com
Thu Oct 21 14:15:45 UTC 2010
> case _IOC_NR(SNDRV_SST_STREAM_SET_PARAMS): {
> - struct snd_sst_params *str_param = (struct snd_sst_params *)arg;
> + struct snd_sst_params str_param;
Now this is a struct...
>
[snip]
> +
> if (!str_id) {
So this becomes bogus check, can be removed :-)
>
> + retval = copy_to_user(dest, &retval, sizeof(__u32));
> if (retval)
> retval = -EFAULT;
Trivial but this can be changed to
if (copy_to_user())
retval = -EFAULT;
Overall its good, tested it as well,
just bogus if() needs to be removed
~Vinod
More information about the devel
mailing list