> 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
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
