Commit 1e49f12793c4d39fffb64a08db0d1fddafaf0338

lhchavez 2021-08-31T05:23:20

Fix a possible null pointer dereference This change fixes a possible null pointer dereference if anything inside `git_commit_graph_writer_add_index_file` fails before the packfile being valid. https://scan6.coverity.com/reports.htm#v52218/p10377/fileInstanceId=122935896&defectInstanceId=32525576&mergedDefectId=1461634

1
2
3
4
5
6
7
8
9
10
11
12
13
14
diff --git a/src/commit_graph.c b/src/commit_graph.c
index 9abe373..71e415e 100644
--- a/src/commit_graph.c
+++ b/src/commit_graph.c
@@ -716,7 +716,8 @@ int git_commit_graph_writer_add_index_file(
 		goto cleanup;
 
 cleanup:
-	git_mwindow_put_pack(p);
+	if (p)
+		git_mwindow_put_pack(p);
 	git_odb_free(state.db);
 	return error;
 }