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

Reply via email to