Make print_branch() handle refs not pointing at commits

The master branch of stable/linux-2.6.20 currently references a tag
object, which makes print_branch() die with a segfault. This teaches
print_branch() to handle such cases more gracefully.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>

1 files changed, 11 insertions, 5 deletions

diff --git a/ui-summary.c b/ui-summary.c
index 04a466a..ba90510 100644
--- a/ui-summary.c
+++ b/ui-summary.c
@@ -49,29 +49,35 @@ static int cmp_tag_age(const void *a, const void *b)
 }
 
 static int print_branch(struct refinfo *ref)
 {
         struct commitinfo *info = ref->commit;
         char *name = (char *)ref->refname;
 
         if (!info)
                 return 1;
         html("<tr><td>");
         cgit_log_link(name, NULL, NULL, name, NULL, NULL, 0);
         html("</td><td>");
-        cgit_print_age(info->commit->date, -1, NULL);
-        html("</td><td>");
-        html_txt(info->author);
-        html("</td><td>");
-        cgit_commit_link(info->subject, NULL, NULL, name, NULL);
+
+        if (ref->object->type == OBJ_COMMIT) {
+                cgit_print_age(info->commit->date, -1, NULL);
+                html("</td><td>");
+                html_txt(info->author);
+                html("</td><td>");
+                cgit_commit_link(info->subject, NULL, NULL, name, NULL);
+        } else {
+                html("</td><td></td><td>");
+                cgit_object_link(ref->object);
+        }
         html("</td></tr>\n");
         return 0;
 }
 
 static void print_tag_header()
 {
         html("<tr class='nohover'><th class='left'>Tag</th>"
              "<th class='left'>Age</th>"
              "<th class='left'>Author</th>"
              "<th class='left'>Reference</th></tr>\n");
         header = 1;
 }