BUG/MINOR: log: Don't dump empty unique-id

If the unique-id value is missing, the build_logline() function dump
anything. It is because the function lf_text() is bypassed. This
function is responsible to dump '-' is the value is not present, and set
the '"' around the value displayed.

This fixes the bug reported by Julient Vehent
This commit is contained in:
Thierry FOURNIER 2014-04-15 01:38:48 +02:00 committed by Willy Tarreau
parent ad9035186e
commit 1be69105ab

View File

@ -1559,8 +1559,7 @@ int build_logline(struct session *s, char *dst, size_t maxsize, struct list *lis
case LOG_FMT_UNIQUEID: // %ID
ret = NULL;
src = s->unique_id;
if (src)
ret = lf_text(tmplog, src, maxsize - (tmplog - dst), tmp);
ret = lf_text(tmplog, src, maxsize - (tmplog - dst), tmp);
if (ret == NULL)
goto out;
tmplog = ret;