commit 4239df3cca84ef6d17b350804eab03fa39f41d20
from: Omar Polo <op@omarpolo.com>
date: Fri Jul 12 13:13:47 2024 UTC

add a comment to remind us to switch back to malloc()

while here also fix the function name in the error message, spotted
by stsp.

commit - e6eaecdc8d833e2ee11303f5b4f08ca419a042cd
commit + 4239df3cca84ef6d17b350804eab03fa39f41d20
blob - 46993566dd31424cd682958ab880fa97a0c65211
blob + c079c5e221939a0486ad03cbb7d76fad283d376f
--- lib/object_qid.c
+++ lib/object_qid.c
@@ -31,9 +31,16 @@
 const struct got_error *
 got_object_qid_alloc_partial(struct got_object_qid **qid)
 {
+	/*
+	 * XXX(op) this should really be malloc(), but there are
+	 * strange interactions in the fileindex and worktree code
+	 * that are creating issues with some of the changes needed
+	 * for sha256 support.  This will have to be revisited once
+	 * that code is fixed.
+	 */
 	*qid = calloc(1, sizeof(**qid));
 	if (*qid == NULL)
-		return got_error_from_errno("malloc");
+		return got_error_from_errno("calloc");
 
 	(*qid)->data = NULL;
 	return NULL;