]> git.kaiwu.me - haproxy.git/commitdiff
[CLEANUP] config: do not allocate an empty argument for rsp* keywords
authorWilly Tarreau <w@1wt.eu>
Thu, 28 Jan 2010 21:31:02 +0000 (22:31 +0100)
committerWilly Tarreau <w@1wt.eu>
Thu, 28 Jan 2010 22:17:16 +0000 (23:17 +0100)
This was a harmless copy-paste typo. Empty parameters are strduped
and passed to chain_regex(). This was spotted and fixed in 1.4
commit ade5ec430d01b4ba06fd67a63fcee85fea46ba28.

src/cfgparse.c

index 1c1452dd8143c170895854bbebad9b4ce6d66fb2..84d2797b2c787b2ee94aa2019eaa4b8eb50e8abf 100644 (file)
@@ -3405,7 +3405,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int inv)
                        goto out;
                }
        
-               err = chain_regex(&curproxy->rsp_exp, preg, ACT_REMOVE, strdup(args[2]));
+               err = chain_regex(&curproxy->rsp_exp, preg, ACT_REMOVE, NULL);
                if (err) {
                        Alert("parsing [%s:%d] : invalid character or unterminated sequence in replacement string near '%c'.\n",
                              file, linenum, *err);
@@ -3436,7 +3436,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int inv)
                        goto out;
                }
        
-               err = chain_regex(&curproxy->rsp_exp, preg, ACT_DENY, strdup(args[2]));
+               err = chain_regex(&curproxy->rsp_exp, preg, ACT_DENY, NULL);
                if (err) {
                        Alert("parsing [%s:%d] : invalid character or unterminated sequence in replacement string near '%c'.\n",
                              file, linenum, *err);
@@ -3499,7 +3499,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int inv)
                        goto out;
                }
        
-               err = chain_regex(&curproxy->rsp_exp, preg, ACT_REMOVE, strdup(args[2]));
+               err = chain_regex(&curproxy->rsp_exp, preg, ACT_REMOVE, NULL);
                if (err) {
                        Alert("parsing [%s:%d] : invalid character or unterminated sequence in replacement string near '%c'.\n",
                              file, linenum, *err);
@@ -3530,7 +3530,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int inv)
                        goto out;
                }
        
-               err = chain_regex(&curproxy->rsp_exp, preg, ACT_DENY, strdup(args[2]));
+               err = chain_regex(&curproxy->rsp_exp, preg, ACT_DENY, NULL);
                if (err) {
                        Alert("parsing [%s:%d] : invalid character or unterminated sequence in replacement string near '%c'.\n",
                              file, linenum, *err);