mptcp: pm: remove duplicated error messages

mptcp_pm_parse_entry() and mptcp_pm_parse_addr() will already set a
error message in case of parsing issue.

Then, no need to override this error message with another less precise
one: "error parsing address".

Reviewed-by: Geliang Tang <geliang@kernel.org>
Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>

diff --git a/net/mptcp/pm_userspace.c b/net/mptcp/pm_userspace.c
index 4cbd234e267017801423f00c4617de692c21c358..ab915716ed41830fb8690140071012218f5e3145 100644 (file)
--- a/net/mptcp/pm_userspace.c
+++ b/net/mptcp/pm_userspace.c
@@ -216,10 +216,8 @@ int mptcp_pm_nl_announce_doit(struct sk_buff *skb, struct genl_info *info)
 
        addr = info->attrs[MPTCP_PM_ATTR_ADDR];
        err = mptcp_pm_parse_entry(addr, info, true, &addr_val);
-       if (err < 0) {
-               GENL_SET_ERR_MSG(info, "error parsing local address");
+       if (err < 0)
                goto announce_err;
-       }
 
        if (addr_val.addr.id == 0) {
                GENL_SET_ERR_MSG(info, "invalid addr id");
@@ -386,10 +384,8 @@ int mptcp_pm_nl_subflow_create_doit(struct sk_buff *skb, struct genl_info *info)
 
        laddr = info->attrs[MPTCP_PM_ATTR_ADDR];
        err = mptcp_pm_parse_entry(laddr, info, true, &entry);
-       if (err < 0) {
-               NL_SET_ERR_MSG_ATTR(info->extack, laddr, "error parsing local addr");
+       if (err < 0)
                goto create_err;
-       }
 
        if (entry.flags & MPTCP_PM_ADDR_FLAG_SIGNAL) {
                GENL_SET_ERR_MSG(info, "invalid addr flags");
@@ -400,10 +396,8 @@ int mptcp_pm_nl_subflow_create_doit(struct sk_buff *skb, struct genl_info *info)
 
        raddr = info->attrs[MPTCP_PM_ATTR_ADDR_REMOTE];
        err = mptcp_pm_parse_addr(raddr, info, &addr_r);
-       if (err < 0) {
-               NL_SET_ERR_MSG_ATTR(info->extack, raddr, "error parsing remote addr");
+       if (err < 0)
                goto create_err;
-       }
 
        if (!mptcp_pm_addr_families_match(sk, &entry.addr, &addr_r)) {
                GENL_SET_ERR_MSG(info, "families mismatch");
@@ -509,17 +503,13 @@ int mptcp_pm_nl_subflow_destroy_doit(struct sk_buff *skb, struct genl_info *info
 
        laddr = info->attrs[MPTCP_PM_ATTR_ADDR];
        err = mptcp_pm_parse_entry(laddr, info, true, &addr_l);
-       if (err < 0) {
-               NL_SET_ERR_MSG_ATTR(info->extack, laddr, "error parsing local addr");
+       if (err < 0)
                goto destroy_err;
-       }
 
        raddr = info->attrs[MPTCP_PM_ATTR_ADDR_REMOTE];
        err = mptcp_pm_parse_addr(raddr, info, &addr_r);
-       if (err < 0) {
-               NL_SET_ERR_MSG_ATTR(info->extack, raddr, "error parsing remote addr");
+       if (err < 0)
                goto destroy_err;
-       }
 
 #if IS_ENABLED(CONFIG_MPTCP_IPV6)
        if (addr_l.addr.family == AF_INET && ipv6_addr_v4mapped(&addr_r.addr6)) {