Check for NULL-subject in patch view

While at it, make sure the commit message ends with a '\n'.

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

1 files changed, 6 insertions, 1 deletions

diff --git a/ui-patch.c b/ui-patch.c
index ef79c7c..e7a010a 100644
--- a/ui-patch.c
+++ b/ui-patch.c
@@ -96,7 +96,12 @@ void cgit_print_patch(char *hex, struct cacheitem *item)
 	htmlf("From: %s%s\n", info->author, info->author_email);
 	html("Date: ");
 	cgit_print_date(info->author_date, "%a, %d  %b  %Y  %H:%M:%S  %z%n");
-	htmlf("Subject: %s\n\n%s", info->subject, info->msg);
+	htmlf("Subject: %s\n\n", info->subject);
+	if (info->msg && *info->msg) {
+		htmlf("%s", info->msg);
+		if (info->msg[strlen(info->msg) - 1] != '\n')
+			html("\n");
+	}
 	html("---\n");
 	cgit_diff_tree(old_sha1, sha1, filepair_cb, NULL);
 	html("--\n");