BUG/MINOR: log: fixing small memory leak in error code path.

since we do not log the sample fetch when it is invalid, we can
free the log data.
This commit is contained in:
David Carlier 2017-09-21 14:36:43 +00:00 committed by Willy Tarreau
parent 6fb4ba38e0
commit 93e8b88f06

View File

@ -484,6 +484,8 @@ int add_sample_to_logformat_list(char *text, char *arg, int arg_len, struct prox
node->options |= LOG_OPT_RES_CAP; /* fetch method is response-compatible */ node->options |= LOG_OPT_RES_CAP; /* fetch method is response-compatible */
if (!(expr->fetch->val & cap)) { if (!(expr->fetch->val & cap)) {
free(node);
node = NULL;
memprintf(err, "sample fetch <%s> may not be reliably used here because it needs '%s' which is not available here", memprintf(err, "sample fetch <%s> may not be reliably used here because it needs '%s' which is not available here",
text, sample_src_names(expr->fetch->use)); text, sample_src_names(expr->fetch->use));
return 0; return 0;