mirror of
https://gitlab.com/apparmor/apparmor.git
synced 2025-03-04 08:24:42 +01:00
39 lines
1.5 KiB
Diff
39 lines
1.5 KiB
Diff
|
From b1cb9d1b4f0d585c271c584da954d9eb2e347b40 Mon Sep 17 00:00:00 2001
|
||
|
|
From: John Johansen <john.johansen@canonical.com>
|
||
|
|
Date: Fri, 29 Jun 2012 17:34:00 -0700
|
||
|
|
Subject: [PATCH 3/6] apparmor: Fix quieting of audit messages for network
|
||
|
|
mediation
|
||
|
|
|
||
|
|
If a profile specified a quieting of network denials for a given rule by
|
||
|
|
either the quiet or deny rule qualifiers, the resultant quiet mask for
|
||
|
|
denied requests was applied incorrectly, resulting in two potential bugs.
|
||
|
|
1. The misapplied quiet mask would prevent denials from being correctly
|
||
|
|
tested against the kill mask/mode. Thus network access requests that
|
||
|
|
should have resulted in the application being killed did not.
|
||
|
|
|
||
|
|
2. The actual quieting of the denied network request was not being applied.
|
||
|
|
This would result in network rejections always being logged even when
|
||
|
|
they had been specifically marked as quieted.
|
||
|
|
|
||
|
|
Signed-off-by: John Johansen <john.johansen@canonical.com>
|
||
|
|
---
|
||
|
|
security/apparmor/net.c | 2 +-
|
||
|
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||
|
|
|
||
|
|
diff --git a/security/apparmor/net.c b/security/apparmor/net.c
|
||
|
|
index 003dd18..6e6e5c9 100644
|
||
|
|
--- a/security/apparmor/net.c
|
||
|
|
+++ b/security/apparmor/net.c
|
||
|
|
@@ -88,7 +88,7 @@ static int audit_net(struct aa_profile *profile, int op, u16 family, int type,
|
||
|
|
} else {
|
||
|
|
u16 quiet_mask = profile->net.quiet[sa.u.net->family];
|
||
|
|
u16 kill_mask = 0;
|
||
|
|
- u16 denied = (1 << sa.aad->net.type) & ~quiet_mask;
|
||
|
|
+ u16 denied = (1 << sa.aad->net.type);
|
||
|
|
|
||
|
|
if (denied & kill_mask)
|
||
|
|
audit_type = AUDIT_APPARMOR_KILL;
|
||
|
|
--
|
||
|
|
1.7.10.4
|
||
|
|
|