Good spot. Thanks for fixing this!

Acked-by: Jan Scheurich <[email protected]>

> -----Original Message-----
> From: [email protected] 
> [mailto:[email protected]] On Behalf Of Ben Pfaff
> Sent: Wednesday, 29 November, 2017 17:59
> To: [email protected]
> Cc: Ben Pfaff <[email protected]>; Bhargava Shastry 
> <[email protected]>
> Subject: [ovs-dev] [PATCH] odp-util: Fix another hang in NSH action parsing.
> 
> Found by libfuzzer.
> 
> Reported-by: Bhargava Shastry <[email protected]>
> Signed-off-by: Ben Pfaff <[email protected]>
> ---
>  lib/odp-util.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/odp-util.c b/lib/odp-util.c
> index 3e30b9ae7719..23d5a01b3bc7 100644
> --- a/lib/odp-util.c
> +++ b/lib/odp-util.c
> @@ -1867,8 +1867,8 @@ parse_odp_encap_nsh_action(const char *s, struct ofpbuf 
> *actions)
>                  }
>                  encap_nsh.mdlen = mdlen + padding;
>                  ofpbuf_uninit(&b);
> +                continue;
>              }
> -            continue;
>          }
> 
>          return -EINVAL;
> --
> 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