http: clear header pointer on realloc failure

Fixes:

detect-engine-hhd.c:188:5: warning: Use of memory after it is freed
    return headers_buffer;

diff --git a/src/detect-engine-hhd.c b/src/detect-engine-hhd.c
index 759c826a022ad3a11dc5449ccf2598b8bac9fc7a..757da0b1bfdc3be31ac12f5a09a144f9c7e93dbe 100644 (file)
--- a/src/detect-engine-hhd.c
+++ b/src/detect-engine-hhd.c
@@ -158,8 +158,10 @@ static uint8_t *DetectEngineHHDGetBufferForTX(htp_tx_t *tx, uint64_t tx_id,
         /* the extra 4 bytes if for ": " and "\r\n" */
         uint8_t *new_headers_buffer = SCRealloc(headers_buffer, headers_buffer_len + size1 + size2 + 4);
         if (unlikely(new_headers_buffer == NULL)) {
-            if (headers_buffer != NULL)
+            if (headers_buffer != NULL) {
                 SCFree(headers_buffer);
+                headers_buffer = NULL;
+            }
             det_ctx->hhd_buffers[index] = NULL;
             det_ctx->hhd_buffers_len[index] = 0;
             goto end;