Changeset 2350 in MondoRescue for branches/2.2.9/mondo/src/mondorestore/mondorestore.c

Timestamp:

Aug 28, 2009, 2:24:19 AM (15 years ago)

Author:

Bruno Cornec

Message:

Change inerface of evaluate_mountlist and spread_flaws_across_three_lines in order to solve bugs linked to strings management in these functions. May fix a restoration crash seen by some customers

File:

• branches/2.2.9/mondo/src/mondorestore/mondorestore.c (modified) (3 diffs)

branches/2.2.9/mondo/src/mondorestore/mondorestore.c

@@ -777 +777 @@
   /** malloc **/
     char tmp[MAX_STR_LEN];
-    char *tmpA;
-    char *tmpB;
-    char *tmpC;
+    char *flaws_str = NULL;
 
     assert(bkpinfo != NULL);
     assert(mountlist != NULL);
     assert(raidlist != NULL);
-
-    mr_asprintf(&tmpA, "%s", "         ");
-    mr_asprintf(&tmpB, "%s", "         ");
-    mr_asprintf(&tmpC, "%s", "         ");
 
     log_msg(2, "nuke_mode --- starting");
@@ -807 +801 @@
         resize_mountlist_proportionately_to_suit_new_drives(mountlist);
     }
-    if (!evaluate_mountlist(mountlist, tmpA, tmpB, tmpC)) {
+    flaws_str = evaluate_mountlist(mountlist, &res);
+    if (!res) {
         sprintf(tmp,
-                "Mountlist analyzed. Result: \"%s %s %s\" Switch to Interactive Mode?",
-                tmpA, tmpB, tmpC);
+                "Mountlist analyzed. Result: \"%s\" Switch to Interactive Mode?",
+                flaws_str);
+        mr_free(flaws_str);
         if (ask_me_yes_or_no(tmp)) {
             retval = interactive_mode(mountlist, raidlist);
@@ -818 +814 @@
         }
     }
+    mr_free(flaws_str);
+
     save_mountlist_to_disk(mountlist, g_mountlist_fname);
     mvaddstr_and_log_it(1, 30, "Restoring Automatically");