]> git.baikalelectronics.ru Git - kernel.git/commit
net: sched: cls_u32: Avoid memcpy() false-positive warning
authorKees Cook <keescook@chromium.org>
Tue, 27 Sep 2022 15:37:01 +0000 (08:37 -0700)
committerJakub Kicinski <kuba@kernel.org>
Fri, 30 Sep 2022 01:44:07 +0000 (18:44 -0700)
commit283a96d38b355c8cf88b9fad90c6748334c0c334
tree8b18c900a236a5a65f6c612d0728687c5ef4abcd
parenta553a1a93f6ff4446d2c0b0b4f4c034edd22cf92
net: sched: cls_u32: Avoid memcpy() false-positive warning

To work around a misbehavior of the compiler's ability to see into
composite flexible array structs (as detailed in the coming memcpy()
hardening series[1]), use unsafe_memcpy(), as the sizing,
bounds-checking, and allocation are all very tightly coupled here.
This silences the false-positive reported by syzbot:

  memcpy: detected field-spanning write (size 80) of single field "&n->sel" at net/sched/cls_u32.c:1043 (size 16)

[1] https://lore.kernel.org/linux-hardening/20220901065914.1417829-2-keescook@chromium.org

Cc: Cong Wang <xiyou.wangcong@gmail.com>
Cc: Jiri Pirko <jiri@resnulli.us>
Reported-by: syzbot+a2c4601efc75848ba321@syzkaller.appspotmail.com
Link: https://lore.kernel.org/lkml/000000000000a96c0b05e97f0444@google.com/
Signed-off-by: Kees Cook <keescook@chromium.org>
Reviewed-by: Jamal Hadi Salim <jhs@mojatatu.com>
Link: https://lore.kernel.org/r/20220927153700.3071688-1-keescook@chromium.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/sched/cls_u32.c