Android CTS 7.0 Release 22 (4800226)
Fix neverallowxperm checking on attributes

(Cherry-picked from commit 44c359aa0504fabe3d9427a95133379234f7d74e)

The following test incorrectly asserts a neverallowxperm failure.

	attribute test1_attr1;
	attribute test1_attr2;
	type test1_type1, test1_attr1, test1_attr2;

	allow test1_type1 test1_attr1:socket ioctl;
	allowxperm test1_type1 test1_attr2:socket ioctl { 1 };
	neverallowxperm test1_attr1 test1_attr1:socket ioctl { 0 }

To handle attributes correctly, the neverallowxperm checking has been
modified. Now when the ioctl permission is granted on an avtab entry
that matches an avrule neverallowxperm entry, the assertion checking
first determines the matching source/target/class sets between the
avtab entry and the neverallowxperm entry. Only the matching sets are
enumerated over to determine if the neverallowed extended permissions
exist and if they are granted. This is similar to how
report_assertion_avtab_matches() reports neverallow failures.

Bug: 28819676
Change-Id: I3c7ebf31fb4e532f9b1c962fdb58d85c4db009d0
Signed-off-by: Jeff Vander Stoep <jeffv@google.com>
1 file changed