shallow: handle missing shallow commits gracefully

dscho · dscho · commit 1f4d57216e7a · 2026-01-20T12:15:45.000+01:00
As pointed out by CodeQL, `lookup_commit()` can return NULL.

Signed-off-by: Johannes Schindelin &lt;johannes.schindelin@gmx.de&gt;
diff --git a/shallow.c b/shallow.c
@@ -703,7 +703,8 @@ void assign_shallow_commits_to_refs(struct shallow_info *info,
 	for (i = 0; i < nr_shallow; i++) {
 		struct commit *c = lookup_commit(the_repository,
 						 &oid[shallow[i]]);
-		c->object.flags |= BOTTOM;
+		if (c)
+			c->object.flags |= BOTTOM;
 	}
 
 	for (i = 0; i < ref->nr; i++)

Original file line number	Diff line number	Diff line change
`@@ -703,7 +703,8 @@ void assign_shallow_commits_to_refs(struct shallow_info *info,`
`703`	`703`	`for (i = 0; i < nr_shallow; i++) {`
`704`	`704`	`struct commit *c = lookup_commit(the_repository,`
`705`	`705`	`&oid[shallow[i]]);`
`706`		`- c->object.flags \|= BOTTOM;`
	`706`	`+ if (c)`
	`707`	`+ c->object.flags \|= BOTTOM;`
`707`	`708`	`}`
`708`	`709`
`709`	`710`	`for (i = 0; i < ref->nr; i++)`