diff options
author | Wei Yongjun <weiyongjun1@huawei.com> | 2022-09-19 19:08:30 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-11-03 17:57:51 +0300 |
commit | 9d00384270b15ed19c3987effe826b061b567171 (patch) | |
tree | b7184279bf882bf3fd6b74e947969a31e03742fc /net/ieee802154 | |
parent | 568e3812b1778b4c0c229649b59977d88f400ece (diff) | |
download | linux-9d00384270b15ed19c3987effe826b061b567171.tar.xz |
net: ieee802154: fix error return code in dgram_bind()
commit 444d8ad4916edec8a9fc684e841287db9b1e999f upstream.
Fix to return error code -EINVAL from the error handling
case instead of 0, as done elsewhere in this function.
Fixes: 94160108a70c ("net/ieee802154: fix uninit value bug in dgram_sendmsg")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Link: https://lore.kernel.org/r/20220919160830.1436109-1-weiyongjun@huaweicloud.com
Signed-off-by: Stefan Schmidt <stefan@datenfreihafen.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net/ieee802154')
-rw-r--r-- | net/ieee802154/socket.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/ieee802154/socket.c b/net/ieee802154/socket.c index ecc0d5fbde04..d4c275e56d82 100644 --- a/net/ieee802154/socket.c +++ b/net/ieee802154/socket.c @@ -503,8 +503,10 @@ static int dgram_bind(struct sock *sk, struct sockaddr *uaddr, int len) if (err < 0) goto out; - if (addr->family != AF_IEEE802154) + if (addr->family != AF_IEEE802154) { + err = -EINVAL; goto out; + } ieee802154_addr_from_sa(&haddr, &addr->addr); dev = ieee802154_get_dev(sock_net(sk), &haddr); |