Re: [PATCH 1/1] Obsolete check is now removed.

[Paul Moore <paul@paul-moore.com>]

On Wednesday, May 27, 2015 04:41:05 PM Mikhail Klementyev wrote:
> From a2e33ce7c9b5ceff2a8b570b570ddd0023ce077f Mon Sep 17 00:00:00 2001
> From: Mikhail Klementyev<jollheef@riseup.net>
> Date: Mon, 25 May 2015 23:20:38 +0300
> Subject: [PATCH 1/1] Obsolete check is now removed.
> 
> Signed-off-by: Mikhail Klementyev<jollheef@riseup.net>
> ---
>   kernel/auditsc.c | 4 +---
>   1 file changed, 1 insertion(+), 3 deletions(-)

Applied, but you are lucky I happened to be in a good mood and your patch was 
small, normally I reject patches that arrive as HTML mail because they are a 
major headache to apply.

Contributions are always welcome, but please read the information in the 
Documentation/SubmittingPatches before submitting another patch.

> diff --git a/kernel/auditsc.c b/kernel/auditsc.c
> index 9fb9d1c..ee09794 100644
> --- a/kernel/auditsc.c
> +++ b/kernel/auditsc.c
> @@ -599,9 +599,7 @@ static int audit_filter_rules(struct task_struct *tsk,
>   				result = match_tree_refs(ctx, rule->tree);
>   			break;
>   		case AUDIT_LOGINUID:
> -			result = 0;
> -			if (ctx)
> -				result = audit_uid_comparator(tsk->loginuid, f->op, f->uid);
> +			result = audit_uid_comparator(tsk->loginuid, f->op, f->uid);
>   			break;
>   		case AUDIT_LOGINUID_SET:
>   			result = audit_comparator(audit_loginuid_set(tsk), f->op, f-
>val);
> --
> 2.0.5

-- 
paul moore
www.paul-moore.com