tree fdab4fdbfa983df7b38725f6b266ec0b970baedf
parent fd8e83f429e39f7eabe737439f4b1ad7ca60f5d4
author Ke Wu <mikewu@google.com> 1618570810 -0700
committer Arnav Kansal <rnv@google.com> 1618946168 +0000

netfilter: x_tables: fix compat match/target pad out-of-bound write

xt_compat_match/target_from_user doesn't check that zeroing the area
to start of next rule won't write past end of allocated ruleset blob.

Remove this code and zero the entire blob beforehand.

Reported-by: syzbot+cfc0247ac173f597aaaa@syzkaller.appspotmail.com
Reported-by: Andy Nguyen <theflow@google.com>
Fixes: 9fa492cdc160c ("[NETFILTER]: x_tables: simplify compat API")
Signed-off-by: Florian Westphal <fw@strlen.de>

BUG=b/184972382
TEST=Presubmit test
RELEASE_NOTE=Fixd an issue in kernel that netfilter/x_tables.c
may cause out-of-bound write
SOURCE=FROMLIST(https://patchwork.ozlabs.org/project/netfilter-devel/patch/20210407193857.21120-1-fw@strlen.de/)
cos-patch: security-high

Change-Id: I2752c7c9eb17d48790adb6cebd0a7824b2c9f414
Reviewed-on: https://cos-review.googlesource.com/c/third_party/kernel/+/15632
Main-Branch-Verified: Cusky Presubmit Bot <presubmit@cos-infra-prod.iam.gserviceaccount.com>
Tested-by: Cusky Presubmit Bot <presubmit@cos-infra-prod.iam.gserviceaccount.com>
Reviewed-by: Roy Yang <royyang@google.com>
(cherry picked from commit 80bec9781ef13e9ae61bb9de5961f701bfd50c99)
Reviewed-on: https://cos-review.googlesource.com/c/third_party/kernel/+/15791
Reviewed-by: Robert Kolchmeyer <rkolchmeyer@google.com>
Tested-by: Robert Kolchmeyer <rkolchmeyer@google.com>