diff options
| author | Kristof Provost <kp@FreeBSD.org> | 2021-01-13 18:41:07 +0000 |
|---|---|---|
| committer | Kristof Provost <kp@FreeBSD.org> | 2021-01-13 18:49:45 +0000 |
| commit | 44117554b1ee8edd66d7383c17802d5799fd18f2 (patch) | |
| tree | dea405f0c483c995afb13c70b1a73d2a5e99211d /tests | |
| parent | ea36212bf5711206bbaf5362a23ebb52c7f7e2a4 (diff) | |
| download | src-44117554b1ee8edd66d7383c17802d5799fd18f2.tar.gz src-44117554b1ee8edd66d7383c17802d5799fd18f2.zip | |
pf tests: pass NULL buffer to DIOCRCLRTSTATS
As discovered by syzcaller this used to provoke panics.
MFC after: 1 week
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/sys/netpfil/pf/ioctl/validation.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/tests/sys/netpfil/pf/ioctl/validation.c b/tests/sys/netpfil/pf/ioctl/validation.c index 5b1f61720f66..0d7f7631e91b 100644 --- a/tests/sys/netpfil/pf/ioctl/validation.c +++ b/tests/sys/netpfil/pf/ioctl/validation.c @@ -265,6 +265,11 @@ ATF_TC_BODY(clrtstats, tc) io.pfrio_size = 1 << 24; if (ioctl(dev, DIOCRCLRTSTATS, &io) != 0) atf_tc_fail("Request with size 1 << 24 failed"); + + io.pfrio_size = sizeof(tbl); + io.pfrio_buffer = NULL; + if (ioctl(dev, DIOCRCLRTSTATS, &io) == 0) + atf_tc_fail("Request with NULL buffer succeeded"); } ATF_TC_CLEANUP(clrtstats, tc) |