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

Joe Stringer joestringer at nicira.com
Tue Aug 25 00:36:42 UTC 2015


On 19 August 2015 at 16:12, Ben Pfaff <blp at nicira.com> wrote:
> 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>

Thanks, applied to master.



More information about the dev mailing list