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 file changed, 6 insertions, 1 deletion

diff --git a/ui-patch.c b/ui-patch.cindex ef79c7ce..e7a010ac 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");
diff --git a/ui-patch.c b/ui-patch.c
index ef79c7ce..e7a010ac 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");