]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
[BUG] buffers: remove remains of wrong obsolete length check
authorWilly Tarreau <w@1wt.eu>
Fri, 15 Jan 2010 22:38:27 +0000 (23:38 +0100)
committerWilly Tarreau <w@1wt.eu>
Fri, 15 Jan 2010 22:38:27 +0000 (23:38 +0100)
A check was performed in buffer_replace2() to compare buffer
length with its read pointer. This has been wrong for a long
time, though it only has an impact when dealing with keep-alive
requests/responses. In theory this should be backported but
the check has no impact without keep-alive.

src/buffers.c

index f9b5aa46bb5c3ab38658357a0a28755483006b4f..4f14507396bd15b4134aff7fff63b918334627df 100644 (file)
@@ -230,12 +230,6 @@ int buffer_replace2(struct buffer *b, char *pos, char *end, const char *str, int
        if (delta + b->r >= b->data + b->size)
                return 0;  /* no space left */
 
-       if (b->data + b->l < end) {
-               /* The data has been stolen, we could have crashed.
-                * Maybe we should abort() ? */
-               return 0;
-       }
-
        /* first, protect the end of the buffer */
        memmove(end + delta, end, b->r - end);