[apparmor] [PATCH] quiet warnings from cache update

Kees Cook kees at ubuntu.com
Wed Sep 15 20:24:21 BST 2010


On Wed, Sep 15, 2010 at 12:16:01PM -0700, Steve Beattie wrote:
> Heh, before committing, John added the extern declaration for
> update_mru_tstamp().  I noticed this while merging the caching
> patches to the 2.5 branch, and did not include the duplicate entry.
> Here's a trivial patch to pull the second declaration back out:
> 
> === modified file 'parser/parser.h'
> --- parser/parser.h	2010-09-14 19:37:59 +0000
> +++ parser/parser.h	2010-09-15 19:11:38 +0000
> @@ -243,7 +243,6 @@
>  extern struct timespec mru_tstamp;
>  extern void update_mru_tstamp(FILE *file);
>  extern void pwarn(char *fmt, ...) __attribute__((__format__(__printf__, 1, 2)));
> -extern void update_mru_tstamp(FILE *file);
>  
>  extern FILE *yyin;
>  extern void yyrestart(FILE *fp);

*face palm* ACK. Committed to trunk.

-Kees

-- 
Kees Cook
Ubuntu Security Team



More information about the AppArmor mailing list