RE: [PATCH] staging: comedi: (regression) channel list must be set for COMEDI_CMD ioctl

2014-10-14 Thread Hartley Sweeten
On Thursday, October 09, 2014 2:03 AM, Ian Abbott wrote: On 09/10/14 00:13, Hartley Sweeten wrote: On Wednesday, October 08, 2014 8:09 AM, Ian Abbott wrote: `do_cmd_ioctl()`, the handler for the `COMEDI_CMD` ioctl can incorrectly call the Comedi subdevice's `do_cmd()` handler with a NULL

Re: [PATCH] staging: comedi: (regression) channel list must be set for COMEDI_CMD ioctl

2014-10-09 Thread Ian Abbott
On 09/10/14 00:13, Hartley Sweeten wrote: On Wednesday, October 08, 2014 8:09 AM, Ian Abbott wrote: `do_cmd_ioctl()`, the handler for the `COMEDI_CMD` ioctl can incorrectly call the Comedi subdevice's `do_cmd()` handler with a NULL channel list pointer. This is a regression as the `do_cmd()`

[PATCH] staging: comedi: (regression) channel list must be set for COMEDI_CMD ioctl

2014-10-08 Thread Ian Abbott
`do_cmd_ioctl()`, the handler for the `COMEDI_CMD` ioctl can incorrectly call the Comedi subdevice's `do_cmd()` handler with a NULL channel list pointer. This is a regression as the `do_cmd()` handler has never been expected to deal with that, leading to a kernel OOPS when it tries to dereference

RE: [PATCH] staging: comedi: (regression) channel list must be set for COMEDI_CMD ioctl

2014-10-08 Thread Hartley Sweeten
On Wednesday, October 08, 2014 8:09 AM, Ian Abbott wrote: `do_cmd_ioctl()`, the handler for the `COMEDI_CMD` ioctl can incorrectly call the Comedi subdevice's `do_cmd()` handler with a NULL channel list pointer. This is a regression as the `do_cmd()` handler has never been expected to deal