]> git.ipfire.org Git - thirdparty/dovecot/core.git/commitdiff
lib-master: Don't dup errors to service->set_pool in master_service_settings_read
authorMartti Rannanjärvi <martti.rannanjarvi@dovecot.fi>
Thu, 15 Feb 2018 15:50:43 +0000 (17:50 +0200)
committerMartti Rannanjärvi <martti.rannanjarvi@dovecot.fi>
Thu, 15 Feb 2018 16:01:03 +0000 (18:01 +0200)
Use t_strdup() instead.

src/lib-master/master-service-settings.c

index 11cc5781e6cf83fb6116fad1053b25b74a49c4e4..052b471380fe9bf1af20b9b86590a94cef23cab9 100644 (file)
@@ -518,7 +518,7 @@ int master_service_settings_read(struct master_service *service,
                                ret = settings_parse_stream_read(parser,
                                                                 istream);
                                if (ret < 0)
-                                       *error_r = p_strdup(service->set_pool,
+                                       *error_r = t_strdup(
                                                settings_parser_get_error(parser));
                        }
                        alarm(0);
@@ -534,7 +534,7 @@ int master_service_settings_read(struct master_service *service,
 
                if (ret != 0) {
                        if (ret > 0) {
-                               *error_r = p_strdup_printf(service->set_pool,
+                               *error_r = t_strdup_printf(
                                        "Timeout reading config from %s", path);
                        }
                        i_close_fd(&fd);
@@ -555,8 +555,7 @@ int master_service_settings_read(struct master_service *service,
 
        if (use_environment || service->keep_environment) {
                if (settings_parse_environ(parser) < 0) {
-                       *error_r = p_strdup(service->set_pool,
-                                       settings_parser_get_error(parser));
+                       *error_r = t_strdup(settings_parser_get_error(parser));
                        settings_parser_deinit(&parser);
                        return -1;
                }
@@ -571,8 +570,7 @@ int master_service_settings_read(struct master_service *service,
        }
 
        if (!settings_parser_check(parser, service->set_pool, &error)) {
-               *error_r = p_strdup_printf(service->set_pool,
-                               "Invalid settings: %s", error);
+               *error_r = t_strdup_printf("Invalid settings: %s", error);
                settings_parser_deinit(&parser);
                return -1;
        }