[apparmor] applying utils patches to 2.9 (was: Re: [patch] fix is_skippable_dir() and add tests)
Christian Boltz
apparmor at cboltz.de
Sat Apr 4 12:10:04 UTC 2015
Hello,
Am Freitag, 3. April 2015 schrieb Steve Beattie:
> The actual fix in
> utils/apparmor/aa.py is fine for 2.9, but we (I?) haven't decided on
> pulling the AATest stuff to 2.9, so it wouldn't be acceptable in
> isolation unless some reworking occurred.
You probably noticed that I proposed the whole patch series for trunk
and 2.9, and I still think this would be a good idea ;-)
I don't see a risk in applying test changes in 2.9 (even if this means
pulling in AATest, which you could call a bigger change). Worst thing
that can happen is a test failure, and that's something we'll notice
easily ;-)
The series has some non-test patches that are bigger than AATest and are
needed to fix real bugs - and backporting only the "core bugfix" without
the cleanup, added tests etc. is much more work and IMHO also more risky
than applying all patches.
Besides that: most of the changed code is covered by (freshly added)
tests, to I'm quite sure the risk of breaking something is quite low
(even if v1 of "logparser.py: change mask only for path events" just
showed that this risk still exists ;-)
TL;DR: IMHO we should apply all utils/** patches from the last weeks to
2.9, which basically means the only difference between trunk and 2.9
would be the CapabilityRule class.
Regards,
Christian Boltz
--
Wo steht der Server eigentlich? Kann den die Putzfrau treten?
Oder mal mit dem Staubsauger überfahren? Denen fallen ab und an
Gemeinheiten ein auf die ein Normalsterblicher nie kommen würde. :\
[Daniel Lord in suse-linux]
More information about the AppArmor
mailing list