Re: [PATCH] Fix a potential NULL pointer deref in znet

From: Al Viro
Date: Sat Mar 18 2006 - 19:49:50 EST


On Sun, Mar 19, 2006 at 01:12:31AM +0100, Jesper Juhl wrote:
>
> The coverity checker spotted that we dereference a pointer before we check it
> for NULL in drivers/net/znet.c::znet_interrupt().
> This fixes the issue.

The hell it does. Either interrupt really isn't shared, in which case
this check should be simply removed, or it can be shared, in which case
the code is still buggered.

Please, stop pulling Bunk - _think_ before submitting "make <program> STFU"
patches.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/