~sbeattie/apparmor/apparmor-2.9-utils_update

From Revision 2904 to 2885

expand all

collapse all

Rev		Summary	Authors	Date	Diff	Files
2904		utils: simplify serialize_parse_profile_start() utils: simplify serialize_parse_profile_start() Merge from trunk revision 3001 Change serialize_parse_profile_start() to use parse_profile_start() instead of using duplicated code. The behaviour is mostly kept, with the exception that the function is more strict now and raises exceptions instead of ignoring errors. In practise, this won't change anything because the profiles are parsed with parse_profile() (which calls parse_profile_start()) - and that already errors out. The tests are updated to match the more strict behaviour. The next step would be to drop serialize_parse_profile_start() completely, but this isn't urgent and can/should be done when we have test coverage for serialize_profile_from_old_profile() one day ;-) Acked-by: Steve Beattie <steve@nxnw.org>	Christian Boltz	9 years ago
2903		utils: fix is_skippable_dir() and add tests utils: fix is_skippable_dir() and add tests Merge from trunk revision 3000 Fix is_skippable_dir() - the regex also matched things like /etc/apparmor.d/dont_disable, while it should match on the full directory name. Also add some tests based on a real-world aa-logprof run (with "print (path)" in is_skippable_dir()) and some additional "funny"[tm] dirs. Needless to say that the tests         ('dont_disable',                False),         ('/etc/apparmor.d/cache_foo',   False), will fail with the old is_skippable_dir(). Acked-by: Steve Beattie <steve@nxnw.org>	Christian Boltz	9 years ago
2902		utils: replace RE_PROFILE_START utils: replace RE_PROFILE_START Merge from trunk revision 2999 Replace RE_PROFILE_START with RE_PROFILE_START_2 and adjust all code sections that used RE_PROFILE_START_2. The only real change is that test_get_flags_invalid_01 and test_get_flags_invalid_02 now expect AppArmorException instead of AppArmorBug. Acked-by: Steve Beattie <steve@nxnw.org> for trunk	Christian Boltz	9 years ago
2901		utils: implement attachment handling after rewriting set_pro... utils: implement attachment handling after rewriting set_profile_flags() to use write_header(), and making set_profile_flags more strict. Merge from trunk revisions 2996, 2997, and 2998. Changes in set_profile_flags(): - rewrite set_profile_flags to use parse_profile_start_line() and   write_header(). - replace the silent failure for non-existing files with a proper   exception (using lazy programming - the check is done by removing the   "if os.path.isfile()" check, open_file_read then raises the   exception ;-) - comment out regex_hat_flag and the code that was supposed to handle   hat flags, which were totally broken. We'll need another patch to fix   it, and we also need to decide if we want to do that because it   introduces a behaviour change (currently, aa-complain etc. don't   change hat flags). The tests for set_profile_flags() are also updated: - prepend a space to comments because write_header always adds a space   between '{' and the comment - remove a test with superfluous quotes that are no longer kept   (that's   just a profile cleanup, so dropping that test is the easiest way) - update test_set_flags_10 and test_set_flags_12 to use the correct   profile name - enable the tests for invalid (empty) flags - update the test for a non-existing file this patch makes set_profile_flags more strict: - raise AppArmorBug if newflags contains only whitespace - raise AppArmorBug if the file doesn't contain the specified profile or   no profile at all The tests are adjusted to expect AppArmorBug instead of a silent failure. Also, some tests are added for profile=None, which means to change the flags for all profiles in a file. - test_set_flags_08 is now test_set_flags_invalid_04 - test_set_flags_invalid_03 is changed to only contain one reason for   a failure, not two ;-) Finally implement attachment handling This patch implements attachment handling - aa-logprof now works with profiles that have an attachment defined, instead of ignoring audit.log entries for those profiles. Changes: - parse_profile_start_line(): remove workaround that merged the   attachment into the profile name - parse_profile_data(): store attachment when parsing a profile - update test_parse_profile_start_03,   test_serialize_parse_profile_start_03,   test_set_flags_nochange_09 and some parse_profile_start_line() tests -   they now expect correct attachment handling Acked-by: Steve Beattie <steve@nxnw.org>	Christian Boltz	9 years ago
2900		utils: rewrite parse_profile_start() utils: rewrite parse_profile_start() Merge from trunk revision 2990 Rewrite parse_profile_start() in aa.py to a more readable version. The behaviour remains unchanged (and is covered by tests). Acked-by: Kshitij Gupta <kgupta8592@gmail.com>	Christian Boltz	9 years ago
2899		utils: add tests for RE_PROFILE_START_2 and parse_profile_st... utils: add tests for RE_PROFILE_START_2 and parse_profile_start_line() Merge from trunk revision 2989 Also add AANamedRegexTest class that can be used to test a regex with named match groups. Acked-by: Steve Beattie <steve@nxnw.org>	Christian Boltz	9 years ago
2898		utils: test new parameters of write_header() utils: test new parameters of write_header() Merge from trunk revision 2988 Change the write_header tests so that the 'profile_keyword' and 'header_comment' parameters can be (and are) tested: - add a None for both to the existing tests - add some tests that come with the profile keyword and/or a comment Acked-by: Steve Beattie <steve@nxnw.org>	Christian Boltz	9 years ago
2897		utils: extend and partially rewrite write_header() utils: extend and partially rewrite write_header() Merge from trunk revision 2987 - add support for prof_data['header_comment'] (comment after '{')   and prof_data['profile_keyword'] (to force the 'profile' keyword, even   if it isn't needed) to write_header().   (set_profile_flags() will be the only user of these two for now) - fix a crash if depth is not an integer - for example,       len('   ')/2   # 3 spaces = 1.5   would cause a crash.   Also add a test for 1.5 and 1.3 spaces. - rewrite the handling of flags to avoid we have to maintain two   different template lines. - update the tests to set 'profile_keyword' and 'header_comment' to None.   This avoids big changes in the test code. I'll send another patch that   makes sure profile_keyword and header_comment are tested ;-) Acked-by: Steve Beattie <steve@nxnw.org>	Christian Boltz	9 years ago
2896		utils: add attachment to parse_profile_start() return values utils: add attachment to parse_profile_start() return values Merge from trunk revision 2986 Add the attachment to the parse_profile_start() and serialize_parse_profile_start() return values, and adjust the functions calling the *parse_profile_start() functions to save the attachment in the "attachment" variable (which isn't used yet). Also adjust the tests for the added return value. (Sorry for not getting the resultset right from the beginning!) Acked-by: Steve Beattie <steve@nxnw.org>	Christian Boltz	9 years ago
2895		utils: Add support for attachments to write_header() utils: Add support for attachments to write_header() Merge from trunk revision 2985 Also fix a little bug that added the profile keyword if the path needed quotes (profile "/foo bar" - but "/foo bar" is enough). This was caused by a regex that always matched on quoted paths (hint: "/ matches ^[^/] ;-) Also add some tests with attachments and update the test for the bugfix mentioned above. Now the remaining part is to make sure that prof_data['attachment'] gets set when parsing the profiles :-) Acked-by: Steve Beattie <steve@nxnw.org>	Christian Boltz	9 years ago
2894		utils: add tests for set_profile_flags() (and some fun) utils: add tests for set_profile_flags() (and some fun) Merge from trunk commit 2983 Add various tests for set_profile_flags, and document various interesting[tm] things I discovered while writing the tests (see the inline comments for details). Also adds a read_file() function to common_test.py. The most interesting[tm] thing I found is:     regex_hat_flag = re.compile('^([a-z]*)\s+([A-Z]*)\s*(#.*)?$') which matches various unexpected things - but not a hat :-/ (see mailinglist for all funny details) Acked-by: Steve Beattie <steve@nxnw.org>	Christian Boltz	9 years ago
2893		utils: add and use parse_profile_start_line(); convert utils: add and use parse_profile_start_line(); convert serialize_parse_profile_start() to use parse_profile_start_line(); update test-aa.py to match parse_profile_start() and get_profile_flags() changes Merge from trunk commits 2978, 2979, and 2982 Add the parse_profile_start_line() function to regex.py, which is a wrapper for RE_PROFILE_START_2 and returns an array with named matches. Also change some places in aa.py from using RE_PROFILE_START to the parse_profile_start_line() function. Notes: - until everything is migrated to the new function, I'll keep the old   RE_PROFILE_START unchanged - that's the reason to add the new   regex as RE_PROFILE_START_2 - the patch changes only aa.py sections that are covered by tests   already (which means some users of RE_PROFILE_START are remaining) - parse_profile_start_line() merges 'profile' and 'attachment' into   'profile' (aka the old, broken behaviour) until aa.py can handle   the attachment properly. The alternative would be to ignore 'attachment', which would be worse. Convert serialize_parse_profile_start() to use parse_profile_start_line(), and adjust a test to expect an AppArmorBug instead of an AttributeError exception. Also add two tests (they succeed with the old and the new code). Note that these tests document interesting[tm] behaviour - I tend to think that those cases should raise an exception, but I'm not sure about this because serialize_profile_from_old_profile() is a good example for interesting[tm] code :-/ I couldn't come up with a real-world test profile that would hit those cases without erroring out aa-logprof earlier - maybe the (more sane-looking) parse_profiles() / serialize_parse_profile_start() protects us from hitting this interesting[tm] behaviour. The previous patch slightly changed the behaviour of parse_profile_start() and get_profile_flags() - they raise AppArmorBug instead of AppArmorException when specifying a line that is not the start of a profile and therefore doesn't match RE_PROFILE_START_2. This patch updates test-aa.py to expect the correct exceptions, and adds another test with quoted profile name to ensure that stripping the quotes works as expected. Acked-by: Steve Beattie <steve@nxnw.org>	Christian Boltz	9 years ago
2892		utils: merge test-only changes from trunk utils: merge test-only changes from trunk Merge from trunk revisions 2976+2980, 2977, 2981, and 2984.	Steve Beattie	9 years ago
2891		write_net_rules() fixes, part 3 write_net_rules() fixes, part 3 Thanks to the used data structure, write_net_rules() replaces bare 'network,' rules with the invalid 'network all,' when saving a profile. This patch makes sure a correct 'network,' rule is written. Also reset 'audit' to avoid all (remaining) rules get the audit flag after writing an audit network rule. Note: The first section of the function (that claims to be responsible for bare 'network,' rules) is probably never hit - but I'm not too keen to remove it and try it out ;-) Acked-by: Steve Beattie <steve@nxnw.org> for trunk and 2.9.	Christian Boltz	9 years ago
2890		Fix doubled arrow in exec rules Fix doubled arrow in exec rules When parsing a profile with named exec rules, the exec target included the arrow. This resulted in two arrows when writing the profile (and one more each time the profile was updated). Fix this by using the match group that only contains the exec target without the arrow in parse_profile_data() and serialize_profile_from_old_profile(). References: https://bugs.launchpad.net/apparmor/+bug/1437901 Acked-by: Steve Beattie <steve@nxnw.org> for trunk and 2.9.	Christian Boltz	9 years ago
2889		profiles: allow ubuntu-helpers to generate texlive fonts profiles: allow ubuntu-helpers to generate texlive fonts Merge from trunk revision 3004 When evince opens a dvi file, it updates the user fonts using texlive commands in /usr/share/texlive/texmf-dist/web2c/ (or possibly /usr/share/texlive/texmf/web2c/ in older releases). This patch adjusts the sanitized_helper profile to allow these tools to run. Bug: https://bugs.launchpad.net/ubuntu/+source/apparmor/+bug/1010909 Signed-off-by: Steve Beattie <steve@nxnw.org> Acked-By: Jamie Strandboge <jamie@canonical.com>	Steve Beattie	9 years ago
2888		Fix writing network rules, part 2 Fix writing network rules, part 2 write_net_rules() doesn't add a space after 'audit' in two of three cases, leading to invalid network rules. This patch adds the missing spaces. Acked-by: Seth Arnold <seth.arnold@canonical.com> for both trunk and 2.9	Christian Boltz	9 years ago
2887		Fix writing network rules Fix writing network rules write_net_rules() creates invalid rules for network rules with one parameter (for example "network bluetooth"). Add a trailing comma to create valid rules. Acked-by: Seth Arnold <seth.arnold@canonical.com> for both trunk and 2.9.	Christian Boltz	9 years ago
2886		logparser.py: merge path handling in add_event_to_tree() logparser.py: merge path handling in add_event_to_tree() Merge path handling for 'inode_*' in add_event_to_tree() with the handling for other path events. The code is slightly more strict now - 'inode_' in e['operation'] is replaced with e['operation'].startswith('inode_'). This patch is a cleanup and also a preparation to fix https://bugs.launchpad.net/apparmor/+bug/1426651 and https://bugs.launchpad.net/apparmor/+bug/1243932 Acked-by: Seth Arnold <seth.arnold@canonical.com> Acked for both 2.9 and trunk.	Christian Boltz	9 years ago
2885		honor 'chmod' events in logparser.py / aa-logprof honor 'chmod' events in logparser.py / aa-logprof aa-logprof doesn't ask anything for type=AVC msg=audit(1427633461.202:281): apparmor="DENIED" operation="chmod" profile="/usr/lib64/firefox/plugin-container" name="/home/cb/.config/ibus/bus/" pid=7779 comm="plugin-containe" requested_mask="w" denied_mask="w" fsuid=1000 ouid=1000 This patch fixes this by adding 'chmod' to the list of file operation types in logparser.py. Acked-by: Seth Arnold <seth.arnold@canonical.com> for both trunk and 2.9.	Christian Boltz	9 years ago

• Older »

Loggerhead is a web-based interface for Breezy
Version: 2.0.1