Changeset 3856 in MondoRescue for branches/3.3/mondo/src/common/libmondo-archive.c

Timestamp:

Mar 6, 2024, 5:29:44 PM (2 months ago)

Author:

Bruno Cornec

Message:

remove snprintf usage and use bkpinfo for serial_string

File:

• branches/3.3/mondo/src/common/libmondo-archive.c (modified) (4 diffs)

branches/3.3/mondo/src/common/libmondo-archive.c

@@ -67 +67 @@
 extern bool g_cd_recovery;
 extern char *g_mondo_home;
-
-/**
- * The serial string (used to differentiate between backups) of the current backup.
- */
-char *g_serial_string = NULL;
 
 extern char *g_getfacl;
@@ -1124 +1119 @@
  * - @c scratchdir
  * - @c tmpdir
+ * - @c serial_string
  * @return The number of errors encountered (0 for success).
  * @ingroup MLarchiveGroup
@@ -1140 +1136 @@
     mr_asprintf(data_disks_file, "%s/all.tar.gz", bkpinfo->tmpdir);
 
-        tmp = call_program_and_get_last_line_of_output("dd if=/dev/urandom bs=16 count=1 2> /dev/null | hexdump | tr -s ' ' '0' | head -n1");
-    snprintf(g_serial_string, MAX_STR_LEN - 8, "%s", tmp);
-        mr_free(tmp);
-    strip_spaces(g_serial_string);
-    strcat(g_serial_string, "...word.");
-    log_msg(2, "g_serial_string = '%s'", g_serial_string);
-    assert(strlen(g_serial_string) < MAX_STR_LEN);
+    tmp = call_program_and_get_last_line_of_output("dd if=/dev/urandom bs=16 count=1 2> /dev/null | hexdump | tr -s ' ' '0' | head -n1");
+    bkpinfo->serial_string = mr_strip_spaces(tmp);
+    mr_free(tmp);
+    mr_strcat(bkpinfo->serial_string, "...word.");
+    log_msg(2, "bkpinfo->serial_string = '%s'", bkpinfo->serial_string);
 
     mr_asprintf(tmpfile, "%s/archives/SERIAL-STRING", bkpinfo->scratchdir);
-    if (write_one_liner_data_file(tmpfile, g_serial_string)) {
+    if (write_one_liner_data_file(tmpfile, bkpinfo->serial_string)) {
         log_msg(1, "%ld: Failed to write serial string", __LINE__);
     }
@@ -2584 +2578 @@
 
         log_msg(2, "paafcd: cd_number = %d", cd_number);
-        log_msg(2, "our serial str = %s; g_serial_string = %s", our_serial_str, g_serial_string);
-        if (cd_number > 0 && !strcmp(our_serial_str, g_serial_string)) {
+        log_msg(2, "our serial str = %s; bkpinfo->serial_string = %s", our_serial_str, bkpinfo->serial_string);
+        if (cd_number > 0 && !strcmp(our_serial_str, bkpinfo->serial_string)) {
             mds = media_descriptor_string(g_backup_media_type);
             log_msg(2, "This %s is part of this backup set!", mds);