]> git.ipfire.org Git - thirdparty/rspamd.git/commitdiff
[Minor] One more mkstemp usage
authorVsevolod Stakhov <vsevolod@rspamd.com>
Sat, 4 Mar 2023 13:41:58 +0000 (13:41 +0000)
committerVsevolod Stakhov <vsevolod@rspamd.com>
Sat, 4 Mar 2023 13:41:58 +0000 (13:41 +0000)
src/libserver/maps/map_helpers.c
src/libserver/re_cache.c

index 2fa7743b4c5543c7c8c7670b296bbf6f5707d97f..a1052326c2067a6e82192eb21b7ada3cadd4a847 100644 (file)
@@ -1093,11 +1093,10 @@ rspamd_try_save_re_map_cache (struct rspamd_regexp_map_helper *re_map)
                return FALSE;
        }
 
-       rspamd_snprintf (fp, sizeof (fp), "%s/%*xs.hsmc.tmp",
-                       re_map->map->cfg->hs_cache_dir,
-                       (gint)rspamd_cryptobox_HASHBYTES / 2, re_map->re_digest);
+       rspamd_snprintf (fp, sizeof (fp), "%s/hsmc-XXXXXXXXXXXXX",
+                       re_map->map->cfg->hs_cache_dir);
 
-       if ((fd = rspamd_file_xopen (fp, O_WRONLY | O_CREAT | O_EXCL, 00644, 0)) != -1) {
+       if ((fd = g_mkstemp_full(fp, O_WRONLY | O_CREAT | O_EXCL, 00644)) != -1) {
                if (hs_serialize_database (rspamd_hyperscan_get_database(re_map->hs_db), &bytes, &len) == HS_SUCCESS) {
                        if (write (fd, bytes, len) == -1) {
                                msg_warn_map ("cannot write hyperscan cache to %s: %s",
index c89536206d83b05aee8ccb6398bfa89127092902..cbf76d1a9405fa8f4ac3601246eaa9a531467a62 100644 (file)
@@ -1954,10 +1954,10 @@ rspamd_re_cache_compile_timer_cb (EV_P_ ev_timer *w, int revents )
 
        g_hash_table_iter_init (&cit, re_class->re);
        n = g_hash_table_size (re_class->re);
-       hs_flags = g_malloc0 (sizeof (*hs_flags) * n);
-       hs_ids = g_malloc (sizeof (*hs_ids) * n);
-       hs_pats = g_malloc (sizeof (*hs_pats) * n);
-       hs_exts = g_malloc0 (sizeof (*hs_exts) * n);
+       hs_flags = g_new0(guint, n);
+       hs_ids = g_new0(guint, n);
+       hs_pats = g_new0(char *, n);
+       hs_exts = g_new0(const hs_expr_ext_t *, n);
        i = 0;
 
        while (g_hash_table_iter_next (&cit, &k, &v)) {