surround: Fix unbalanced surround handling

We should skip surround handling if we don't find the ending surround part. Fixes #17 (comment)
karlb · Feb 2, 2025 · 4fe5afa · 4fe5afa
1 parent 4789e62
commit 4fe5afa
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/smu.c b/smu.c
@@ -653,8 +653,9 @@ dosurround(const char *begin, const char *end, int newblock) {
 			stop = p;
 			p = strstr(p + 1, surround[i].search);
 		} while (p && p[-1] == '\\');
-		if (p && p[-1] != '\\')
-			stop = p;
+		if (!p || p[-1] == '\\')  /* No unescaped closing marker found */
+			continue;
+		stop = p;
 		if (!stop || stop < start || stop >= end)
 			continue;
 		fputs(surround[i].before, stdout);

diff --git a/tests/surround.html b/tests/surround.html
@@ -0,0 +1 @@
+<p>hello_world</p>
diff --git a/tests/surround.text b/tests/surround.text
@@ -0,0 +1 @@
+hello_world