]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
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 a42e296ee2631ed2a022851cba90c1c15b11e4fb..bb3d8d0f09db94f9124643aea04f88566ac2bf17 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;
        }