Changeset 2322 in MondoRescue for branches/2.2.10/mondo/src/common/libmondo-string.c

Timestamp:

Aug 18, 2009, 2:37:58 PM (15 years ago)

Author:

Bruno Cornec

Message:

r3333@localhost: bruno | 2009-08-08 01:58:31 +0200

• bkpinfo->prefix is now dynamically assigned
• Fix newt mondoarchive

File:

• branches/2.2.10/mondo/src/common/libmondo-string.c (modified) (9 diffs)

branches/2.2.10/mondo/src/common/libmondo-string.c

@@ -203 +203 @@
     int i;
     char *tmp;
+    char *tmp1 = NULL;
     char ch;
 
@@ -224 +225 @@
     tmp[i] = '\0';
     outval = atol(tmp);
+    paranoid_free(tmp);
     if (ch == 'g' || ch == 'G') {
         outval = outval * 1024;
@@ -239 +241 @@
         fatal_error("Integer overflow.");
     } else if (ch != 'm' && ch != 'M') {
-        sprintf(tmp, "Re: parameter '%s' - bad multiplier ('%c')",
-                incoming, ch);
-        fatal_error(tmp);
-    }
-    paranoid_free(tmp);
+        mr_asprintf(&tmp1, "Re: parameter '%s' - bad multiplier ('%c')", incoming, ch);
+        fatal_error(tmp1);
+    }
     return (outval);
 }
@@ -549 +549 @@
     /*@ buffers **************************************************** */
     static char output[MAX_STR_LEN];
-    static char suffix[MAX_STR_LEN];
+    char *suffix = NULL;
 
     /*@ end vars *************************************************** */
@@ -555 +555 @@
     assert_string_is_neither_NULL_nor_zerolength(path);
     if (s[0] != '\0') {
-        sprintf(suffix, ".%s", s);
+        mr_asprintf(&suffix, ".%s", s);
     } else {
-        suffix[0] = '\0';
-    }
-    sprintf(output, "%s/slice-%07ld.%05ld.dat%s", path, bigfileno, sliceno,
-            suffix);
+        mr_asprintf(&suffix, "");
+    }
+    sprintf(output, "%s/slice-%07ld.%05ld.dat%s", path, bigfileno, sliceno, suffix);
+    mr_free(suffix);
     return (output);
 }
@@ -925 +925 @@
 {
     int sev;
-    char *reason;
-    char *filename;
-
-    malloc_string(reason);
-    malloc_string(filename);
-    
+    char *reason = NULL;
+    char *filename = NULL;
+
     // out_reason might be null on purpose, so don't bomb if it is :) OK?
     assert_string_is_neither_NULL_nor_zerolength(fn);
     if (!strncmp(fn, MNT_RESTORING, strlen(MNT_RESTORING))) {
-        strcpy(filename, fn + strlen(MNT_RESTORING));
+        mr_asprintf(&filename, "%s", fn + strlen(MNT_RESTORING));
     } else if (fn[0] != '/') {
-        sprintf(filename, "/%s", fn);
+        mr_asprintf(&filename, "/%s", fn);
     } else {
-        strcpy(filename, fn);
+        mr_asprintf(&filename, "%s", fn);
     }
 
     sev = 3;
-    sprintf(reason,
-            "Changed since backup. Consider running a differential backup in a day or two.");
+    mr_asprintf(&reason, "Changed since backup. Consider running a differential backup in a day or two.");
+
     if (!strncmp(filename, "/var/", 5)) {
         sev = 2;
-        sprintf(reason,
-                "/var's contents will change regularly, inevitably.");
+        mr_free(reason);
+        mr_asprintf(&reason, "/var's contents will change regularly, inevitably.");
     }
     if (!strncmp(filename, "/home", 5)) {
         sev = 2;
-        sprintf(reason,
-                "It's in your /home directory. Therefore, it is important.");
+        mr_free(reason);
+        mr_asprintf(&reason, "It's in your /home directory. Therefore, it is important.");
     }
     if (!strncmp(filename, "/usr/", 5)) {
         sev = 3;
-        sprintf(reason,
-                "You may have installed/removed software during the backup.");
+        mr_free(reason);
+        mr_asprintf(&reason, "You may have installed/removed software during the backup.");
     }
     if (!strncmp(filename, "/etc/", 5)) {
         sev = 3;
-        sprintf(reason,
-                "Do not edit config files while backing up your PC.");
+        mr_free(reason);
+        mr_asprintf(&reason, "Do not edit config files while backing up your PC.");
     }
     if (!strcmp(filename, "/etc/adjtime")
         || !strcmp(filename, "/etc/mtab")) {
         sev = 1;
-        sprintf(reason, "This file changes all the time. It's OK.");
+        mr_free(reason);
+        mr_asprintf(&reason, "This file changes all the time. It's OK.");
     }
     if (!strncmp(filename, "/root/", 6)) {
         sev = 3;
-        sprintf(reason, "Were you compiling/editing something in /root?");
+        mr_free(reason);
+        mr_asprintf(&reason, "Were you compiling/editing something in /root?");
     }
     if (!strncmp(filename, "/root/.", 7)) {
         sev = 2;
-        sprintf(reason, "Temp or 'dot' files changed in /root.");
+        mr_free(reason);
+        mr_asprintf(&reason, "Temp or 'dot' files changed in /root.");
     }
     if (!strncmp(filename, "/var/lib/", 9)) {
         sev = 2;
-        sprintf(reason, "Did you add/remove software during backing?");
+        mr_free(reason);
+        mr_asprintf(&reason, "Did you add/remove software during backing?");
     }
     if (!strncmp(filename, "/var/lib/rpm", 12)) {
         sev = 3;
-        sprintf(reason, "Did you add/remove software during backing?");
+        mr_free(reason);
+        mr_asprintf(&reason, "Did you add/remove software during backing?");
     }
     if (!strncmp(filename, "/var/lib/slocate", 16)) {
         sev = 1;
-        sprintf(reason,
-                "The 'update' daemon ran during backup. This does not affect the integrity of your backup.");
+        mr_free(reason);
+        mr_asprintf(&reason, "The 'update' daemon ran during backup. This does not affect the integrity of your backup.");
     }
     if (!strncmp(filename, "/var/log/", 9)
@@ -994 +996 @@
         || !strcmp(filename + strlen(filename) - 4, ".log")) {
         sev = 1;
-        sprintf(reason,
-                "Log files change frequently as the computer runs. Fret not.");
+        mr_free(reason);
+        mr_asprintf(&reason, "Log files change frequently as the computer runs. Fret not.");
     }
     if (!strncmp(filename, "/var/spool", 10)) {
         sev = 1;
-        sprintf(reason,
-                "Background processes or printers were active. This does not affect the integrity of your backup.");
+        mr_free(reason);
+        mr_asprintf(&reason, "Background processes or printers were active. This does not affect the integrity of your backup.");
     }
     if (!strncmp(filename, "/var/spool/mail", 10)) {
         sev = 2;
-        sprintf(reason, "Mail was sent/received during backup.");
+        mr_free(reason);
+        mr_asprintf(&reason, "Mail was sent/received during backup.");
     }
     if (filename[strlen(filename) - 1] == '~') {
         sev = 1;
-        sprintf(reason,
-                "Backup copy of another file which was modified recently.");
+        mr_free(reason);
+        mr_asprintf(&reason, "Backup copy of another file which was modified recently.");
     }
     if (strstr(filename, "cache")) {
         sev = 1;
-        sprintf(reason,
-                "Part of a cache of data. Caches change from time to time. Don't worry.");
+        mr_free(reason);
+        mr_asprintf(&reason, "Part of a cache of data. Caches change from time to time. Don't worry.");
     }
     if (!strncmp(filename, "/var/run/", 9)
@@ -1021 +1024 @@
         || strstr(filename, "/.Xauthority")) {
         sev = 1;
-        sprintf(reason,
-                "Temporary file (a lockfile, perhaps) used by software such as X or KDE to register its presence.");
+        mr_free(reason);
+        mr_asprintf(&reason, "Temporary file (a lockfile, perhaps) used by software such as X or KDE to register its presence.");
     }
     if (out_reason) {
         strcpy(out_reason, reason);
     }
-    paranoid_free(filename);
-    paranoid_free(reason);
+    mr_free(filename);
+    mr_free(reason);
     return (sev);
 }
@@ -1078 +1081 @@
     /*@ buffers ******************************************* */
     char *outstr = NULL;
-    char *pos_w_commas, *tmp;
+    char *pos_w_commas = NULL;
     char *mds = NULL;
+    char *tmp = NULL;
 
     assert(bkpinfo != NULL);
 
     if (bkpinfo->media_size[g_current_media_number] <= 0) {
-        malloc_string(pos_w_commas);
-        malloc_string(tmp);
-        sprintf(tmp, "%lld", g_tape_posK);
-        strcpy(pos_w_commas, commarize(tmp));
-        paranoid_free(tmp);
+        mr_asprintf(&tmp, "%lld", g_tape_posK);
+        mr_asprintf(&pos_w_commas, "%s", commarize(tmp));
+        mr_free(tmp);
         mr_asprintf(&outstr, "Volume %d: %s kilobytes archived so far", g_current_media_number, pos_w_commas);
-        paranoid_free(pos_w_commas);
+        mr_free(pos_w_commas);
         return (outstr);
     }