Re: [PATCH-3.13.y] staging: comedi: fix result of memdup_user for user chanlist

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jan 21, 2014 at 01:58:37PM +0000, Ian Abbott wrote:
> From: Bernd Porr <mail@xxxxxxxxxxxxxxx>
> 
> commit e56b1401056288a725d50942ef300dcbed5e519a upstream
> 
> If the channel list is not set in userspace we get an error at
> PTR_ERR(async->cmd.chanlist). However, do_become_nonbusy(dev, s) cleans
> up this pointer which causes a kernel ooops. Setting the channel list in
> async to NULL and checking this in do_become_nonbusy prevents the oops.
> 
> [Ian Abbott] Also do the same for the chanlist allocated in
> do_cmdtest_ioctl().
> 
> Signed-off-by: Bernd Porr <mail@xxxxxxxxxxxxxxx>
> Signed-off-by: Ian Abbott <abbotti@xxxxxxxxx>
> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> ---
>  drivers/staging/comedi/comedi_fops.c | 2 ++
>  1 file changed, 2 insertions(+)

Applied, thanks.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]