Hello, Am Freitag, 30. Dezember 2016, 09:13:56 CET schrieb Steve Beattie: > On Fri, Dec 30, 2016 at 02:54:31PM +0100, Christian Boltz wrote: > > For 2.10 and 2.9, I'd prefer to have a small patch (using netstat's > > --protocol option) instead of a full aa-unconfined rewrite. > > Okay to apply to 2.10 and 2.9? > > Subject: utils/aa-unconfined: fix netstat invocation regression > > It was reported[1] that converting the netstat command to examine
(you might want to drop the "[1]" from the patch description - or add the link to the ML archive) > processes bound to ipv6 addresses broke on OpenSUSE due to the version > of nettools not supporting the short -4 -6 arguments. > > This patch fixes the invocation of netstat to use the "--protocol > inet,inet6" arguments instead, which should return the same results > as the short options. > > Signed-off-by: Steve Beattie <[email protected]> > --- > utils/aa-unconfined | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > Index: b/utils/aa-unconfined > =================================================================== > --- a/utils/aa-unconfined > +++ b/utils/aa-unconfined > @@ -49,7 +49,7 @@ else: > output = subprocess.check_output("LANG=C netstat -nlp46", > shell=True).split("\n") > else: > #Python3 needs to translate a stream of bytes to string with > specified encoding >- output = str(subprocess.check_output("LANG=C netstat -nlp46", >shell=True), encoding='utf8').split("\n") >+ output = str(subprocess.check_output("LANG=C netstat -nlp --protocol >inet,inet6", shell=True), encoding='utf8').split("\n") Half-Acked-by: Christian Boltz <[email protected]> for 2.9 and 2.10. You can earn the other 50% of the Ack by also fixing the python2 branch 3 lines above ;-)) (no need to resend the patch) Regards, Christian Boltz -- ln -sf /dev/null /var/log und nie wieder irgendwelche Systemfehler. :-))))) [Ratti in fontlinge-devel]
signature.asc
Description: This is a digitally signed message part.
-- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
