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 <pkusunyif...@gmail.com>
> 
> On Sun, Dec 31, 2017 at 9:16 PM, Ben Pfaff <b...@ovn.org> wrote:
> 
> > Signed-off-by: Ben Pfaff <b...@ovn.org>
> > ---
> >  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
> > d...@openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to