Changeset 2230 in MondoRescue for branches/2.2.9/mondo/src/common

Timestamp:

Jun 17, 2009, 10:53:23 PM (16 years ago)

Author:

Bruno Cornec

Message:

• Simplify log_it calls when possible (with just a fixed string)
• issue with truncate_to_drive_name reported by valgrind at restore time. Needs rewrite IMO

Location:

branches/2.2.9/mondo/src/common

Files:

• libmondo-archive.c (modified) (5 diffs)
• libmondo-filelist.c (modified) (3 diffs)
• libmondo-mountlist.c (modified) (2 diffs)
• libmondo-stream.c (modified) (3 diffs)
• libmondo-string.c (modified) (2 diffs)
• libmondo-tools.c (modified) (3 diffs)
• libmondo-verify.c (modified) (6 diffs)
• newt-specific.c (modified) (4 diffs)

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

@@ -1475 +1475 @@
                     maintain_collection_of_recent_archives(bkpinfo->tmpdir,
                                                        storing_afioball_fname);
-                    log_it("%s","Writing EXAT files");
+                    log_it("Writing EXAT files");
                     res +=
                         write_EXAT_files_to_tape(curr_xattr_list_fname,
@@ -1999 +1999 @@
     log_msg(1, "--> tmp = '%s'", tmp);
     if (strstr(tmp, "NTFS")) {
-        log_it("%s","TRUE");
+        log_it("TRUE");
         ret = TRUE;
     } else {
-        log_it("%s","FALSE");
+        log_it("FALSE");
         ret = FALSE;
     }
@@ -2271 +2271 @@
             register_in_tape_catalog(fileset, curr_set_no, -1, curr_afioball_fname);
             maintain_collection_of_recent_archives(bkpinfo->tmpdir, curr_afioball_fname);
-            log_it("%s","Writing EXAT files");
+            log_it("Writing EXAT files");
             res +=
                 write_EXAT_files_to_tape(curr_xattr_list_fname,
@@ -3450 +3450 @@
                  media_descriptor_string(g_backup_media_type));
             chdir("/");
-            log_it("%s","Before calling verification of image()");
+            log_it("Before calling verification of image()");
             if (bkpinfo->backup_media_type == usb) {
                 res += verify_usb_image();
@@ -3456 +3456 @@
                 res += verify_cd_image();
             }
-            log_it("%s","After calling verification of image()");
+            log_it("After calling verification of image()");
         }
         if (!res) {

branches/2.2.9/mondo/src/common/libmondo-filelist.c

@@ -592 +592 @@
     pout = popen(syscall_pout, "w");
     if (!pout) {
-        log_it("%s","Unable to openout to syscall_pout");
+        log_it("Unable to openout to syscall_pout");
         return (1);
     }
@@ -598 +598 @@
     if (!pin) {
         pclose(pout);
-        log_it("%s","Unable to openin from syscall");
+        log_it("Unable to openin from syscall");
         return (1);
     }
@@ -605 +605 @@
         pclose(pin);
         pclose(pout);
-        log_it("%s","Unable to openin masklist");
+        log_it("Unable to openin masklist");
         return (1);
     }

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

@@ -652 +652 @@
     /*@ buffers ********************************************************* */
     char *drive = NULL;
+    char *truncdrive = NULL;
 
     long long size;
@@ -676 +677 @@
         log_msg(8, "Putting %s with size %lli in list of drives", drive, size);
 
-        (void) truncate_to_drive_name(drive);
-        log_msg(8, "drive truncated to %s", drive);
+        truncdrive = truncate_to_drive_name(drive);
+        log_msg(8, "drive truncated to %s", truncdrive);
 
         for (j = 0;
              j < noof_drives
-             && strcmp(drivelist->el[j].device, drive) != 0; j++) {
+             && strcmp(drivelist->el[j].device, truncdrive) != 0; j++) {
             continue;
         }
         if (j == noof_drives) {
-            strncpy(drivelist->el[noof_drives++].device, drive, 63);
+            strcpy(drivelist->el[noof_drives++].device, truncdrive);
         }
         paranoid_free(drive);
+        paranoid_free(truncdrive);
     }
     drivelist->entries = noof_drives;

branches/2.2.9/mondo/src/common/libmondo-stream.c

@@ -555 +555 @@
     last = g_tapecatalog->entries - 1;
     if (last <= 0) {
-        log_it("%s","Too early to start deleting from collection.");
+        log_it("Too early to start deleting from collection.");
         return (0);
     }
@@ -568 +568 @@
     }
     if (curr < 0) {
-        log_it("%s","Not far enough into tape to start deleting old archives from collection.");
+        log_it("Not far enough into tape to start deleting old archives from collection.");
         return (0);
     }
@@ -1716 +1716 @@
                    g_tape_stream) / 1024;
         if (g_sigpipe) {
-            log_it("%s","Sigpipe occurred recently. I'll start a new tape.");
+            log_it("Sigpipe occurred recently. I'll start a new tape.");
             fclose(fin);
             g_sigpipe = FALSE;

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

@@ -841 +841 @@
     int i = strlen(partition) - 1;
     char *c;
+    char *trunc = NULL;
+
+    mr_asprintf(&trunc, "%s", partition);
 
 #ifdef __FreeBSD__
 
-    if (islower(partition[i]))  // BSD subpartition
+    if (islower(trunc[i]))  // BSD subpartition
         i--;
-    if (partition[i - 1] == 's') {
-        while (isdigit(partition[i]))
+    if (trunc[i-1] == 's') {
+        while (isdigit(trunc[i]))
             i--;
         i--;
     }
-    partition[i + 1] = '\0';
+    trunc[i+1] = '\0';
 
 #else
@@ -857 +860 @@
     assert_string_is_neither_NULL_nor_zerolength(partition);
     /* first see if it's a devfs style device */
-    c = strrchr(partition, '/');
+    c = strrchr(trunc, '/');
     if (c && strncmp(c, "/part", 5) == 0) {
         /* yup it's devfs, return the "disc" path */
         strcpy(c + 1, "disc");
-        return partition;
+        return trunc;
     }
     /* then see if it's a dm style device */
     if (c && strncmp(c, "/dm-", 4) == 0) {
         /* yup it's dm, return the full path */
-        return partition;
-    }
-
-
-    for (i = strlen(partition); isdigit(partition[i - 1]); i--)
+        return trunc;
+    }
+
+
+    for (i = strlen(trunc); isdigit(trunc[i-1]); i--)
         continue;
-    if (partition[i - 1] == 'p' && isdigit(partition[i - 2])) {
+    if (trunc[i-1] == 'p' && isdigit(trunc[i-2])) {
         i--;
     } else {
         /* Some full devices like this /dev/mapper/mpath0
            /dev/cciss/c0d0 may be used as partition names */
-        if ((strstr(partition,"/dev/mapper/mpath") != NULL) ||
-            (strstr(partition,"/dev/cciss/c") != NULL) ||
-            (strstr(partition,"/dev/rd/") != NULL)) {
-                return partition;
+        if ((strstr(trunc,"/dev/mapper/mpath") != NULL) ||
+            (strstr(trunc,"/dev/cciss/c") != NULL) ||
+            (strstr(trunc,"/dev/rd/") != NULL)) {
+                return trunc;
             }
     }
-    partition[i] = '\0';
+    trunc[i] = '\0';
 
 #endif
 
-    return partition;
+    return trunc;
 }
 

branches/2.2.9/mondo/src/common/libmondo-tools.c

@@ -1345 +1345 @@
 
     if (debug_level <= g_loglevel) {
-        va_start(args, fmt);
         if (!(fout = fopen(MONDO_LOGFILE, "a"))) {
             return;
@@ -1361 +1360 @@
                 fprintf(fout, "[TH=%d] %s->%s#%d: ", getpid(), szFile, szFunction, nLine);
         }
+        va_start(args, fmt);
         vfprintf(fout, fmt, args);
+        va_end(args);
 
         // do not slow down the progran if standard debug level
@@ -1368 +1369 @@
         //if (g_dwDebugLevel != 1)
 
-        va_end(args);
         fprintf(fout, "\n");
         paranoid_fclose(fout);

branches/2.2.9/mondo/src/common/libmondo-verify.c

@@ -281 +281 @@
     }
 
-    log_it("%s","before vsbf");
+    log_it("before vsbf");
     sprintf(tmp, "Verifying %s#%d's big files",
             media_descriptor_string(bkpinfo->backup_media_type),
             g_current_media_number);
     open_evalcall_form(tmp);
-    log_it("%s","after vsbf");
+    log_it("after vsbf");
     sprintf(mountpoint, "%s/archives", mtpt);
     if (last_bigfile_num == -1) {
@@ -780 +780 @@
     res = read_header_block_from_stream(&size, fname, &ctrl_chr);
     if (ctrl_chr != BLK_START_AFIOBALLS) {
-        log_it("%s","YOU SHOULD NOT GET HERE");
-        log_it("%s","Grabbing the EXAT files");
+        log_it("YOU SHOULD NOT GET HERE");
+        log_it("Grabbing the EXAT files");
         if (ctrl_chr == BLK_START_EXTENDED_ATTRIBUTES) {
             res =
@@ -805 +805 @@
         }
         if (ctrl_chr == BLK_START_EXTENDED_ATTRIBUTES) {
-            log_it("%s","Reading EXAT files from tape");
+            log_it("Reading EXAT files from tape");
             res =
                 read_EXAT_files_from_tape(&size, fname, &ctrl_chr,
@@ -895 +895 @@
     if (ctrl_chr != BLK_START_BIGGIEFILES) {
         if (ctrl_chr == BLK_START_EXTENDED_ATTRIBUTES) {
-            log_it("%s","Grabbing the EXAT biggiefiles");
+            log_it("Grabbing the EXAT biggiefiles");
             res =
                 read_EXAT_files_from_tape(&size, orig_fname,
@@ -1019 +1019 @@
     paranoid_free(tmp);
     verify_afioballs_on_CD(mountpoint);
-    log_it("%s","before verify_all_slices");
+    log_it("before verify_all_slices");
     verify_all_slices_on_CD(mountpoint);
 
@@ -1141 +1141 @@
     }
     verify_afioballs_on_CD(mountpoint);
-    log_it("%s","before verify_all_slices");
+    log_it("before verify_all_slices");
     verify_all_slices_on_CD(mountpoint);
 

branches/2.2.9/mondo/src/common/newt-specific.c

@@ -1400 +1400 @@
         assert_string_is_neither_NULL_nor_zerolength(source_file);
 
-        log_it("%s","entering");
+        log_it("entering");
         if (!(fin = fopen(source_file, "r"))) {
             log_OS_error(source_file);
@@ -1483 +1483 @@
         }
         paranoid_free(tmp);
-        log_it("%s","leaving");
+        log_it("leaving");
         return (0);
     }
@@ -1499 +1499 @@
         char *tmp = NULL;
 
-        log_it("%s","entering");
+        log_it("entering");
         assert(flentry != NULL);
         if (flentry->severity == 0) {
@@ -1515 +1515 @@
         paranoid_free(tmp);
 
-        log_it("%s","leaving");
+        log_it("leaving");
         return (comment);
     }