devlink: use direct return of genlmsg_reply

This can remove redundant check

Signed-off-by: Li RongQing <lirongqing@baidu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/net/core/devlink.c b/net/core/devlink.c
index d3dfdca..283c3ed 100644 (file)
--- a/net/core/devlink.c
+++ b/net/core/devlink.c
@@ -4356,11 +4356,8 @@ static int devlink_fmsg_snd(struct devlink_fmsg *fmsg,
                err = -EMSGSIZE;
                goto nla_put_failure;
        }
-       err = genlmsg_reply(skb, info);
-       if (err)
-               return err;
 
-       return 0;
+       return genlmsg_reply(skb, info);
 
 nla_put_failure:
        nlmsg_free(skb);