Re: [PATCH 08/12] staging: dgap: tty.c: fixes ioctl param list

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

 



On Fri, Aug 23, 2013 at 4:23 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> On Wed, Aug 21, 2013 at 09:48:38PM -0400, Lidza Louina wrote:
>> -static int dgap_tty_ioctl(struct tty_struct *tty, struct file *file, unsigned int cmd,
>> +static int dgap_tty_ioctl(struct tty_struct *tty, unsigned int cmd,
>>               unsigned long arg)
>
> Btw, now that this file compiles I see it in my static checker runs.
>
> dgap_tty_ioctl() takes a lock but always returns before unlocking.
>
>>  {
>>       struct board_t *bd;
>> @@ -3423,41 +3423,6 @@ static int dgap_tty_ioctl(struct tty_struct *tty, struct file *file, unsigned in
>>
>>               return(-ENOIOCTLCMD);
>
> In other words, these return statements should be breaks so we hit
> the call to:
>
>         DGNC_UNLOCK(ch->ch_lock, lock_flags);

Okay I am going to fix that today. Thanks. =]
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux