fix: check for NULL return from g_uri_escape_string in escape_url

andy5995 · claude · andy5995 · commit fa4961e9223c · 2026-04-30T15:02:10.000-05:00
g_uri_escape_string returns NULL for invalid UTF-8 input; guard it with
fatal_malloc() to match the existing check in unescape_url.

Co-Authored-By: Claude Sonnet 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/src/utils.c b/src/utils.c
@@ -180,7 +180,10 @@ user_verify(void)
 char *
 escape_url(const char *str)
 {
-  return (char *) g_uri_escape_string(str, "/", FALSE);
+  char *result = (char *) g_uri_escape_string(str, "/", FALSE);
+  if (!result)
+    fatal_malloc();
+  return result;
 }
 
 

Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,10 @@ user_verify(void)`
`180`	`180`	`char *`
`181`	`181`	`escape_url(const char *str)`
`182`	`182`	`{`
`183`		`- return (char *) g_uri_escape_string(str, "/", FALSE);`
	`183`	`+ char result = (char ) g_uri_escape_string(str, "/", FALSE);`
	`184`	`+ if (!result)`
	`185`	`+ fatal_malloc();`
	`186`	`+ return result;`
`184`	`187`	`}`
`185`	`188`
`186`	`189`