summaryrefslogtreecommitdiff
path: root/security/apparmor/policy_unpack_test.c
diff options
context:
space:
mode:
authorJohn Johansen <john.johansen@canonical.com>2022-10-03 11:30:38 +0300
committerJohn Johansen <john.johansen@canonical.com>2022-10-04 00:49:04 +0300
commit32490541682bf8ea445e9bd29c866981851e0912 (patch)
tree2c929579b8d05ec3d1c3df7740799e8e8542c8b1 /security/apparmor/policy_unpack_test.c
parenta2f31df06b7aa1769f12ec6f9ae7f18e78582cad (diff)
downloadlinux-32490541682bf8ea445e9bd29c866981851e0912.tar.xz
apparmor: Fix kunit test for out of bounds array
The apparmor kunit tests are failing on the out of bounds array check with the following failure # policy_unpack_test_unpack_array_out_of_bounds: EXPECTATION FAILED at security/apparmor/policy_unpack_test.c:178 Expected unpack_array(puf->e, name, &array_size) == 1, but unpack_array(puf->e, name, &array_size) == -1 # policy_unpack_test_unpack_array_out_of_bounds: EXPECTATION FAILED at security/apparmor/policy_unpack_test.c:180 Expected array_size == 0, but array_size == 64192 not ok 5 - policy_unpack_test_unpack_array_out_of_bounds This is because unpack_array changed to allow distinguishing between the array not being present and an error. In the error case the array size is not set and should not be tested. Reported-by: kernel test robot <yujie.liu@intel.com> Fixes: 995a5b64620e ("apparmor: make unpack_array return a trianary value") Signed-off-by: John Johansen <john.johansen@canonical.com>
Diffstat (limited to 'security/apparmor/policy_unpack_test.c')
-rw-r--r--security/apparmor/policy_unpack_test.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/security/apparmor/policy_unpack_test.c b/security/apparmor/policy_unpack_test.c
index 1a43d538c4c0..b214f6ea8a72 100644
--- a/security/apparmor/policy_unpack_test.c
+++ b/security/apparmor/policy_unpack_test.c
@@ -176,8 +176,7 @@ static void policy_unpack_test_unpack_array_out_of_bounds(struct kunit *test)
puf->e->end = puf->e->start + TEST_ARRAY_BUF_OFFSET + sizeof(u16);
KUNIT_EXPECT_EQ(test, unpack_array(puf->e, name, &array_size),
- TRI_TRUE);
- KUNIT_EXPECT_EQ(test, array_size, 0);
+ TRI_FALSE);
KUNIT_EXPECT_PTR_EQ(test, puf->e->pos,
puf->e->start + TEST_NAMED_ARRAY_BUF_OFFSET);
}