Some bugfix (#516)

* Some bugfix

* Update apparmor.py

* Update apparmor.py

* Update apparmor.py

* Update apparmor.py
This commit is contained in:
Pascal Vizeli 2018-06-20 23:25:08 +02:00 committed by GitHub
parent f48182a69c
commit 9aa5eda2c8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 6 additions and 6 deletions

View File

@ -554,7 +554,7 @@ class Addon(CoreSysAttributes):
with TemporaryDirectory(dir=self.sys_config.path_tmp) as tmp_folder:
profile_file = Path(tmp_folder, 'apparmor.txt')
adjust_profile(self.slug, self.path_apparmor, self.profile_file)
adjust_profile(self.slug, self.path_apparmor, profile_file)
await self.sys_host.apparmor.load_profile(self.slug, profile_file)
@property

View File

@ -66,7 +66,7 @@ class AppArmorControl(CoreSysAttributes):
async def load_profile(self, profile_name, profile_file):
"""Load/Update a new/exists profile into AppArmor."""
if not validate_profile(profile_file, profile_name):
if not validate_profile(profile_name, profile_file):
_LOGGER.error("profile is not valid with name %s", profile_name)
raise HostAppArmorError()

View File

@ -14,8 +14,8 @@ def get_profile_name(profile_file):
profiles = set()
try:
with profile_file.open('r') as profile:
for line in profile:
with profile_file.open('r') as profile_data:
for line in profile_data:
match = RE_PROFILE.match(line)
if not match:
continue
@ -31,14 +31,14 @@ def get_profile_name(profile_file):
return profiles.pop()
def validate_profile(profile_file, profile_name):
def validate_profile(profile_name, profile_file):
"""Check if profile from file is valid with profile name."""
if profile_name == get_profile_name(profile_file):
return True
return False
def adjust_profile(profile_file, profile_name, profile_new):
def adjust_profile(profile_name, profile_file, profile_new):
"""Fix the profile name."""
org_profile = get_profile_name(profile_file)
profile_data = []