CLEANUP: http-htx: Set buffer area to NULL instead of malloc(0)
authorChristopher Faulet <cfaulet@haproxy.com>
Fri, 5 Feb 2021 09:29:29 +0000 (10:29 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Fri, 5 Feb 2021 10:51:44 +0000 (11:51 +0100)
During error files conversion to HTX message, in http_str_to_htx(), if a
file is empty, the corresponding buffer's area is initialized with a
malloc(0) and its size is set to 0. There is no problem here. The behaviour
is totally defined. But it is not really intuitive. Instead, we can simply
set the area to NULL.

This patch should fix the issue #1022.

src/http_htx.c

index a42e296..bb3d8d0 100644 (file)
@@ -904,7 +904,7 @@ int http_str_to_htx(struct buffer *buf, struct ist raw, char **errmsg)
        b_reset(buf);
        if (!raw.len) {
                buf->size = 0;
-               buf->area = malloc(raw.len);
+               buf->area = NULL;
                return 1;
        }