Hi Dan,
On Sat, Nov 5, 2011 at 2:44 PM, Dan Carpenter <[email protected]> wrote:
> This looks like another info leak the same as the one in
> IOCTL_BCM_REGISTER_READ_PRIVATE.  Could you change
> "IoBuffer.OutputLength" to "bytes" here?
>
>>                               Status = -EFAULT;
>> +             } else {
>> +                     Status = bytes;
>> +             }

Sure, I noticed this also, but thought we would fix in another patch.

> When you've made that change can you resend with a:
> Reviewed-by: Dan Carpenter <[email protected]>

Will do.

Thanks,
Kevin
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to