Typo was introduced in 57bc891 ("BUG/MEDIUM: log: fix risk of
segfault when logging HTTP fields in TCP mode") which inverted the
condition in the test and caused <BADREQ> to be logged when using
%HP.

Signed-off-by: Nenad Merdanovic <nmer...@anine.io>
---
 src/log.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/log.c b/src/log.c
index dffef54..2d02247 100644
--- a/src/log.c
+++ b/src/log.c
@@ -1917,7 +1917,7 @@ int build_logline(struct stream *s, char *dst, size_t 
maxsize, struct list *list
                                while (spc < end && *spc != '?' && 
!HTTP_IS_SPHT(*spc))
                                        spc++;
 
-                               if (!txn || txn->uri || nspaces == 0) {
+                               if (!txn || !txn->uri || nspaces == 0) {
                                        chunk.str = "<BADREQ>";
                                        chunk.len = strlen("<BADREQ>");
                                } else {
-- 
2.7.0


Reply via email to