Acked-by: Alin Gabriel Serdean <[email protected]>

> -----Mesaj original-----
> De la: [email protected] <ovs-dev-
> [email protected]> În numele Ben Pfaff
> Trimis: Thursday, November 15, 2018 7:08 PM
> Către: [email protected]
> Cc: Ben Pfaff <[email protected]>
> Subiect: [ovs-dev] [PATCH] dpif-netlink: Fix error behavior in
> dpif_netlink_port_add__().
> 
> Until now, the code here would report an error to its caller as success.
> This fixes the problem.
> 
> Found by inspection.
> 
> Signed-off-by: Ben Pfaff <[email protected]>
> ---
>  lib/dpif-netlink.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/dpif-netlink.c b/lib/dpif-netlink.c index
> f2011f22e548..7f6a6cc59592 100644
> --- a/lib/dpif-netlink.c
> +++ b/lib/dpif-netlink.c
> @@ -716,7 +716,8 @@ dpif_netlink_port_add__(struct dpif_netlink *dpif,
> const char *name,
>      int error = 0;
> 
>      if (dpif->handlers) {
> -        if (nl_sock_create(NETLINK_GENERIC, &socksp)) {
> +        error = nl_sock_create(NETLINK_GENERIC, &socksp);
> +        if (error) {
>              return error;
>          }
>      }
> --
> 2.16.1
> 
> _______________________________________________
> 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