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


Ignore:
Timestamp:
Jun 17, 2009, 10:53:23 PM (15 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:
8 edited

Legend:

Unmodified
Added
Removed

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

    r2227 r2230  
    14751475                    maintain_collection_of_recent_archives(bkpinfo->tmpdir,
    14761476                                                       storing_afioball_fname);
    1477                     log_it("%s","Writing EXAT files");
     1477                    log_it("Writing EXAT files");
    14781478                    res +=
    14791479                        write_EXAT_files_to_tape(curr_xattr_list_fname,
    … …  
    19991999    log_msg(1, "--> tmp = '%s'", tmp);
    20002000    if (strstr(tmp, "NTFS")) {
    2001         log_it("%s","TRUE");
     2001        log_it("TRUE");
    20022002        ret = TRUE;
    20032003    } else {
    2004         log_it("%s","FALSE");
     2004        log_it("FALSE");
    20052005        ret = FALSE;
    20062006    }
    … …  
    22712271            register_in_tape_catalog(fileset, curr_set_no, -1, curr_afioball_fname);
    22722272            maintain_collection_of_recent_archives(bkpinfo->tmpdir, curr_afioball_fname);
    2273             log_it("%s","Writing EXAT files");
     2273            log_it("Writing EXAT files");
    22742274            res +=
    22752275                write_EXAT_files_to_tape(curr_xattr_list_fname,
    … …  
    34503450                 media_descriptor_string(g_backup_media_type));
    34513451            chdir("/");
    3452             log_it("%s","Before calling verification of image()");
     3452            log_it("Before calling verification of image()");
    34533453            if (bkpinfo->backup_media_type == usb) {
    34543454                res += verify_usb_image();
    … …  
    34563456                res += verify_cd_image();
    34573457            }
    3458             log_it("%s","After calling verification of image()");
     3458            log_it("After calling verification of image()");
    34593459        }
    34603460        if (!res) {

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

    r2227 r2230  
    592592    pout = popen(syscall_pout, "w");
    593593    if (!pout) {
    594         log_it("%s","Unable to openout to syscall_pout");
     594        log_it("Unable to openout to syscall_pout");
    595595        return (1);
    596596    }
    … …  
    598598    if (!pin) {
    599599        pclose(pout);
    600         log_it("%s","Unable to openin from syscall");
     600        log_it("Unable to openin from syscall");
    601601        return (1);
    602602    }
    … …  
    605605        pclose(pin);
    606606        pclose(pout);
    607         log_it("%s","Unable to openin masklist");
     607        log_it("Unable to openin masklist");
    608608        return (1);
    609609    }

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

    r2219 r2230  
    652652    /*@ buffers ********************************************************* */
    653653    char *drive = NULL;
     654    char *truncdrive = NULL;
    654655
    655656    long long size;
    … …  
    676677        log_msg(8, "Putting %s with size %lli in list of drives", drive, size);
    677678
    678         (void) truncate_to_drive_name(drive);
    679         log_msg(8, "drive truncated to %s", drive);
     679        truncdrive = truncate_to_drive_name(drive);
     680        log_msg(8, "drive truncated to %s", truncdrive);
    680681
    681682        for (j = 0;
    682683             j < noof_drives
    683              && strcmp(drivelist->el[j].device, drive) != 0; j++) {
     684             && strcmp(drivelist->el[j].device, truncdrive) != 0; j++) {
    684685            continue;
    685686        }
    686687        if (j == noof_drives) {
    687             strncpy(drivelist->el[noof_drives++].device, drive, 63);
     688            strcpy(drivelist->el[noof_drives++].device, truncdrive);
    688689        }
    689690        paranoid_free(drive);
     691        paranoid_free(truncdrive);
    690692    }
    691693    drivelist->entries = noof_drives;

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

    r2229 r2230  
    555555    last = g_tapecatalog->entries - 1;
    556556    if (last <= 0) {
    557         log_it("%s","Too early to start deleting from collection.");
     557        log_it("Too early to start deleting from collection.");
    558558        return (0);
    559559    }
    … …  
    568568    }
    569569    if (curr < 0) {
    570         log_it("%s","Not far enough into tape to start deleting old archives from collection.");
     570        log_it("Not far enough into tape to start deleting old archives from collection.");
    571571        return (0);
    572572    }
    … …  
    17161716                   g_tape_stream) / 1024;
    17171717        if (g_sigpipe) {
    1718             log_it("%s","Sigpipe occurred recently. I'll start a new tape.");
     1718            log_it("Sigpipe occurred recently. I'll start a new tape.");
    17191719            fclose(fin);
    17201720            g_sigpipe = FALSE;

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

    r2211 r2230  
    841841    int i = strlen(partition) - 1;
    842842    char *c;
     843    char *trunc = NULL;
     844
     845    mr_asprintf(&trunc, "%s", partition);
    843846
    844847#ifdef __FreeBSD__
    845848
    846     if (islower(partition[i]))  // BSD subpartition
     849    if (islower(trunc[i]))  // BSD subpartition
    847850        i--;
    848     if (partition[i - 1] == 's') {
    849         while (isdigit(partition[i]))
     851    if (trunc[i-1] == 's') {
     852        while (isdigit(trunc[i]))
    850853            i--;
    851854        i--;
    852855    }
    853     partition[i + 1] = '\0';
     856    trunc[i+1] = '\0';
    854857
    855858#else
    … …  
    857860    assert_string_is_neither_NULL_nor_zerolength(partition);
    858861    /* first see if it's a devfs style device */
    859     c = strrchr(partition, '/');
     862    c = strrchr(trunc, '/');
    860863    if (c && strncmp(c, "/part", 5) == 0) {
    861864        /* yup it's devfs, return the "disc" path */
    862865        strcpy(c + 1, "disc");
    863         return partition;
     866        return trunc;
    864867    }
    865868    /* then see if it's a dm style device */
    866869    if (c && strncmp(c, "/dm-", 4) == 0) {
    867870        /* yup it's dm, return the full path */
    868         return partition;
    869     }
    870 
    871 
    872     for (i = strlen(partition); isdigit(partition[i - 1]); i--)
     871        return trunc;
     872    }
     873
     874
     875    for (i = strlen(trunc); isdigit(trunc[i-1]); i--)
    873876        continue;
    874     if (partition[i - 1] == 'p' && isdigit(partition[i - 2])) {
     877    if (trunc[i-1] == 'p' && isdigit(trunc[i-2])) {
    875878        i--;
    876879    } else {
    877880        /* Some full devices like this /dev/mapper/mpath0
    878881           /dev/cciss/c0d0 may be used as partition names */
    879         if ((strstr(partition,"/dev/mapper/mpath") != NULL) ||
    880             (strstr(partition,"/dev/cciss/c") != NULL) ||
    881             (strstr(partition,"/dev/rd/") != NULL)) {
    882                 return partition;
     882        if ((strstr(trunc,"/dev/mapper/mpath") != NULL) ||
     883            (strstr(trunc,"/dev/cciss/c") != NULL) ||
     884            (strstr(trunc,"/dev/rd/") != NULL)) {
     885                return trunc;
    883886            }
    884887    }
    885     partition[i] = '\0';
     888    trunc[i] = '\0';
    886889
    887890#endif
    888891
    889     return partition;
     892    return trunc;
    890893}
    891894

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

    r2226 r2230  
    13451345
    13461346    if (debug_level <= g_loglevel) {
    1347         va_start(args, fmt);
    13481347        if (!(fout = fopen(MONDO_LOGFILE, "a"))) {
    13491348            return;
    … …  
    13611360                fprintf(fout, "[TH=%d] %s->%s#%d: ", getpid(), szFile, szFunction, nLine);
    13621361        }
     1362        va_start(args, fmt);
    13631363        vfprintf(fout, fmt, args);
     1364        va_end(args);
    13641365
    13651366        // do not slow down the progran if standard debug level
    … …  
    13681369        //if (g_dwDebugLevel != 1)
    13691370
    1370         va_end(args);
    13711371        fprintf(fout, "\n");
    13721372        paranoid_fclose(fout);

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

    r2227 r2230  
    281281    }
    282282
    283     log_it("%s","before vsbf");
     283    log_it("before vsbf");
    284284    sprintf(tmp, "Verifying %s#%d's big files",
    285285            media_descriptor_string(bkpinfo->backup_media_type),
    286286            g_current_media_number);
    287287    open_evalcall_form(tmp);
    288     log_it("%s","after vsbf");
     288    log_it("after vsbf");
    289289    sprintf(mountpoint, "%s/archives", mtpt);
    290290    if (last_bigfile_num == -1) {
    … …  
    780780    res = read_header_block_from_stream(&size, fname, &ctrl_chr);
    781781    if (ctrl_chr != BLK_START_AFIOBALLS) {
    782         log_it("%s","YOU SHOULD NOT GET HERE");
    783         log_it("%s","Grabbing the EXAT files");
     782        log_it("YOU SHOULD NOT GET HERE");
     783        log_it("Grabbing the EXAT files");
    784784        if (ctrl_chr == BLK_START_EXTENDED_ATTRIBUTES) {
    785785            res =
    … …  
    805805        }
    806806        if (ctrl_chr == BLK_START_EXTENDED_ATTRIBUTES) {
    807             log_it("%s","Reading EXAT files from tape");
     807            log_it("Reading EXAT files from tape");
    808808            res =
    809809                read_EXAT_files_from_tape(&size, fname, &ctrl_chr,
    … …  
    895895    if (ctrl_chr != BLK_START_BIGGIEFILES) {
    896896        if (ctrl_chr == BLK_START_EXTENDED_ATTRIBUTES) {
    897             log_it("%s","Grabbing the EXAT biggiefiles");
     897            log_it("Grabbing the EXAT biggiefiles");
    898898            res =
    899899                read_EXAT_files_from_tape(&size, orig_fname,
    … …  
    10191019    paranoid_free(tmp);
    10201020    verify_afioballs_on_CD(mountpoint);
    1021     log_it("%s","before verify_all_slices");
     1021    log_it("before verify_all_slices");
    10221022    verify_all_slices_on_CD(mountpoint);
    10231023
    … …  
    11411141    }
    11421142    verify_afioballs_on_CD(mountpoint);
    1143     log_it("%s","before verify_all_slices");
     1143    log_it("before verify_all_slices");
    11441144    verify_all_slices_on_CD(mountpoint);
    11451145

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

    r2227 r2230  
    14001400        assert_string_is_neither_NULL_nor_zerolength(source_file);
    14011401
    1402         log_it("%s","entering");
     1402        log_it("entering");
    14031403        if (!(fin = fopen(source_file, "r"))) {
    14041404            log_OS_error(source_file);
    … …  
    14831483        }
    14841484        paranoid_free(tmp);
    1485         log_it("%s","leaving");
     1485        log_it("leaving");
    14861486        return (0);
    14871487    }
    … …  
    14991499        char *tmp = NULL;
    15001500
    1501         log_it("%s","entering");
     1501        log_it("entering");
    15021502        assert(flentry != NULL);
    15031503        if (flentry->severity == 0) {
    … …  
    15151515        paranoid_free(tmp);
    15161516
    1517         log_it("%s","leaving");
     1517        log_it("leaving");
    15181518        return (comment);
    15191519    }
Note: See TracChangeset for help on using the changeset viewer.