On 04/06/17 00:16, Bill Fischofer wrote:
> Resolve Bug https://bugs.linaro.org/show_bug.cgi?id=2779 by checking
> the return code from odph_eth_addr_parse() and failing the call if
> dst_mac is unparseable.
> 
> Signed-off-by: Bill Fischofer <bill.fischo...@linaro.org>
> ---
>  example/l3fwd/odp_l3fwd_db.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/example/l3fwd/odp_l3fwd_db.c b/example/l3fwd/odp_l3fwd_db.c
> index 082b2c2..7d1efd5 100644
> --- a/example/l3fwd/odp_l3fwd_db.c
> +++ b/example/l3fwd/odp_l3fwd_db.c
> @@ -394,7 +394,8 @@ int create_fwd_db_entry(char *input, char **oif, uint8_t 
> **dst_mac)
>                       *oif = entry->oif;
>                       break;
>               case 2:
> -                     odph_eth_addr_parse(&entry->dst_mac, token);
> +                     if (odph_eth_addr_parse(&entry->dst_mac, token) < 0)
> +                             return -1;

free(local) is missing

Maxim.

>                       *dst_mac = entry->dst_mac.addr;
>                       break;
>  
> 

Reply via email to