[PATCH 3/4] line-log: initialize diff queue in process_ranges_ordinary_commit()

From: SZEDER Gábor <szeder.dev@gmail.com>
Date: Aug 24, 2025, 19:06 UTC
Message-ID: <20250824190644.2573279-4-szeder.dev@gmail.com>
In-Reply-To: <20250824190644.2573279-1-szeder.dev@gmail.com>

process_ranges_ordinary_commit() uses a local diff queue variable, which it leaves uninitialized before passing its address to queue_diffs(). This is not an issue, because at the end of that function the contents of an other diff queue is moved into it by simply overwriting whatever is in there, i.e. without reading any uninitialized memory.

Still, seeing the uninitialized diff queue being passed around scared me more than once, so out of caution let's make sure that it's initialized.

Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
---
 line-log.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/line-log.c b/line-log.c
index b2a31ae956..71fa857ee8 100644
--- a/line-log.c
+++ b/line-log.c
@@ -1182,7 +1182,7 @@ static int process_ranges_ordinary_commit(struct rev_info *rev, struct commit *c
 					  struct line_log_data *range)
 {
 	struct commit *parent = NULL;
-	struct diff_queue_struct queue;
+	struct diff_queue_struct queue = DIFF_QUEUE_INIT;
 	struct line_log_data *parent_range;
 	int changed;

-- 
2.51.0.433.g1a66b3fb12