From: Timo Sirainen <timo.sirainen@dovecot.fi>
Date: Wed, 25 Apr 2018 12:22:37 +0000 (+0300)
Subject: lib-storage: Don't ignore uni_utf8_get_valid_data() return value
X-Git-Tag: 2.3.2.rc1~136
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=fac3484be95ddaaff8675899adf3465db0783e1d;p=thirdparty%2Fdovecot%2Fcore.git

lib-storage: Don't ignore uni_utf8_get_valid_data() return value

It's not supposed to happen, so call i_unreached()
---

diff --git a/src/lib-storage/list/mailbox-list-fs-iter.c b/src/lib-storage/list/mailbox-list-fs-iter.c
index fa90ebc8f9..63afbc83f2 100644
--- a/src/lib-storage/list/mailbox-list-fs-iter.c
+++ b/src/lib-storage/list/mailbox-list-fs-iter.c
@@ -111,8 +111,9 @@ fs_list_rename_invalid(struct fs_list_iterate_context *ctx,
 					  MAILBOX_LIST_PATH_TYPE_MAILBOX);
 	src = t_strconcat(root, "/", storage_name, NULL);
 
-	(void)uni_utf8_get_valid_data((const void *)storage_name,
-				      strlen(storage_name), destname);
+	if (uni_utf8_get_valid_data((const void *)storage_name,
+				    strlen(storage_name), destname))
+		i_unreached(); /* already checked that it was invalid */
 
 	str_append(dest, root);
 	str_append_c(dest, '/');
diff --git a/src/lib-storage/list/mailbox-list-maildir-iter.c b/src/lib-storage/list/mailbox-list-maildir-iter.c
index 5889a1f072..923101a56c 100644
--- a/src/lib-storage/list/mailbox-list-maildir-iter.c
+++ b/src/lib-storage/list/mailbox-list-maildir-iter.c
@@ -300,8 +300,9 @@ maildir_fill_readdir_entry(struct maildir_list_iterate_context *ctx,
 		string_t *destvname = t_str_new(128);
 		string_t *dest = t_str_new(128);
 
-		(void)uni_utf8_get_valid_data((const void *)fname,
-					      strlen(fname), destvname);
+		if (uni_utf8_get_valid_data((const void *)fname,
+					    strlen(fname), destvname))
+			i_unreached(); /* already checked that it was invalid */
 
 		str_append(dest, ctx->dir);
 		str_append_c(dest, '/');