Re: drivers/net/xen-netfront.c: bogus code

2007-07-22 Thread Jeremy Fitzhardinge
Adrian Bunk wrote: > The Coverity checker spotted the following bogus code > in drivers/net/xen-netfront.c: > > <-- snip --> > > ... > static void xennet_alloc_rx_buffers(struct net_device *dev) > { > ... > for (nr_flips = i = 0; ; i++) { > skb = __skb_dequeue(&np->rx_batc

drivers/net/xen-netfront.c: bogus code

2007-07-22 Thread Adrian Bunk
The Coverity checker spotted the following bogus code in drivers/net/xen-netfront.c: <-- snip --> ... static void xennet_alloc_rx_buffers(struct net_device *dev) { ... for (nr_flips = i = 0; ; i++) { skb = __skb_dequeue(&np->rx_batch); if (skb == NULL)