Subject: [PATCH] Don't coredump on bad refs in update-server-info.

From: "Shawn O. Pearce" <[email-masked]>
Date: Wed, 31 Jan 2007 02:24:44 -0500

Apparently if we are unable to parse an object update-server-info
coredumps, as it doesn't bother to check the return value of its
call to parse_object.

Instead of coredumping, skip the ref.

Signed-off-by: Shawn O. Pearce <[email-masked]>
---

 I don't know how often folks might run into this.  I intentionally
 broke pack reading in one of my topics and discovered that
 update-server-info crashed with a segfault when the pack access code
 was unable to decompress the object behind the ref.  That's less
 than optimal.

 server-info.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/server-info.c b/server-info.c
index 6cd38be..f9be5a7 100644
--- a/server-info.c
+++ b/server-info.c
@@ -10,6 +10,8 @@ static FILE *info_ref_fp;
 static int add_info_ref(const char *path, const unsigned char *sha1, int flag, void *cb_data)
 {
 	struct object *o = parse_object(sha1);
+	if (!o)
+		return -1;
 
 	fprintf(info_ref_fp, "%s	%s\n", sha1_to_hex(sha1), path);
 	if (o->type == OBJ_TAG) {
-- 
1.5.0.rc2.81.g73a2