lib-storage: Fix potential crash when userdb-returns "key+=value" for…

… a nonexistent setting. type would have also happened to be initialized to SET_STR for the crash to happen.
dovecot · Sep 13, 2016 · 79fff45 · 79fff45
1 parent 9474b66
commit 79fff45
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/lib-storage/mail-storage-service.c b/src/lib-storage/mail-storage-service.c
@@ -175,7 +175,7 @@ static int set_line(struct mail_storage_service_ctx *ctx,
 		enum setting_type type;
 
 		value = settings_parse_get_value(set_parser, key, &type);
-		if (type == SET_STR) {
+		if (value != NULL && type == SET_STR) {
 			const char *const *strp = value;
 
 			line = t_strdup_printf("%s=%s%s",