[apparmor] [parser patch] fix apparmor cache tempfile location to use passed arg

Christian Boltz apparmor at cboltz.de
Tue Jul 23 20:31:56 UTC 2013


Hello,

Am Dienstag, 23. Juli 2013 schrieb Steve Beattie:
> The second issue is that it would generate the temporary file that it
> stores the cache file in [basedir]/cache even if an alternate cache
> location was specified on the command line. This causes a problem
> if [basedir]/cache is on a separate device than the alternate cache
> location, because the rename() of the tempfile into the final location
> would fail 

Looks like it was a good idea to package /etc/apparmor/cache as a 
symlink to /var/cache/apparmor on openSUSE ;-)

> (which the parser would not check the return code of).

Nice[tm].

That said - your patch looks like something that should be backported to 
the 2.8 branch (even if it isn't needed for openSUSE thanks to the 
symlink).


Regards,

Christian Boltz
-- 
Aren't most of SUSE-employed community members part of the
Research&Destroy department? [Sascha Peilicke in opensuse-project]




More information about the AppArmor mailing list