mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-08-09 16:47:18 +02:00
Code Issues Projects Releases Wiki Activity

BUG/MINOR: cache: Don't needlessly test "cache" keyword in parse_cache_flt()

Browse Source 
parse_cache_flt() is the registered callback for the "cache" filter keyword. It
is only called when the "cache" keyword is found on a filter line. So, it is
useless to test the filter name in the callback function.

This patch should fix the issue #634. It may be backported as far as 1.9.
This commit is contained in:
Christopher Faulet 2020-05-18 11:58:16 +02:00
parent 3b78809de0
commit 2a37cdbe6b
1 changed files with 10 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
src/cache.c
View File

@ -1411,19 +1411,17 @@ parse_cache_flt(char **args, int *cur_arg, struct proxy *px,
char *name = NULL; char *name = NULL;
int pos = *cur_arg; int pos = *cur_arg;
/* Get the cache filter name*/ /* Get the cache filter name. <pos> point on "cache" keyword */
if (!strcmp(args[pos], "cache")) { if (!*args[pos + 1]) {
if (!*args[pos + 1]) { memprintf(err, "%s : expects an <id> argument", args[pos]);
memprintf(err, "%s : expects an <id> argument", args[pos]); goto error;
goto error;
}
name = strdup(args[pos + 1]);
if (!name) {
memprintf(err, "%s '%s' : out of memory", args[pos], args[pos + 1]);
goto error;
}
pos += 2;
} }
name = strdup(args[pos + 1]);
if (!name) {
memprintf(err, "%s '%s' : out of memory", args[pos], args[pos + 1]);
goto error;
}
pos += 2;
/* Check if an implicit filter with the same name already exists. If so, /* Check if an implicit filter with the same name already exists. If so,
* we remove the implicit filter to use the explicit one. */ * we remove the implicit filter to use the explicit one. */