Ignore:
Timestamp:
Nov 10, 2012, 5:05:37 AM (11 years ago)
Author:
Bruno Cornec
Message:

r5035@localhost: bruno | 2012-11-09 03:17:01 +0100

  • Fix a compilation error and most compilation warnings
File:
1 edited

Legend:

Unmodified
Added
Removed

  • branches/3.0/mondo/src/mondorestore/mondo-prep.c

    r3057 r3060  
    110110                }
    111111            }
    112             system("sync");
    113             system("sync");
    114             system("sync");
     112            paranoid_system("sync");
     113            paranoid_system("sync");
     114            paranoid_system("sync");
    115115            popup_and_OK
    116116                ("I must now reboot. Please leave the boot media in the drive and repeat your actions - e.g. type 'nuke' - and it should work fine.");
    117             system("reboot");
     117            paranoid_system("reboot");
    118118        }
    119119    }
    … …  
    197197    char *p;
    198198    char *q;
     199    char *r;
    199200
    200201    /** int ***************************************************/
    … …  
    272273    log_msg(1, "OK, rewound i-want-my-lvm. Doing funky stuff...");
    273274    rewind(fin);
    274     for (fgets(incoming, MAX_STR_LEN - 1, fin); !feof(fin); fgets(incoming, MAX_STR_LEN - 1, fin)) {
     275    for (r = fgets(incoming, MAX_STR_LEN - 1, fin); !feof(fin) && (r != NULL); r = fgets(incoming, MAX_STR_LEN - 1, fin)) {
    275276        fgetpos(fin, &orig_pos);
    276277        if (incoming[0] != '#') {
    … …  
    279280        if (res && strstr(command, "create") && vacuum_pack) {
    280281            sleep(2);
    281             system("sync");
    282             system("sync");
    283             system("sync");
     282            paranoid_system("sync");
     283            paranoid_system("sync");
     284            paranoid_system("sync");
    284285        }
    285286        if ((p = strstr(incoming, "vgcreate"))) {
    286287// include next line(s) if they end in /dev (cos we've got a broken i-want-my-lvm)
    287             for (fgets(tmp, MAX_STR_LEN - 1, fin); !feof(fin); fgets(tmp, MAX_STR_LEN - 1, fin)) {
     288            for (q = fgets(tmp, MAX_STR_LEN - 1, fin); !feof(fin) && (q != NULL); q = fgets(tmp, MAX_STR_LEN - 1, fin)) {
    288289                if (tmp[0] == '#') {
    289290                    fsetpos(fin, &orig_pos);
    … …  
    393394        }
    394395        sprintf(tmp, "echo \"%s\" >> /tmp/out.sh", command);
    395         system(tmp);
     396        paranoid_system(tmp);
    396397        sleep(1);
    397398    }
    … …  
    410411    paranoid_free(vgremove_sz);
    411412//  paranoid_free(do_this_last);
    412     system("sync");
    413     system("sync");
    414     system("sync");
     413    paranoid_system("sync");
     414    paranoid_system("sync");
     415    paranoid_system("sync");
    415416    sleep(1);
    416417    log_it("ENDING");
    … …  
    451452    /** pointers *********************************************************/
    452453    char *p;
     454    char *q;
    453455
    454456    /** init *************************************************************/
    … …  
    480482                finish(1);
    481483            }
    482             for (fgets(incoming, MAX_STR_LEN - 1, fin); !feof(fin)
     484            for (q = fgets(incoming, MAX_STR_LEN - 1, fin); !feof(fin) && (q != NULL)
    483485                 && !strstr(incoming, old_mountlist->el[lino].device);
    484                  fgets(incoming, MAX_STR_LEN - 1, fin));
     486                 q = fgets(incoming, MAX_STR_LEN - 1, fin));
    485487            if (!feof(fin)) {
    486488                sprintf(tmp, "Investigating %s",
    487489                        old_mountlist->el[lino].device);
    488490                log_it(tmp);
    489                 for (fgets(incoming, MAX_STR_LEN - 1, fin); !feof(fin)
     491                for (q = fgets(incoming, MAX_STR_LEN - 1, fin); !feof(fin) && (q != NULL)
    490492                     && !strstr(incoming, "raiddev");
    491                      fgets(incoming, MAX_STR_LEN - 1, fin)) {
     493                     q = fgets(incoming, MAX_STR_LEN - 1, fin)) {
    492494                    if (strstr(incoming, OSSWAP("device", "drive"))
    493495                        && !strchr(incoming, '#')) {
    … …  
    778780                    "for plex in `vinum lv -r %s | grep '^P' | tr '\t' ' ' | tr -s ' ' | cut -d' ' -f2`; do echo $plex; done > /tmp/plexes",
    779781                    basename(device));
    780             system(program);
     782            paranoid_system(program);
    781783            if (g_fprep) {
    782784                fprintf(g_fprep, "%s\n", program);
    … …  
    792794
    793795                mr_asprintf(&tmp, "vinum init %s", line);
    794                 system(tmp);
     796                paranoid_system(tmp);
    795797                paranoid_free(tmp);
    796798
    … …  
    801803
    802804                    char status[MAX_STR_LEN / 4];
    803                     fgets(status, MAX_STR_LEN / 4 - 1, pin);
     805                    if (fgets(status, MAX_STR_LEN / 4 - 1, pin)) {
     806                        // FIXME
     807                    }
    804808                    pclose(pin);
    805809
    … …  
    823827        log_to_screen("Stopping %s", device);
    824828        stop_raid_device(device);
    825         system("sync");
     829        paranoid_system("sync");
    826830        sleep(1);
    827831        if (g_fprep) {
    … …  
    838842            res = run_program_and_log_output(program, 1);
    839843            log_msg(1, "%s returned %d", program, res);
    840             system("sync");
     844            paranoid_system("sync");
    841845            sleep(3);
    842846            start_raid_device(device);
    … …  
    845849            }
    846850        }
    847         system("sync");
     851        paranoid_system("sync");
    848852        sleep(2);
    849853//      log_to_screen("Starting %s", device);
    … …  
    851855//      res = run_program_and_log_output(program, 1);
    852856//      log_msg(1, "%s returned %d", program, res);
    853 //      system("sync"); sleep(1);
    854 #endif
    855         system("sync");
     857//      paranoid_system("sync"); sleep(1);
     858#endif
     859        paranoid_system("sync");
    856860        sleep(1);
    857861        newtResume();
    … …  
    917921    retval += res;
    918922    paranoid_free(program);
    919     system("sync");
     923    paranoid_system("sync");
    920924    sleep(1);
    921925    return (retval);
    … …  
    973977    log_msg(1, "Stopping all RAID devices");
    974978    stop_all_raid_devices(mountlist);
    975     system("sync");
    976     system("sync");
    977     system("sync");
     979    paranoid_system("sync");
     980    paranoid_system("sync");
     981    paranoid_system("sync");
    978982    sleep(2);
    979983    log_msg(1, "Prepare soft-RAIDs");   // prep and format too
    … …  
    9971001        }
    9981002    }
    999     system("sync");
    1000     system("sync");
    1001     system("sync");
     1003    paranoid_system("sync");
     1004    paranoid_system("sync");
     1005    paranoid_system("sync");
    10021006    sleep(2);
    10031007// This last step is probably necessary
    10041008//  log_to_screen("Re-starting software RAIDs...");
    10051009//  start_all_raid_devices(mountlist);
    1006 //  system("sync"); system("sync"); system("sync");
     1010//  paranoid_system("sync"); paranoid_system("sync"); paranoid_system("sync");
    10071011//  sleep(5);
    10081012// do LVMs now
    … …  
    11271131                ("Please choose 'yes' to reboot and try again; or 'no' to ignore this warning and continue."))
    11281132            {
    1129                 system("sync");
    1130                 system("sync");
    1131                 system("sync");
    1132                 system("reboot");
     1133                paranoid_system("sync");
     1134                paranoid_system("sync");
     1135                paranoid_system("sync");
     1136                paranoid_system("reboot");
    11331137            }
    11341138        } else {
    … …  
    11401144    }
    11411145    newtSuspend();
    1142     system("clear");
     1146    paranoid_system("clear");
    11431147    newtResume();
    11441148    paranoid_free(tmp);
    … …  
    15151519    char *format;
    15161520    char *tmp;
     1521    char *tmp1 = NULL;
    15171522
    15181523    /** end *************************************************************/
    … …  
    16341639                }
    16351640            }
    1636             system("sync");
     1641            paranoid_system("sync");
    16371642#else
    16381643            log_it("New, kernel-friendly partition remover");
    … …  
    16951700        fput_string_one_char_at_a_time(pout_to_fdisk, "w\n");
    16961701        paranoid_pclose(pout_to_fdisk);
    1697         system("sync");
     1702        paranoid_system("sync");
    16981703        log_msg(0,"------------------- fdisk.log looks like this ------------------");
    16991704        sprintf(tmp, "cat %s >> %s", FDISK_LOG, MONDO_LOGFILE);
    1700         system(tmp);
     1705        paranoid_system(tmp);
    17011706        // mark relevant partition as bootable
    17021707        mr_asprintf(&tmp1,"make-me-bootable /tmp/mountlist.txt %s noaction",drivename);
    … …  
    17041709        mr_free(tmp1);
    17051710        log_msg(0,"------------------- end of fdisk.log...       ------------------");
    1706         system("sync");
     1711        paranoid_system("sync");
    17071712        sprintf(tmp, "tail -n6 %s | grep -F \"16: \"", FDISK_LOG);
    17081713        if (!run_program_and_log_output(tmp, 5)) {
    … …  
    20322037    }
    20332038    newtSuspend();
    2034     system("clear");
     2039    paranoid_system("clear");
    20352040    newtResume();
    20362041    paranoid_free(drivelist);
    … …  
    23142319    /** int *************************************************************/
    23152320    int retval = 0;
    2316 #ifndef __FreeBSD__
    2317     int res;
    2318 #endif
    23192321
    23202322    /** char ************************************************************/
    … …  
    23282330#endif
    23292331    FILE *fin;
     2332    char *q;
    23302333    int i;
    23312334
    … …  
    23472350            return (1);
    23482351        }
    2349         for (fgets(incoming, MAX_STR_LEN - 1, fin); !feof(fin);
    2350              fgets(incoming, MAX_STR_LEN - 1, fin)) {
     2352        for (q = fgets(incoming, MAX_STR_LEN - 1, fin); !feof(fin) && (q != NULL);
     2353             q = fgets(incoming, MAX_STR_LEN - 1, fin)) {
    23512354            retval += stop_raid_device(incoming);
    23522355        }
    … …  
    23592362            return (1);
    23602363        }
    2361         for (fgets(incoming, MAX_STR_LEN - 1, fin); !feof(fin);
    2362              fgets(incoming, MAX_STR_LEN - 1, fin)) {
     2364        for (q = fgets(incoming, MAX_STR_LEN - 1, fin); !feof(fin) && (q != NULL);
     2365             q = fgets(incoming, MAX_STR_LEN - 1, fin)) {
    23632366            for (p = incoming;
    23642367                 *p != '\0' && (*p != 'm' || *(p + 1) != 'd'
    … …  
    23682371                for (p = dev; *p > 32; p++);
    23692372                *p = '\0';
    2370                 res = stop_raid_device(dev);
     2373                retval += stop_raid_device(dev);
    23712374            }
    23722375        }
    … …  
    23792382    paranoid_free(dev);
    23802383    paranoid_free(incoming);
    2381     system("sync");
    2382     system("sync");
    2383     system("sync");
     2384    paranoid_system("sync");
     2385    paranoid_system("sync");
     2386    paranoid_system("sync");
    23842387    sleep(1);
    23852388    return (retval);
Note: See TracChangeset for help on using the changeset viewer.