Re: [apparmor] [PATCH 08/31] split routines for loading binary policy into its own file

2014-12-09 Thread Tyler Hicks
On 2014-12-06 15:21:28, Christian Boltz wrote: > Hello, > > Am Freitag, 5. Dezember 2014 schrieb Tyler Hicks: > > From: John Johansen > > > > Signed-off-by: John Johansen > > [tyhicks: Handle inverted return from find_subdomainfs_mountpoint()] > > It would be a good idea to rename this functio

Re: [apparmor] [PATCH 08/31] split routines for loading binary policy into its own file

2014-12-06 Thread Christian Boltz
Hello, Am Freitag, 5. Dezember 2014 schrieb Tyler Hicks: > From: John Johansen > > Signed-off-by: John Johansen > [tyhicks: Handle inverted return from find_subdomainfs_mountpoint()] It would be a good idea to rename this function to something like find_apparmor_mountpoint(), and also rename

[apparmor] [PATCH 08/31] split routines for loading binary policy into its own file

2014-12-05 Thread Tyler Hicks
From: John Johansen Signed-off-by: John Johansen [tyhicks: Handle inverted return from find_subdomainfs_mountpoint()] [tyhicks: Link test progs to libapparmor to fix make check build fail] [tyhicks: Migrate from opendir() to open() for opening apparmorfs] [tyhicks: Make some of the split out fun