| Message ID | 20221005014750.3685555-1-aahringo@redhat.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [net,1/2] Revert "net/ieee802154: reject zero-sized raw_sendmsg()" | expand |
Hello. On 05.10.22 03:47, Alexander Aring wrote: > This reverts commit 3a4d061c699bd3eedc80dc97a4b2a2e1af83c6f5. > > There is a v2 which does return zero if zero length is given. > > Signed-off-by: Alexander Aring <aahringo@redhat.com> > --- > net/ieee802154/socket.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/net/ieee802154/socket.c b/net/ieee802154/socket.c > index cbd0e2ac4ffe..7889e1ef7fad 100644 > --- a/net/ieee802154/socket.c > +++ b/net/ieee802154/socket.c > @@ -251,9 +251,6 @@ static int raw_sendmsg(struct sock *sk, struct msghdr *msg, size_t size) > return -EOPNOTSUPP; > } > > - if (!size) > - return -EINVAL; > - > lock_sock(sk); > if (!sk->sk_bound_dev_if) > dev = dev_getfirstbyhwtype(sock_net(sk), ARPHRD_IEEE802154); This patch has been applied to the wpan tree and will be part of the next pull request to net. Thanks! regards Stefan Schmidt
diff --git a/net/ieee802154/socket.c b/net/ieee802154/socket.c index cbd0e2ac4ffe..7889e1ef7fad 100644 --- a/net/ieee802154/socket.c +++ b/net/ieee802154/socket.c @@ -251,9 +251,6 @@ static int raw_sendmsg(struct sock *sk, struct msghdr *msg, size_t size) return -EOPNOTSUPP; } - if (!size) - return -EINVAL; - lock_sock(sk); if (!sk->sk_bound_dev_if) dev = dev_getfirstbyhwtype(sock_net(sk), ARPHRD_IEEE802154);
This reverts commit 3a4d061c699bd3eedc80dc97a4b2a2e1af83c6f5. There is a v2 which does return zero if zero length is given. Signed-off-by: Alexander Aring <aahringo@redhat.com> --- net/ieee802154/socket.c | 3 --- 1 file changed, 3 deletions(-)