From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Paris Subject: Re: [PATCH] Fix the bug of using AUDIT_STATUS_RATE_LIMIT when set fail, no error output. Date: Wed, 30 Jul 2008 23:15:50 -0400 Message-ID: <1217474150.2902.76.camel@localhost.localdomain> References: <48911F47.50606@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <48911F47.50606@cn.fujitsu.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-audit-bounces@redhat.com Errors-To: linux-audit-bounces@redhat.com To: zhangxiliang Cc: Linux Kernel Mailing List , viro@zeniv.linux.org.uk, Linux Audit List-Id: linux-audit@redhat.com On Thu, 2008-07-31 at 10:11 +0800, zhangxiliang wrote: > When the "status_get->mask" is "AUDIT_STATUS_RATE_LIMIT || AUDIT_STATUS_BACKLOG_LIMIT". > If "audit_set_rate_limit" fails and "audit_set_backlog_limit" succeeds, the "err" value will be greater than or equal to 0. It will miss the failure of rate set. > > Signed-off-by: Zhang Xiliang man, it gives me the heebee jeebies with the coding style but it follows everything else Acked-by: Eric Paris > --- > kernel/audit.c | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/kernel/audit.c b/kernel/audit.c > index e092f1c..38a4080 100644 > --- a/kernel/audit.c > +++ b/kernel/audit.c > @@ -725,9 +725,11 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh) > audit_pid = new_pid; > audit_nlk_pid = NETLINK_CB(skb).pid; > } > - if (status_get->mask & AUDIT_STATUS_RATE_LIMIT) > + if (status_get->mask & AUDIT_STATUS_RATE_LIMIT) { > err = audit_set_rate_limit(status_get->rate_limit, > loginuid, sessionid, sid); > + if (err < 0) return err; > + } > if (status_get->mask & AUDIT_STATUS_BACKLOG_LIMIT) > err = audit_set_backlog_limit(status_get->backlog_limit, > loginuid, sessionid, sid); > -- > 1.5.4.2 > >