]> git.ipfire.org Git - thirdparty/dovecot/core.git/commitdiff
global: Use settings_defaults_dup() where possible
authorTimo Sirainen <timo.sirainen@open-xchange.com>
Tue, 10 Sep 2024 14:27:59 +0000 (17:27 +0300)
committerAki Tuomi <aki.tuomi@open-xchange.com>
Fri, 17 Jan 2025 08:39:59 +0000 (10:39 +0200)
src/lib-dict-extra/dict-client.c
src/lib-dict/dict-redis.c
src/lib-fs/fs-posix.c
src/lib-sql/driver-cassandra.c
src/lib-sql/driver-mysql.c
src/lib-sql/driver-pgsql.c
src/lib-sql/driver-sqlite.c
src/stats/client-reader.c

index 86d98017d2f4e283e80ef413c46f4fc2ccba7dba..481688158c9499ebaaee52c0d8c82bd2ad6ac19a 100644 (file)
@@ -744,9 +744,7 @@ client_dict_init_legacy(struct dict *dict_driver, const char *uri,
        const char *error;
 
        pool_t pool = pool_alloconly_create("dict_proxy_settings", 128);
-       set = p_new(pool, struct dict_proxy_settings, 1);
-       *set = dict_proxy_default_settings;
-       set->pool = pool;
+       set = settings_defaults_dup(pool, &dict_proxy_setting_parser_info);
 
        /* uri = [idle_timeout=<n>:] [slow_warn=<n>:] [<path>] ":" <uri> */
        for (;;) {
index 110ead9f352569d0bcf255a132a51314f61c7ca1..f2c623e994ff54c99079f393a04191b1b548d06a 100644 (file)
@@ -483,9 +483,7 @@ redis_dict_init_legacy(struct dict *dict_driver, const char *uri,
 {
        pool_t pool = pool_alloconly_create("redis_settings", 128);
        struct dict_redis_settings *set =
-               p_new(pool, struct dict_redis_settings, 1);
-       *set = redis_default_settings;
-       set->pool = pool;
+               settings_defaults_dup(pool, &redis_setting_parser_info);
        if (net_addr2ip(set->redis_host, &set->redis_ip) < 0)
                i_unreached();
 
index 6548c9cbdc1eb526a7eda7412680706240e98549..e39f8ed47c30ed0ad8f66a62bd5c8502c8845e8b 100644 (file)
@@ -167,9 +167,7 @@ fs_posix_legacy_init(struct fs *_fs, const char *args,
 
        pool_t pool = pool_alloconly_create("fs_posix_settings", 128);
        struct fs_posix_settings *set =
-               p_new(pool, struct fs_posix_settings, 1);
-       *set = fs_posix_default_settings;
-       set->pool = pool;
+               settings_defaults_dup(pool, &fs_posix_setting_parser_info);
        set->parsed_lock_method = FS_POSIX_LOCK_METHOD_FLOCK;
        fs->set = set;
 
index 2ab46ea7a41997b5f7f8b2af9097a82ea39ff7d9..04a8230349e6df89834ec409f92d27d83d20a466 100644 (file)
@@ -857,9 +857,7 @@ driver_cassandra_parse_connect_string(pool_t pool, const char *connect_string,
        bool read_fallback_set = FALSE, write_fallback_set = FALSE;
        bool delete_fallback_set = FALSE;
 
-       set = p_new(pool, struct cassandra_settings, 1);
-       *set = cassandra_default_settings;
-       set->pool = pool;
+       set = settings_defaults_dup(pool, &cassandra_setting_parser_info);
 
        struct ssl_settings *ssl_set = p_new(pool, struct ssl_settings, 1);
        *ssl_set = ssl_default_settings;
index afc90e682adbcff28732fa8dea4d15add287965a..88c2f44409ae453a201c7eb1e4aaa371ec1727cb 100644 (file)
@@ -298,13 +298,8 @@ driver_mysql_parse_connect_string(pool_t pool, const char *connect_string,
        const char *const *args, *name, *value;
        const char **field;
 
-       set = p_new(pool, struct mysql_settings, 1);
-       *set = mysql_default_settings;
-       set->pool = pool;
-
-       ssl_set = p_new(pool, struct ssl_settings, 1);
-       *ssl_set = ssl_default_settings;
-       ssl_set->pool = pool;
+       set = settings_defaults_dup(pool, &mysql_setting_parser_info);
+       ssl_set = settings_defaults_dup(pool, &ssl_setting_parser_info);
 
        ssl_set->ssl_cipher_list = "HIGH";
 
index 39153cd6e06d2c83049e626ef05741e18289479d..300c9e5df2f8bf8605d9fd040e9bafe4c11cff92 100644 (file)
@@ -500,9 +500,8 @@ driver_pgsql_init_full_v(const struct sql_legacy_settings *legacy_set,
        const char *value;
 
        pool_t pool = pool_alloconly_create("pgsql settings", 128);
-       struct pgsql_settings *set = p_new(pool, struct pgsql_settings, 1);
-       *set = pgsql_default_settings;
-       set->pool = pool;
+       struct pgsql_settings *set =
+               settings_defaults_dup(pool, &pgsql_setting_parser_info);
 
        /* NOTE: Connection string will be parsed by pgsql itself
                 We only pick the host part here */
index 9a65f901d6ac0cff3a33530f9b9c9aaa0f253916..ef4fc1eccc9ef1088ac6722881cfb64c5228f971 100644 (file)
@@ -218,9 +218,8 @@ driver_sqlite_parse_connect_string(pool_t pool,
                return -1;
        }
 
-       struct sqlite_settings *set = p_new(pool, struct sqlite_settings, 1);
-       *set = sqlite_default_settings;
-       set->pool = pool;
+       struct sqlite_settings *set =
+               settings_defaults_dup(pool, &sqlite_setting_parser_info);
 
        for (; *params != NULL; params++) {
                if (str_begins(*params, "journal_mode=", &arg)) {
index d313e067a077062ec6ff07f7cadbe2fc36edd57c..e04dbb2a722fe3d9d3f19be9746b2e5cb4e492ce 100644 (file)
@@ -169,9 +169,7 @@ reader_client_input_metrics_add(struct reader_client *client,
 
        pool_t pool = pool_alloconly_create("dynamic stats metrics", 128);
        struct stats_metric_settings *set =
-               p_new(pool, struct stats_metric_settings, 1);
-       *set = stats_metric_default_settings;
-       set->pool = pool;
+               settings_defaults_dup(pool, &stats_metric_setting_parser_info);
        set->name = p_strdup(pool, args[0]);
        set->description = p_strdup(pool, args[1]);
        set->filter = p_strdup(pool, args[4]);