[PATCH] staging: Check for Null return of allocated skb in fw_download_code
Valdis.Kletnieks at vt.edu
Valdis.Kletnieks at vt.edu
Wed Aug 13 15:17:19 EDT 2014
On Wed, 13 Aug 2014 14:03:08 -0400, Nick Krause said:
> I did test my patch by doing a kernel build and I get this error,
> drivers/staging/rtl8192e/rtl8192e/r8192E_firmware.c:66:4: error:
> implicit declaration of function âskb_quene_purgeâ
> [-Werror=implicit-function-declaration]
> skb_quene_purge(&priv->rtllib->skb_waitQ[tcb_desc->queue_index]);
Well, this is a C 101 problem.
> I am wondering how do I fix this,
You fix this by not doing any further kernel hacking until you've gotten
a handle on *VERY BASIC* C development concepts.
Also, you're going to have to justify why you're being a total
dumbass and coding skb_waitQ[tcb_desc->queue_index] when the *obvious*
code is skb_waitQ[TXCMD_QUEUE] - yes, an optimizing compiler will do
that substitution, but code clarity is important.
Argh. I may have to break out my +5 Trout of Smacking....
On Wed, 13 Aug 2014 15:01:45 -0400, Nick Krause said:
> This is the fixed patch, I do get an error about uninitialized
> variables
Are you *trying* to get put in *everybody's* killfile?
> If someone wants to send this out, please do so as this is has been
> built and applied tested.
And you think that *anybody* wants to upstream a patch from you that even
*you* admit still has trouble??!?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 848 bytes
Desc: not available
Url : http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20140813/d83ebb86/attachment.bin
More information about the Kernelnewbies
mailing list