diff options
| author | Zhengchao Shao <shaozhengchao@huawei.com> | 2022-07-27 17:33:12 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-09-05 10:28:59 +0200 |
| commit | 4931af31c402b20c528b43d90f98bc6f4bcb5810 (patch) | |
| tree | d7a1adefc594dc1b6039f8172c070779422665e5 | |
| parent | 64f6da455b66f17da52a479b23ca7e953f3552f5 (diff) | |
| download | linux-stable-4931af31c402b20c528b43d90f98bc6f4bcb5810.tar.gz linux-stable-4931af31c402b20c528b43d90f98bc6f4bcb5810.tar.bz2 linux-stable-4931af31c402b20c528b43d90f98bc6f4bcb5810.zip | |
net/af_packet: check len when min_header_len equals to 0
commit dc633700f00f726e027846a318c5ffeb8deaaeda upstream.
User can use AF_PACKET socket to send packets with the length of 0.
When min_header_len equals to 0, packet_snd will call __dev_queue_xmit
to send packets, and sock->type can be any type.
Reported-by: syzbot+5ea725c25d06fb9114c4@syzkaller.appspotmail.com
Fixes: fd1894224407 ("bpf: Don't redirect packets with invalid pkt_len")
Signed-off-by: Zhengchao Shao <shaozhengchao@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | net/packet/af_packet.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/packet/af_packet.c b/net/packet/af_packet.c index 5ee600d108a0..b70b06e312bd 100644 --- a/net/packet/af_packet.c +++ b/net/packet/af_packet.c @@ -2986,8 +2986,8 @@ static int packet_snd(struct socket *sock, struct msghdr *msg, size_t len) if (err) goto out_free; - if (sock->type == SOCK_RAW && - !dev_validate_header(dev, skb->data, len)) { + if ((sock->type == SOCK_RAW && + !dev_validate_header(dev, skb->data, len)) || !skb->len) { err = -EINVAL; goto out_free; } |