Thanks for the review!
On Tue, Jan 16, 2018 at 09:13:43AM -0800, Yifeng Sun wrote: > Looks good to me, thanks for the patch. > > Reviewed-by: Yifeng Sun <[email protected]> > > On Sun, Dec 31, 2017 at 9:16 PM, Ben Pfaff <[email protected]> wrote: > > > Signed-off-by: Ben Pfaff <[email protected]> > > --- > > lib/json.c | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/lib/json.c b/lib/json.c > > index 5e93190b8a03..07ca87b2130f 100644 > > --- a/lib/json.c > > +++ b/lib/json.c > > @@ -576,9 +576,9 @@ json_equal(const struct json *a, const struct json *b) > > { > > if (a == b) { > > return true; > > - } > > - > > - if (a->type != b->type) { > > + } else if (!a || !b) { > > + return false; > > + } else if (a->type != b->type) { > > return false; > > } > > > > -- > > 2.10.2 > > > > _______________________________________________ > > dev mailing list > > [email protected] > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
