summaryrefslogtreecommitdiff
path: root/net/sctp
diff options
context:
space:
mode:
authorKees Cook <keescook@chromium.org>2023-01-28 01:40:37 +0300
committerPaolo Abeni <pabeni@redhat.com>2023-01-31 12:37:58 +0300
commitde5ca4c3852f896cacac2bf259597aab5e17d9e3 (patch)
treef8444769663180faad7673b3d7c88f81d12d4723 /net/sctp
parentf3eceaed9edd7c0e0d9fb057613131f92973626f (diff)
downloadlinux-de5ca4c3852f896cacac2bf259597aab5e17d9e3.tar.xz
net: sched: sch: Bounds check priority
Nothing was explicitly bounds checking the priority index used to access clpriop[]. WARN and bail out early if it's pathological. Seen with GCC 13: ../net/sched/sch_htb.c: In function 'htb_activate_prios': ../net/sched/sch_htb.c:437:44: warning: array subscript [0, 31] is outside array bounds of 'struct htb_prio[8]' [-Warray-bounds=] 437 | if (p->inner.clprio[prio].feed.rb_node) | ~~~~~~~~~~~~~~~^~~~~~ ../net/sched/sch_htb.c:131:41: note: while referencing 'clprio' 131 | struct htb_prio clprio[TC_HTB_NUMPRIO]; | ^~~~~~ Cc: Jamal Hadi Salim <jhs@mojatatu.com> Cc: Cong Wang <xiyou.wangcong@gmail.com> Cc: Jiri Pirko <jiri@resnulli.us> Cc: "David S. Miller" <davem@davemloft.net> Cc: Eric Dumazet <edumazet@google.com> Cc: Jakub Kicinski <kuba@kernel.org> Cc: Paolo Abeni <pabeni@redhat.com> Cc: netdev@vger.kernel.org Signed-off-by: Kees Cook <keescook@chromium.org> Reviewed-by: Simon Horman <simon.horman@corigine.com> Reviewed-by: Cong Wang <cong.wang@bytedance.com> Link: https://lore.kernel.org/r/20230127224036.never.561-kees@kernel.org Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Diffstat (limited to 'net/sctp')
0 files changed, 0 insertions, 0 deletions