From: Li RongQing <lirongqing@baidu.com>
Date: Mon, 11 Feb 2019 11:09:07 +0000 (+0800)
Subject: devlink: use direct return of genlmsg_reply
X-Git-Tag: v5.1-rc1~178^2~264
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=fde55ea74ce69b1ba2fb108b197f7c3c6b8749e1;p=uclinux-h8%2Flinux.git

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 d3dfdcacf7eb..283c3ed9f25e 100644
--- 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);