From abcf8573005d548fe213aa7b4579394fa18692a1 Mon Sep 17 00:00:00 2001 From: Denton Liu Date: Fri, 6 Dec 2019 12:16:31 -0800 Subject: [PATCH] range-diff: clear `other_arg` at end of function We were leaking memory by not clearing `other_arg` after we were done using it. Clear it after we've finished using it. Note that this isn't strictly necessary since the memory will be reclaimed once the command exits. However, since we are releasing the strbufs, we should also clear `other_arg` for consistency. Signed-off-by: Denton Liu Signed-off-by: Junio C Hamano --- builtin/range-diff.c | 1 + 1 file changed, 1 insertion(+) diff --git a/builtin/range-diff.c b/builtin/range-diff.c index 98acf3533e..d8a4670629 100644 --- a/builtin/range-diff.c +++ b/builtin/range-diff.c @@ -84,6 +84,7 @@ int cmd_range_diff(int argc, const char **argv, const char *prefix) res = show_range_diff(range1.buf, range2.buf, creation_factor, simple_color < 1, &diffopt, &other_arg); + argv_array_clear(&other_arg); strbuf_release(&range1); strbuf_release(&range2); -- 2.32.0.93.g670b81a890