Changeset 1303 in MondoRescue

Timestamp:

Apr 15, 2007, 2:33:22 AM (18 years ago)

Author:

Bruno Cornec

Message:

• Some FreeBSD fixes
• Better messages for partition names
• May improve the gap issue on cciss

Location:

branches/2.2.2/mondo/src

Files:

• common/libmondo-devices.c (modified) (1 diff)
• common/libmondo-mountlist.c (modified) (8 diffs)
• mondorestore/mondo-prep.c (modified) (5 diffs)

branches/2.2.2/mondo/src/common/libmondo-devices.c

@@ -561 +561 @@
 #ifdef __FreeBSD__
     // We assume here that this is running from mondorestore. (It is.)
-    sprintf(program, "ls %s >/dev/null 2>&1", drive,
+    sprintf(program, "ls %s %s >/dev/null 2>&1", drive,
             build_partition_name(tmp, drive, partno));
     return system(program);

branches/2.2.2/mondo/src/common/libmondo-mountlist.c

@@ -202 +202 @@
     npos = pos = 0;
     for (curr_part_no = 1; curr_part_no < 99; curr_part_no++) {
-        sprintf(device, "%ss%d", drive, curr_part_no);
+        build_partition_name(device, drive, curr_part_no);
         pos = find_device_in_mountlist(mountlist, device);
         npos = 0;
@@ -216 +216 @@
         }
 
-        sprintf(device, "%ss%d", drive, curr_part_no);
+        build_partition_name(device, drive, curr_part_no);
         strcpy(mountpoint, mountlist->el[pos].mountpoint);
         if (pos > 0 && !npos) {
@@ -397 +397 @@
     char *tmp;
     char *device;
-    char *mountpoint;
 
     /*@ long *********************************************************** */
@@ -413 +412 @@
     malloc_string(tmp);
     malloc_string(device);
-    malloc_string(mountpoint);
     flaws_str[0] = '\0';
     prev_part_no = 0;
@@ -438 +436 @@
             continue;
         }
-        strcpy(mountpoint, mountlist->el[pos].mountpoint);
         /* gap in the partition list? */
         if (curr_part_no - prev_part_no > 1) {
@@ -448 +445 @@
             } else if (curr_part_no > 5
                        || (curr_part_no <= 4 && prev_part_no > 0)) {
-                sprintf(tmp, " Gap between %s%d and %d.", drive,
+                sprintf(tmp, " Gap on %s between %d and %d.", drive,
                         prev_part_no, curr_part_no);
                 log_it(tmp);
@@ -460 +457 @@
         if ((curr_part_no >= 5 && prev_part_no == 4)
             && (strcmp(part_table_fmt, "MBR") == 0)) {
-            sprintf(tmp, " Partition %s4 is occupied.", drive);
+            sprintf(tmp, " Partition 4 of %s is occupied.", drive);
             log_it(tmp);
             strcat(flaws_str, tmp);
@@ -531 +528 @@
     paranoid_free(tmp);
     paranoid_free(device);
-    paranoid_free(mountpoint);
 
     if (res) {

branches/2.2.2/mondo/src/mondorestore/mondo-prep.c

@@ -1284 +1284 @@
 
     if (devno_we_must_allow_for >= 5) {
-        sprintf(tmp, "Making dummy primary %s%d", drivename, 1);
+        sprintf(tmp, "Making dummy primary 1 on %s", drivename);
         log_it(tmp);
         g_maximum_progress++;
@@ -1298 +1298 @@
     }
     for (; current_devno < devno_we_must_allow_for; current_devno++) {
-        sprintf(tmp, "Creating dummy partition %s%d", drivename,
-                current_devno);
+        sprintf(tmp, "Creating dummy partition %d on %s", current_devno, drivename);
         log_it(tmp);
         g_maximum_progress++;
@@ -1979 +1978 @@
         strcpy(tmp, last_line_of_file(FDISK_LOG));
         if (strstr(tmp, " (m ")) {
-            log_msg(1, "Successfully created %s%d", drive, partno);
+            log_msg(1, "Successfully created partition %d on %s", partno, drive);
         } else {
             log_msg(1, "last line = %s", tmp);
-            log_msg(1, "Failed to create %s%d; sending 'Enter'...", drive,
-                    partno);
+            log_msg(1, "Failed to create partition %d on %s; sending 'Enter'...", partno, drive);
         }
         if (!retval) {
-            log_msg(1, "Trying to set %s%d's partition type now", drive,
-                    partno);
+            log_msg(1, "Trying to set partition %d type now on %s", partno, drive);
             retval =
                 set_partition_type(pout_to_fdisk, drive, partno, format,
@@ -2207 +2204 @@
     } else if (strcmp(format, "ext2") == 0
                || strcmp(format, "reiserfs") == 0
-               || strcmp(format, "ext3") == 0 || strcmp(format, "xfs") == 0
+               || strcmp(format, "ext3") == 0 
+               || strcmp(format, "xfs") == 0
                || strcmp(format, "jfs") == 0) {
         strcpy(partcode, "83");
@@ -2584 +2582 @@
         } else {
             sprintf(tmp, "Skipping %s", mountlist->el[partno].device);
-//                      log_to_screen(tmp);
+//            log_to_screen(tmp);
         }
     }