[ovs-dev] [PATCH 1/1 net-next] openvswitch: use PTR_ERR_OR_ZERO

Pravin Shelar pshelar at nicira.com
Fri Nov 14 20:34:06 UTC 2014


On Fri, Nov 14, 2014 at 10:32 AM, Fabian Frederick <fabf at skynet.be> wrote:
> Signed-off-by: Fabian Frederick <fabf at skynet.be>
Acked-by: Pravin B Shelar <pshelar at nicira.com>

> ---
>  net/openvswitch/flow_netlink.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/net/openvswitch/flow_netlink.c b/net/openvswitch/flow_netlink.c
> index c0d066d..c8fccdd 100644
> --- a/net/openvswitch/flow_netlink.c
> +++ b/net/openvswitch/flow_netlink.c
> @@ -1425,10 +1425,8 @@ static int add_action(struct sw_flow_actions **sfa, int attrtype,
>         struct nlattr *a;
>
>         a = __add_action(sfa, attrtype, data, len, log);
> -       if (IS_ERR(a))
> -               return PTR_ERR(a);
>
> -       return 0;
> +       return PTR_ERR_OR_ZERO(a);
>  }
>
>  static inline int add_nested_action_start(struct sw_flow_actions **sfa,
> --
> 1.9.3
>



More information about the dev mailing list