Pablo, It fixed the strdup(). But now: cc1: warnings being treated as errors read_config_lex.c: In function 'yylex': read_config_lex.c:3846: warning: label 'find_rule' defined but not used make[1]: *** [read_config_lex.o] Error 1 if I strip this label (find_rule), then I get: read_config_lex.c:3593: warning: 'yy_full_match' defined but not used make[1]: *** [read_config_lex.o] Error 1 make[1]: Leaving directory `/usr/local/src/conntrack-tools/src' make: *** [all-recursive] Error 1 If I remove this function (yy_full_match), then I get: cc1: warnings being treated as errors read_config_lex.c: In function 'yylex': read_config_lex.c:3782: warning: 'yy_act' may be used uninitialized in this function make[1]: *** [read_config_lex.o] Error 1 If I initialize this var (register int yy_act = 0;), then I get: everything compile now. I'm using a version I compiled without the -Werror flag, it was more clean. Nick On Wed, May 21, 2008 at 10:04 AM, Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx> wrote: > NICOLAS BOULIANE wrote: >> Hi, >> >> thank you for your comments. >> --- >> >> Pablo: >> >> cc1: warnings being treated as errors >> read_config_lex.l: In function 'yylex': >> read_config_lex.l:113: warning: incompatible implicit declaration of >> built-in function 'strdup' >> read_config_lex.l:114: warning: incompatible implicit declaration of >> built-in function 'strdup' >> read_config_lex.l:115: warning: incompatible implicit declaration of >> built-in function 'strdup' >> read_config_lex.l:126: warning: incompatible implicit declaration of >> built-in function 'strdup' >> read_config_lex.c:3845: warning: label 'find_rule' defined but not used >> make[1]: *** [read_config_lex.o] Error 1 >> >> I get this while compiling the last git snapshot >> probably string.h is missing ? > > Does the patch attached fix it? > > -- > "Los honestos son inadaptados sociales" -- Les Luthiers > > diff --git a/src/read_config_lex.l b/src/read_config_lex.l > index eb3368a..7daaeab 100644 > --- a/src/read_config_lex.l > +++ b/src/read_config_lex.l > @@ -19,6 +19,8 @@ > * Description: configuration file syntax > */ > > +#include <string.h> > + > #include "read_config_yy.h" > %} > > > -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
