Eric Dumazet
f4191e8943
net: sched: validate stab values
...
commit e323d865b36134e8c5c82c834df89109a5c60dab upstream.
iproute2 package is well behaved, but malicious user space can
provide illegal shift values and trigger UBSAN reports.
Add stab parameter to red_check_params() to validate user input.
syzbot reported:
UBSAN: shift-out-of-bounds in ./include/net/red.h:312:18
shift exponent 111 is too large for 64-bit type 'long unsigned int'
CPU: 1 PID: 14662 Comm: syz-executor.3 Not tainted 5.12.0-rc2-syzkaller #0
Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
Call Trace:
__dump_stack lib/dump_stack.c:79 [inline]
dump_stack+0x141/0x1d7 lib/dump_stack.c:120
ubsan_epilogue+0xb/0x5a lib/ubsan.c:148
__ubsan_handle_shift_out_of_bounds.cold+0xb1/0x181 lib/ubsan.c:327
red_calc_qavg_from_idle_time include/net/red.h:312 [inline]
red_calc_qavg include/net/red.h:353 [inline]
choke_enqueue.cold+0x18/0x3dd net/sched/sch_choke.c:221
__dev_xmit_skb net/core/dev.c:3837 [inline]
__dev_queue_xmit+0x1943/0x2e00 net/core/dev.c:4150
neigh_hh_output include/net/neighbour.h:499 [inline]
neigh_output include/net/neighbour.h:508 [inline]
ip6_finish_output2+0x911/0x1700 net/ipv6/ip6_output.c:117
__ip6_finish_output net/ipv6/ip6_output.c:182 [inline]
__ip6_finish_output+0x4c1/0xe10 net/ipv6/ip6_output.c:161
ip6_finish_output+0x35/0x200 net/ipv6/ip6_output.c:192
NF_HOOK_COND include/linux/netfilter.h:290 [inline]
ip6_output+0x1e4/0x530 net/ipv6/ip6_output.c:215
dst_output include/net/dst.h:448 [inline]
NF_HOOK include/linux/netfilter.h:301 [inline]
NF_HOOK include/linux/netfilter.h:295 [inline]
ip6_xmit+0x127e/0x1eb0 net/ipv6/ip6_output.c:320
inet6_csk_xmit+0x358/0x630 net/ipv6/inet6_connection_sock.c:135
dccp_transmit_skb+0x973/0x12c0 net/dccp/output.c:138
dccp_send_reset+0x21b/0x2b0 net/dccp/output.c:535
dccp_finish_passive_close net/dccp/proto.c:123 [inline]
dccp_finish_passive_close+0xed/0x140 net/dccp/proto.c:118
dccp_terminate_connection net/dccp/proto.c:958 [inline]
dccp_close+0xb3c/0xe60 net/dccp/proto.c:1028
inet_release+0x12e/0x280 net/ipv4/af_inet.c:431
inet6_release+0x4c/0x70 net/ipv6/af_inet6.c:478
__sock_release+0xcd/0x280 net/socket.c:599
sock_close+0x18/0x20 net/socket.c:1258
__fput+0x288/0x920 fs/file_table.c:280
task_work_run+0xdd/0x1a0 kernel/task_work.c:140
tracehook_notify_resume include/linux/tracehook.h:189 [inline]
Fixes: 8afa10cbe281 ("net_sched: red: Avoid illegal values")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Reported-by: syzbot <syzkaller@googlegroups.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2021-03-30 14:41:42 +02:00
..
2019-10-29 09:15:12 +01:00
2018-04-13 19:48:33 +02:00
2017-03-22 12:43:34 +01:00
2018-03-22 09:17:42 +01:00
2018-01-17 09:38:54 +01:00
2019-08-11 12:22:18 +02:00
2017-08-30 10:21:42 +02:00
2016-03-01 17:15:23 -05:00
2016-09-19 21:55:28 -04:00
2020-01-29 10:24:33 +01:00
2016-07-25 21:49:19 -07:00
2019-11-28 18:28:04 +01:00
2016-09-19 22:04:14 -04:00
2018-06-26 08:08:06 +08:00
2016-07-25 21:49:19 -07:00
2018-04-13 19:48:35 +02:00
2018-08-24 13:12:37 +02:00
2016-10-03 21:40:50 -04:00
2017-01-15 13:42:53 +01:00
2016-11-28 10:47:35 -05:00
2016-11-28 10:47:35 -05:00
2016-11-28 10:47:35 -05:00
2016-11-28 10:47:35 -05:00
2020-03-11 07:53:05 +01:00
2016-09-19 22:04:14 -04:00
2020-02-28 15:42:44 +01:00
2020-04-02 17:20:29 +02:00
2020-02-14 16:30:58 -05:00
2021-01-30 13:27:16 +01:00
2018-09-15 09:42:55 +02:00
2016-04-14 00:35:30 -04:00
2020-02-05 13:05:42 +00:00
2016-09-19 21:55:28 -04:00
2016-09-19 21:55:28 -04:00
2021-03-17 16:10:14 +01:00
2016-06-25 12:19:35 -04:00
2018-07-22 14:27:36 +02:00
2019-10-07 18:53:24 +02:00
2021-03-30 14:41:42 +02:00
2019-08-11 12:22:17 +02:00
2016-06-25 12:19:35 -04:00
2019-10-07 18:53:23 +02:00
2016-09-19 01:47:18 -04:00
2020-05-20 08:15:24 +02:00
2020-03-20 09:07:40 +01:00
2020-06-30 15:38:37 -04:00
2021-03-30 14:41:42 +02:00
2016-08-08 16:06:47 -07:00
2019-11-10 11:23:31 +01:00
2018-09-15 09:43:02 +02:00
2016-06-07 16:59:53 -07:00
2019-12-05 15:35:30 +01:00
2019-12-05 15:35:30 +01:00
2019-12-05 15:35:30 +01:00
2020-01-29 10:24:34 +01:00
2016-09-19 01:47:18 -04:00
2016-06-25 12:19:35 -04:00
2020-01-12 11:24:27 +01:00
2016-09-23 06:52:31 -04:00
2021-03-30 14:41:42 +02:00
2019-11-10 11:23:31 +01:00
2021-03-30 14:41:42 +02:00
2019-02-20 10:18:28 +01:00
2016-06-25 12:19:35 -04:00