3 files changed, 10 insertions, 9 deletions
diff --git a/source/html.c b/source/html.c
index 1a6b7c5..b6e62a6 100644
--- a/source/html.c
+++ b/source/html.c
@@ -247,6 +247,12 @@ ssize_t html_ntxt(const char *txt, size_t len)
 		}
 		p++;
 	}
+	// A cut inside a multibyte character would leave an invalid sequence
+	// on the page, so it backs up to where that character starts. The
+	// escaped bytes are all ASCII, so no character spans one.
+	if (left < 0)
+		while (p > txt && (*p & 0xC0) == 0x80)
+			p--;
 	if (p != txt)
 		html_raw(txt, p - txt);
 	return left;
diff --git a/source/html.h b/source/html.h
index 095804c..c6c0e19 100644
--- a/source/html.h
+++ b/source/html.h
@@ -55,8 +55,9 @@ extern void html_attrf(const char *format, ...);
 extern void html_txt(const char *txt);
 
 /*
- * Escape at most len bytes of txt. The return is what is left of that budget,
- * and a caller prints an ellipsis when it comes back negative.
+ * Escape at most len bytes of txt, cutting short of a character the budget
+ * would split. The return is what is left of that budget, and a caller prints
+ * an ellipsis when it comes back negative.
  */
 extern ssize_t html_ntxt(const char *txt, size_t len);
 
diff --git a/source/ui-shared.c b/source/ui-shared.c
index 7d28c39..f3d815f 100644
--- a/source/ui-shared.c
+++ b/source/ui-shared.c
@@ -1039,13 +1039,7 @@ void cgit_commit_link(const char *name, const char *title, const char *class,
 	html("'>");
 	if (name && name[0] != '\0') {
 		if (ctx.cfg.max_msg_len >= MIN_TRUNCATE_LEN && strlen(name) > (size_t)ctx.cfg.max_msg_len) {
-			size_t len = ctx.cfg.max_msg_len - ELLIPSIS_LEN;
-
-			// A cut inside a multibyte character would leave an
-			// invalid sequence on the page.
-			while (len > 0 && (name[len] & 0xC0) == 0x80)
-				len--;
-			html_ntxt(name, len);
+			html_ntxt(name, ctx.cfg.max_msg_len - ELLIPSIS_LEN);
 			html("...");
 		} else {
 			html_txt(name);
-rw-r--r--	source/html.c	6
-rw-r--r--	source/html.h	5
-rw-r--r--	source/ui-shared.c	8