[PATCH]: go7007/go7007-usb.c: 2 * minor quibble

d binderman dcb314 at hotmail.com
Fri Mar 19 09:36:30 UTC 2010



Hello there,

I just ran the sourceforge tool cppcheck over the source code of the
new Linux kernel 2.6.34-rc1

It said

[staging/go7007/go7007-usb.c:1251]: (style) Redundant condition. It is safe to deallocate a NULL poi
nter
[staging/go7007/go7007-usb.c:1258]: (style) Redundant condition. It is safe to deallocate a NULL poi
nter

The source code is

                        if (vurb->transfer_buffer)
                                kfree(vurb->transfer_buffer);

and

                       if (aurb->transfer_buffer)
                                kfree(aurb->transfer_buffer);

Proposed patch file attached.


Regards

David Binderman
 		 	   		  
_________________________________________________________________
Send us your Hotmail stories and be featured in our newsletter
http://clk.atdmt.com/UKM/go/195013117/direct/01/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug42.patch
Type: text/x-patch
Size: 659 bytes
Desc: not available
URL: <http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/attachments/20100319/646d9d0f/attachment.bin>


More information about the devel mailing list