On Mon, Oct 28, 2013 at 6:35 PM, Richard Guy Briggs <rgb(a)redhat.com> wrote:
 On Mon, Oct 28, 2013 at 04:31:30PM -0700, William Roberts wrote:
 > On Mon, Oct 28, 2013 at 4:30 PM, William Roberts
 > <bill.c.roberts(a)gmail.com>wrote:
 >
 > > I've been working off of Richard Guy Brigs git repo on branch
 > > audit-for-next prepping my patch and I noticed a build warning:
 > >
 > > kernel/audit.c:832:8: warning: format ‘%A’ expects argument of type
 > > ‘double’, but argument 3 has type ‘char *’ [-Wformat]
 > >
 > > Looking at the code, it looks wrong:
 > >
 > >                                 audit_log_format(ab,
 >                                                  "
 > msg='%.AUDIT_MESSAGE_TEXT_MAXs'",
 >                                                  (char *)data);
 >
 > The issue appears on the % specifier in there, it picks it up as %.A,
 which
 > is of type double. Is this what was intended?
 Hmmm, that should have picked up a macro from 06051fbe in
 audit-for-next.  It should be pre-processed to "%.8560s".
 > William C Roberts
 - RGB
 
The qoutes are wrong for that.
 --
 Richard Guy Briggs <rbriggs(a)redhat.com>
 Senior Software Engineer
 Kernel Security
 AMER ENG Base Operating Systems
 Remote, Ottawa, Canada
 Voice: +1.647.777.2635
 Internal: (81) 32635
 Alt: +1.613.693.0684x3545
 
-- 
Respectfully,
William C Roberts