[apparmor] [PATCH 0/3] Fix minor process_file_entries() bugs

Tyler Hicks tyhicks at canonical.com
Wed Sep 11 08:42:29 UTC 2013


The C++ parser patches, which will hit the list shortly, made the parser
segfault on a couple of the test profiles in parser/tst/simple_tests/xtrans. It
was a double free that was caused by some code in process_file_entries(). That
function can return while the cod->entries linked list is partially modified,
resulting in the segfault when cod and its members are being freed.

The segfault doesn't happen until C++ is mixed in, which causes strange pointer
values.

These patches (mainly patch 2/2) fix the issue prior to the C++ patches
hitting. They are written against sbeattie's memory leak fixes.

Tyler



More information about the AppArmor mailing list