threads / patch / 30537

patchdiff --no-index: don't leak buffers in queue_diff

Subject: [PATCH] diff --no-index: don't leak buffers in queue_diff

## tl;dr

One message between May 16, 2012 and May 16, 2012. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Bobby Powers· May 16, 2012, 14:50 UTC · lore

queue_diff uses two strbufs, and at the end of the function strbuf_reset was called. This only reset the length of the buffer - any allocated memory was leaked. Using strbuf_release fixes this.

Signed-off-by: Bobby Powers <bobbypowers@gmail.com>
---
 diff-no-index.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
Show changes to diff-no-index.c +2 −2
diff --git a/diff-no-index.c b/diff-no-index.c
index 3080b66..f0b0010 100644
--- a/diff-no-index.c
+++ b/diff-no-index.c
@@ -123,8 +123,8 @@ static int queue_diff(struct diff_options *o,
 		}
 		string_list_clear(&p1, 0);
 		string_list_clear(&p2, 0);
-		strbuf_reset(&buffer1);
-		strbuf_reset(&buffer2);
+		strbuf_release(&buffer1);
+		strbuf_release(&buffer2);
 
 		return ret;
 	} else {
-- 
1.7.10.2.521.g8ddb639

← back to recent threads