about summary refs log tree commit diff
path: root/ui-patch.c
diff options
context:
space:
mode:
authorLars Hjemli <hjemli@gmail.com>2008-03-17 23:13:16 +0100
committerLars Hjemli <hjemli@gmail.com>2008-03-17 23:51:47 +0100
commite01f1403b9b501c563b1fe0e0052460a95945528 (patch)
treeeff9fd11272714db70bf2ac7116511f03d800af6 /ui-patch.c
parentUse GIT-1.5.4.4 (diff)
downloadcgit-pink-e01f1403b9b501c563b1fe0e0052460a95945528.tar.gz
cgit-pink-e01f1403b9b501c563b1fe0e0052460a95945528.zip
Fix segfault in patch view for root commit
The code for patch view assumed the current commit would always have a parent,
which made cgit segfault when that wasn't the case.

This fixes the bug and adds a test-script for patch view which includes a test
for the inital commit.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Diffstat (limited to 'ui-patch.c')
-rw-r--r--ui-patch.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/ui-patch.c b/ui-patch.c
index e7a010a..7ee2c41 100644
--- a/ui-patch.c
+++ b/ui-patch.c
@@ -88,7 +88,11 @@ void cgit_print_patch(char *hex, struct cacheitem *item)
 		return;
 	}
 	info = cgit_parse_commit(commit);
-	hashcpy(old_sha1, commit->parents->item->object.sha1);
+
+	if (commit->parents && commit->parents->item)
+		hashcpy(old_sha1, commit->parents->item->object.sha1);
+	else
+		hashclr(old_sha1);
 
 	patchname = fmt("%s.patch", sha1_to_hex(sha1));
 	cgit_print_snapshot_start("text/plain", patchname, item);