On Fri, 18 May 2007 14:09:03 -0400
"John W. Linville" [off-list ref] wrote:

On Wed, May 16, 2007 at 05:01:27PM -0400, Florin Malita wrote:

quoted

In libertas_process_rxed_packet() and process_rxed_802_11_packet() the 
skb is dereferenced after being passed to netif_rx (called from 
libertas_upload_rx_packet). Spotted by Coverity (1658, 1659).

 
Relocating the libertas_upload_rx_packet call is fine, but...

quoted

Also, libertas_upload_rx_packet() unconditionally returns 0 so the error 
check is dead code - might as well take it out.

Is this merely an implementation detail?  Or an absolute fact?
If the former is true, then we should preserve the error
checking.  If the latter, then we should change the signature of
libertas_upload_rx_packet to return void.

netif_rx() used to return a value in older kernels.