Re: [2.6 patch] drivers/usb/misc/sisusbvga/sisusb.c: remove dead code

On Wed, Nov 23, 2005 at 11:02:37AM -0800, Greg KH wrote:
> On Mon, Nov 21, 2005 at 12:22:39AM +0100, Adrian Bunk wrote:
> > The Coverity checker found this obviously dead code.
> 
> I think the checker is wrong, this does not look correct to me.

Why?

Due to the while(length), length can't be 0 at the switch.

> thanks,
> 
> greg k-h

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Follow-Ups:
- Re: [2.6 patch] drivers/usb/misc/sisusbvga/sisusb.c: remove dead code
  - From: Greg KH <[email protected]>

References:
- [2.6 patch] drivers/usb/misc/sisusbvga/sisusb.c: remove dead code
  - From: Adrian Bunk <[email protected]>
- Re: [2.6 patch] drivers/usb/misc/sisusbvga/sisusb.c: remove dead code
  - From: Greg KH <[email protected]>

Prev by Date: [RFC] [PATCH 1/3] ioat: DMA subsystem
Next by Date: Re: [discuss] Re: [LinuxBIOS] x86_64: apic id lift patch
Previous by thread: Re: [2.6 patch] drivers/usb/misc/sisusbvga/sisusb.c: remove dead code
Next by thread: Re: [2.6 patch] drivers/usb/misc/sisusbvga/sisusb.c: remove dead code
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Newbies] [Netfilter] [Bugtraq] [Photo] [Stuff] [Gimp] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Video 4 Linux] [Linux for the blind] [Linux Resources]