]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-client.git/commit
apparmor: validate DFA start states are in bounds in unpack_pdb
authorMassimiliano Pellizzer <massimiliano.pellizzer@canonical.com>
Thu, 15 Jan 2026 14:30:50 +0000 (15:30 +0100)
committerJohn Johansen <john.johansen@canonical.com>
Mon, 9 Mar 2026 23:05:42 +0000 (16:05 -0700)
commit9063d7e2615f4a7ab321de6b520e23d370e58816
tree7f864e220ad12b531ddce35c747a97c5739a2edc
parent1f318b96cc84d7c2ab792fcc0bfd42a7ca890681
apparmor: validate DFA start states are in bounds in unpack_pdb

Start states are read from untrusted data and used as indexes into the
DFA state tables. The aa_dfa_next() function call in unpack_pdb() will
access dfa->tables[YYTD_ID_BASE][start], and if the start state exceeds
the number of states in the DFA, this results in an out-of-bound read.

==================================================================
 BUG: KASAN: slab-out-of-bounds in aa_dfa_next+0x2a1/0x360
 Read of size 4 at addr ffff88811956fb90 by task su/1097
 ...

Reject policies with out-of-bounds start states during unpacking
to prevent the issue.

Fixes: ad5ff3db53c6 ("AppArmor: Add ability to load extended policy")
Reported-by: Qualys Security Advisory <qsa@qualys.com>
Tested-by: Salvatore Bonaccorso <carnil@debian.org>
Reviewed-by: Georgia Garcia <georgia.garcia@canonical.com>
Reviewed-by: Cengiz Can <cengiz.can@canonical.com>
Signed-off-by: Massimiliano Pellizzer <massimiliano.pellizzer@canonical.com>
Signed-off-by: John Johansen <john.johansen@canonical.com>
security/apparmor/policy_unpack.c