[ovs-dev] [PATCH] ofp-print: Fix redundant expression.

Ben Pfaff blp at nicira.com
Wed Aug 19 23:12:29 UTC 2015


On Wed, Aug 05, 2015 at 01:31:25PM -0700, Joe Stringer wrote:
> 'features' is always a valid pointer to something on the stack, so
> checking (!features || ...) is the same as checking (false || ...).
> Simplify the expression.
> 
> Found by MIT STACK undefined behaviour checker.
> 
> Signed-off-by: Joe Stringer <joestringer at nicira.com>

Acked-by: Ben Pfaff <blp at nicira.com>



More information about the dev mailing list