~talkless/apparmor/fix_user_download_nonlatin

From Revision 3647 to 3628

expand all

collapse all

Rev		Summary	Authors	Date	Diff	Files
3647		parser: Preserve unknown profiles when restarting apparmor i... parser: Preserve unknown profiles when restarting apparmor init/job/unit CVE-2017-6507 https://launchpad.net/bugs/1668892 The common AppArmor 'restart' code used by some init scripts, upstart jobs, and/or systemd units contained functionality that is no longer appropriate to retain. Any profiles not found /etc/apparmor.d/ were assumed to be obsolete and were unloaded. That behavior became problematic now that there's a growing number of projects that maintain their own internal set of AppArmor profiles outside of /etc/apparmor.d/. It resulted in the AppArmor 'restart' code leaving some important processes running unconfined. A couple examples are profiles managed by LXD and Docker. Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Seth Arnold <seth.arnold@canonical.com> Acked-by: John Johansen <john.johansen@canonical.com>	Tyler Hicks	7 years ago
3646		parser: Fix delete after new[] -- patch from Oleg Strikov parser: Fix delete after new[] -- patch from Oleg Strikov <oleg.strikov@gmail.com>	Seth Arnold	7 years ago
3645		profiles: Update nvidia abstraction for newer nvidia drivers profiles: Update nvidia abstraction for newer nvidia drivers Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Jamie Strandboge <jamie@ubuntu.com>	Tyler Hicks	7 years ago
3644		Fix regressions caused by init_aa() Fix regressions caused by init_aa() With the init_aa() patch series commited, minitools_test.py showed several test failures - which effectively means the -d option of aa-complain, aa-cleanprof etc. was broken. These failures were caused by - calling init_aa() too late in tools.py - _after_ setting the   profiledir, which then got overwritten by init_aa() - calling init_aa() twice (because apparmor.aa gets imported in two   modules used by aa-cleanprof), which overwrote the manually set values   on the second run This patch fixes the call order in tools.py and adds a check to init_aa() so that it only runs once and ignores additional calls. Acked-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Seth Arnold <seth.arnold@canonical.com>	Christian Boltz	7 years ago
3643		test-parser-simple-tests.py: No longer skip testing generate... test-parser-simple-tests.py: No longer skip testing generated_perms_leading profiles FileRule understands leading permissions, so the reason to skip those (generated) test profiles in test-parser-simple-tests.py is gone. However, the gen-xtrans.pl script generates profiles with a not-so-valid mix of uppercase and lowercase, for example "Pux" and "Cux". The parser accepts this, but the tools complain about such rules. Therefore add the affected profiles to the exception list. In total, this means we now test 319 of the 380 generated_perms_leading test profiles. The patch also moves some lines around to get the \-escaped profiles out of the mixed uppercase/lowercase exec rule section. Acked-by: Seth Arnold <seth.arnold@canonical.com>	Christian Boltz	7 years ago
3642		utils: Fix apparmor.easyprof import in test-aa-easyprof.py utils: Fix apparmor.easyprof import in test-aa-easyprof.py The test-aa-easyprof.py script was attempting to do its own special setup to import the in-tree easyprof module. However, this proved to be very flaky and resulted in the test periodically failing due to an AttributeError the first time easyprof.parse_args() was called. This patch removes the flakiness by trusting that PYTHONPATH is set up appropriately before the test script is ran. PYTHONPATH is already initialized appropriately by utils/test/Makefile according to the USE_SYSTEM make variable. Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Seth Arnold <seth.arnold@canonical.com> Acked-by: Christian Boltz <apparmor@cboltz.de>	Tyler Hicks	7 years ago
3641		utils: Set parser executable path according to USE_SYSTEM ma... utils: Set parser executable path according to USE_SYSTEM make variable if USE_SYSTEM is not set, the utils make check target will instruct test-aa-easyprof.py to provide the path of the in-tree parser executable to aa-easyprof. If USE_SYSTEM is set, the default parser path (/sbin/apparmor_parser or the result of `which apparmor_parser`) is used. The test-aa-easyprof.py script receives the parser path by checking the __AA_PARSER environment variable. This environment variable is strictly used by the test script and not any user-facing code so two leading underscores were used. Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Christian Boltz <apparmor@cboltz.de> Acked-by: Seth Arnold <seth.arnold@canonical.com>	Tyler Hicks	7 years ago
3640		utils: Add option to aa-easyprof to specify the apparmor_par... utils: Add option to aa-easyprof to specify the apparmor_parser path When testing against a clean system without the apparmor_parser binary installed, the test-aa-easyprof.py script ends up skipping profile verification because it can't find the parser binary. This even causes a test failure due to the test_genpolicy_invalid_template_policy test. Adding a --parser option to aa-easyprof is the first step in addressing this problem. Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Christian Boltz <apparmor@cboltz.de> Acked-by: Seth Arnold <seth.arnold@canonical.com>	Tyler Hicks	7 years ago
3639		utils: Set parser base path according to USE_SYSTEM make var... utils: Set parser base path according to USE_SYSTEM make variable If USE_SYSTEM is not set, the utils make check target will instruct test-aa-easyprof.py to provide the path of the in-tree profiles/apparmor.d directory to aa-easyprof as the parser base directory. If USE_SYSTEM is set, the default base directory (/etc/apparmor.d) is used. The test-aa-easyprof.py script receives the base path by checking the __AA_BASEDIR environment variable. This environment variable is strictly used by the test script and not any user-facing code so two leading underscores were used. Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Christian Boltz <apparmor@cboltz.de> Acked-by: Seth Arnold <seth.arnold@canonical.com>	Tyler Hicks	7 years ago
3638		utils: Accept parser base and include options in aa-easyprof utils: Accept parser base and include options in aa-easyprof https://launchpad.net/bugs/1521031 aa-easyprof accepts a list of abstractions to include and, by default, execs apparmor_parser to verify the generated profile including any abstractions. However, aa-easyprof didn't provide the same flexibility as apparmor_parser when it came to where in the filesystem the abstraction files could exist. The parser supports --base (defaulting to /etc/apparmor.d) and --Include (defaulting to unset) options to specify the search paths for abstraction files. This patch adds the same options to aa-easyprof to aide in two different situations:  1) Some Ubuntu packages use aa-easyprof to generate AppArmor profiles     at build time. Something that has been previously needed is a way     for those packages to ship their own abstractions file(s) that are     #included in the easyprof-generated profile. That's not been     possible since the abstraction file(s) have not yet been installed     during the package build.  2) The test-aa-easyprof.py script contains some tests that specify     abstractions that should be #included. Without the ability to     specify a different --base or --Include directory, the abstractions     were required to be present in /etc/apparmor.d/abstractions/ or the     tests would fail. This prevents the Python utils from being able to     strictly test against in-tree code/profiles/etc. I don't like the names of the command line options --base and --Include. They're not particularly descriptive and the capital 'I' is not user friendly. However, I decided to preserve the name of the options from apparmor_parser. Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Christian Boltz <apparmor@cboltz.de> Acked-by: Seth Arnold <seth.arnold@canonical.com>	Tyler Hicks	7 years ago
3637		utils: Require apparmor.aa users to call init_aa() utils: Require apparmor.aa users to call init_aa() Introduce an apparmor.aa.init_aa() method and move the initialization code of the apparmor.aa module into it. Note that this change will break any external users of apparmor.aa because global variables that were previously initialized when importing apparmor.aa will not be initialized unless a call to the new apparmor.aa.init_aa() method is made. The main purpose of this change is to allow the utils tests to be able to set a non-default location for configuration files. Instead of hard-coding the location of logprof.conf and other utils related configuration files to /etc/apparmor/, this patch allows it to be configured by calling apparmor.aa.init_aa(confdir=PATH). This allows for the make check target to use the in-tree config file, profiles, and parser by default. A helper method, setup_aa(), is added to common_test.py that checks for an environment variable containing a non-default configuration directory path prior to calling apparmor.aa.init_aa(). All test scripts that use apparmor.aa are updated to call setup_aa(). Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Suggested-by: Christian Boltz <apparmor@cboltz.de> Acked-by: Seth Arnold <seth.arnold@canonical.com> Acked-by: Christian Boltz <apparmor@cboltz.de>	Tyler Hicks	7 years ago
3636		utils: Update the logprof.conf in the test dir to point to i... utils: Update the logprof.conf in the test dir to point to in-tree paths The utils tests should make use of the logprof.conf that resides in utils/test/ when testing against the in-tree parser and profiles. When testing against the system, it the utils tests should continue to use the system logprof.conf. This patch updates the parser and profiles paths to point to the in-tree paths. Another patch is needed to get aa.py to honor a non-hardcoded search path for logprof.conf and other configuration files. Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Christian Boltz <apparmor@cboltz.de> Acked-by: Seth Arnold <seth.arnold@canonical.com>	Tyler Hicks	7 years ago
3635		utils: Improve error messages when profiles/parser is not fo... utils: Improve error messages when profiles/parser is not found When aa.py is imported, it looks for a set of profiles and it also looks for the parser. Both of these paths are configured by logprof.conf but it isn't always obvious which logprof.conf file was used and, therefore, it isn't always obvious where aa.py is looking. This patch includes the paths in the error messages. Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Christian Boltz <apparmor@cboltz.de> Acked-by: Seth Arnold <seth.arnold@canonical.com>	Tyler Hicks	7 years ago
3634		utils: Don't enforce ordering of dbus rule attributes utils: Don't enforce ordering of dbus rule attributes https://launchpad.net/bugs/1628286 The utils were enforcing that the dbus rule attributes were strictly ordered in the following fashion:  bus -> path -> interface -> member -> peer However, the parser has always accepted the attributes in any order. If the system contained a profile which did not use the strict ordering enforced by the utils, the utils would refuse to operate at all. This patch eases the restriction on the ordering at the expense of the utils no longer being able to detect and reject a single attribute that is repeated multiple times. In that situation, only the last occurrence of the attribute will be honored by the utils. Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Christian Boltz <apparmor@cboltz.de>	Tyler Hicks	7 years ago
3633		utils: Fix failing tests in test-aa.py utils: Fix failing tests in test-aa.py The merged /usr patches to the policy broke some utils tests due to a change in the expected output. Fixes: r3600 update lots of profiles for usrMerge Signed-off-by: Tyler Hicks <tyhicks@canonical.com> Acked-by: Christian Boltz <apparmor@cboltz.de> Acked-by: Seth Arnold <seth.arnold@canonical.com>	Tyler Hicks	7 years ago
3632		Ignore change_hat events with error=-1 and "unconfined can n... Ignore change_hat events with error=-1 and "unconfined can not change_hat" That's much better than crashing aa-logprof ;-)  (use the log line in the added testcase if you want to see the crash) Reported by pfak on IRC. Acked-by: Seth Arnold <seth.arnold@canonical.com> for trunk, 2.10 and 2.9.	Christian Boltz	7 years ago
3631		Remove re.LOCALE flag Remove re.LOCALE flag Starting with python 3.6, the re.LOCALE flag can only be used with byte patterns, and errors out if used with str. This patch removes the flag in get_translated_hotkey(). References: https://bugs.launchpad.net/apparmor/+bug/1661766 Acked-by: Steve Beattie <steve@nxnw.org> for trunk, 2.10 and 2.9	Christian Boltz	7 years ago
3630		regression tests: fix environ fail case regression tests: fix environ fail case In the environ regression test, when the exec() of the child process fails, we don't report FAIL to stdout, so the regression tests consider it an error rather than a failure and abort, short-circuiting the test script. This commit fixes this by emitting the FAIL message when the result from the wait() syscall indicates the child process did not succeed. Signed-off-by: Steve Beattie <steve@nxnw.org> Acked-by: Seth Arnold <seth.arnold@canonical.com>	Steve Beattie	7 years ago
3629		Rename global variable "pid" to "log_pid" Rename global variable "pid" to "log_pid" aa.py has a global variable "pid", but it also has several functions that use "pid" as a local variable name. do_logprof_pass() even uses both - first, it passes the global variable to ReadLog, and then it creates a local variable in the "for pid in ..." loop. This patch renames the global variable to log_pid to get rid of the confusion. Note that the global variable is only handed over to ReadLog, and the only case where its previous content _might_ be used is aa-genprof which does multipe do_logprof_pass() runs. Maybe we could even get rid of this variable in aa.py and make it local to the ReadLog class, but I'm not sure if that would affect aa-genprof in interesting[tm] ways. Acked-by: John Johansen <john.johansen@canonical.com>	Christian Boltz	7 years ago
3628		Dovecot profile: change Px to mrPx for /usr/lib/dovecot/* Dovecot profile: change Px to mrPx for /usr/lib/dovecot/* Some of the /usr/lib/dovecot/* rules already have mrPx permissions, while others don't. With a more recent kernel, I noticed that at least auth, config, dict, lmtp, pop3 and ssl-params need mrPx instead of just Px (confirmed by the audit.log and actual breakage caused by the missing mr permissions). The mr additions for anvil, log and managesieve are just a wild guess, but I would be very surprised if they don't need mr. Acked-by: Seth Arnold <seth.arnold@canonical.com> for trunk, 2.10 and 2.9.	Christian Boltz	7 years ago

• Older »

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