diff options
author | Pablo Neira Ayuso <pablo@netfilter.org> | 2017-02-01 18:03:57 +0100 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2017-02-12 15:00:18 +0100 |
commit | ff843bcb3b9059e95e12630faad322809bde33de (patch) | |
tree | 396573be09aa925d1eccdd7a97a9358c371fe7eb | |
parent | 832166dc4780e259424ebab07b42ecf42cde6fe3 (diff) |
conntrackd: config: Do not strdup() tokens
We can just remove the strdup() from the lexer, given that we always
copy these strings in the parser.
Reported-by: Kevin Cernekee <cernekee@chromium.org>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
-rw-r--r-- | src/read_config_lex.l | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/read_config_lex.l b/src/read_config_lex.l index 0282534..a378269 100644 --- a/src/read_config_lex.l +++ b/src/read_config_lex.l @@ -141,9 +141,9 @@ notrack [N|n][O|o][T|t][R|r][A|a][C|c][K|k] {is_off} { return T_OFF; } {integer} { yylval.val = atoi(yytext); return T_NUMBER; } {signed_integer} { yylval.val = atoi(yytext); return T_SIGNED_NUMBER; } -{ip4} { yylval.string = strdup(yytext); return T_IP; } -{ip6} { yylval.string = strdup(yytext); return T_IP; } -{path} { yylval.string = strdup(yytext); return T_PATH_VAL; } +{ip4} { yylval.string = yytext; return T_IP; } +{ip6} { yylval.string = yytext; return T_IP; } +{path} { yylval.string = yytext; return T_PATH_VAL; } {alarm} { return T_ALARM; } {persistent} { dlog(LOG_WARNING, "Now `persistent' mode " "is called `alarm'. Please, update " @@ -155,7 +155,7 @@ notrack [N|n][O|o][T|t][R|r][A|a][C|c][K|k] "your conntrackd.conf file.\n"); return T_FTFW; } {notrack} { return T_NOTRACK; } -{string} { yylval.string = strdup(yytext); return T_STRING; } +{string} { yylval.string = yytext; return T_STRING; } {comment} ; {ws} ; |