summaryrefslogtreecommitdiffstats
path: root/net/sctp
diff options
context:
space:
mode:
authorYajun Deng <yajun.deng@linux.dev>2021-07-13 04:48:24 +0200
committerDavid S. Miller <davem@davemloft.net>2021-07-13 18:28:29 +0200
commit01757f536ac825e3614d583fee9acb48c64ed084 (patch)
tree0b31ad3ca2f4175b3c113a74c645a9cf28881548 /net/sctp
parentocteontx2-pf: Fix uninitialized boolean variable pps (diff)
downloadlinux-01757f536ac825e3614d583fee9acb48c64ed084.tar.xz
linux-01757f536ac825e3614d583fee9acb48c64ed084.zip
net: Use nlmsg_unicast() instead of netlink_unicast()
It has 'if (err >0 )' statement in nlmsg_unicast(), so use nlmsg_unicast() instead of netlink_unicast(), this looks more concise. v2: remove the change in netfilter. Signed-off-by: Yajun Deng <yajun.deng@linux.dev> Reviewed-by: David Ahern <dsahern@kernel.org> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sctp')
-rw-r--r--net/sctp/diag.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/net/sctp/diag.c b/net/sctp/diag.c
index 493fc01e5d2b..760b367644c1 100644
--- a/net/sctp/diag.c
+++ b/net/sctp/diag.c
@@ -284,10 +284,8 @@ static int sctp_tsp_dump_one(struct sctp_transport *tsp, void *p)
goto out;
}
- err = netlink_unicast(net->diag_nlsk, rep, NETLINK_CB(in_skb).portid,
- MSG_DONTWAIT);
- if (err > 0)
- err = 0;
+ err = nlmsg_unicast(net->diag_nlsk, rep, NETLINK_CB(in_skb).portid);
+
out:
return err;
}