Changeset 359


Ignore:
Timestamp:
Jul 26, 2006, 11:32:14 PM (19 years ago)
Author:
root
Message:

Check more run time errors

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/dll/filldir.c

    r282 r359  
    77
    88  Copyright (c) 1993-98 M. Kimes
    9   Copyright (c) 2001, 2005 Steven H. Levine
     9  Copyright (c) 2001, 2006 Steven H. Levine
    1010
    1111  10 Jan 04 SHL ProcessDirectory: avoid most large drive failures
    … …  
    2323  24 Oct 05 SHL FillInRecordFromFSA: correct longname display enable
    2424  24 Oct 05 SHL Drop obsolete code
     25  22 Jul 06 SHL Check more run time errors
    2526
    2627***********************************************************************/
    … …  
    4041#include "fm3dll.h"
    4142#include "fm3str.h"
     43
     44static PSZ pszSrcFile = __FILE__;
    4245
    4346#pragma alloc_text(FILLDIR,FillInRecordFromFFB,FillInRecordFromFSA,IDFile)
    … …  
    192195    CHAR      *value;
    193196
    194     pgealist = malloc(sizeof(GEA2LIST) + 32);
    195     if (pgealist)
    196     {
    197       memset(pgealist,0,sizeof(GEA2LIST) + 32);
     197    pgealist = xmallocz(sizeof(GEA2LIST) + 32,pszSrcFile,__LINE__);
     198    if (pgealist) {
    198199      pgea = &pgealist->list[0];
    199200      strcpy(pgea->szName,SUBJECT);
    … …  
    201202      pgea->oNextEntryOffset = 0;
    202203      pgealist->cbList = (sizeof(GEA2LIST) + pgea->cbName);
    203       pfealist = malloc(1532);
    204       if (pfealist)
    205       {
    206         memset(pfealist,0,1532);
     204      pfealist = xmallocz(1532,pszSrcFile,__LINE__);
     205      if (pfealist) {
    207206        pfealist->cbList = 1024;
    208207        eaop.fpGEA2List = pgealist;
    … …  
    242241    CHAR      *value;
    243242
    244     pgealist = malloc(sizeof(GEA2LIST) + 32);
    245     if (pgealist)
    246     {
    247       memset(pgealist,0,sizeof(GEA2LIST) + 32);
     243    pgealist = xmallocz(sizeof(GEA2LIST) + 32,pszSrcFile,__LINE__);
     244    if (pgealist) {
    248245      pgea = &pgealist->list[0];
    249246      strcpy(pgea->szName,LONGNAME);
    … …  
    251248      pgea->oNextEntryOffset = 0;
    252249      pgealist->cbList = (sizeof(GEA2LIST) + pgea->cbName);
    253       pfealist = malloc(1532);
     250      pfealist = xmallocz(1532,pszSrcFile,__LINE__);
    254251      if (pfealist) {
    255         memset(pfealist,0,1532);
    256252        pfealist->cbList = 1024;
    257253        eaop.fpGEA2List = pgealist;
    … …  
    445441    CHAR      *value;
    446442
    447     pgealist = malloc(sizeof(GEA2LIST) + 32);
     443    pgealist = xmallocz(sizeof(GEA2LIST) + 32,pszSrcFile,__LINE__);
    448444    if (pgealist) {
    449       memset(pgealist,0,sizeof(GEA2LIST) + 32);
    450445      pgea = &pgealist->list[0];
    451446      strcpy(pgea->szName,SUBJECT);
    … …  
    453448      pgea->oNextEntryOffset = 0;
    454449      pgealist->cbList = (sizeof(GEA2LIST) + pgea->cbName);
    455       pfealist = malloc(1532);
     450      pfealist = xmallocz(1532,pszSrcFile,__LINE__);
    456451      if (pfealist) {
    457         memset(pfealist,0,1532);
    458452        pfealist->cbList = 1024;
    459453        eaop.fpGEA2List = pgealist;
    … …  
    493487    CHAR      *value;
    494488
    495     pgealist = malloc(sizeof(GEA2LIST) + 32);
     489    pgealist = xmallocz(sizeof(GEA2LIST) + 32,pszSrcFile,__LINE__);
    496490    if (pgealist) {
    497       memset(pgealist,0,sizeof(GEA2LIST) + 32);
    498491      pgea = &pgealist->list[0];
    499492      strcpy(pgea->szName,LONGNAME);
    … …  
    501494      pgea->oNextEntryOffset = 0;
    502495      pgealist->cbList = (sizeof(GEA2LIST) + pgea->cbName);
    503       pfealist = malloc(1532);
     496      pfealist = xmallocz(1532,pszSrcFile,__LINE__);
    504497      if (pfealist) {
    505         memset(pfealist,0,1532);
    506498        pfealist->cbList = 1024;
    507499        eaop.fpGEA2List = pgealist;
    … …  
    700692
    701693  ulFileCnt = ulM;
    702   pszFileSpec = malloc(CCHMAXPATH + 2);
    703   paffbFound = malloc((ulM + 1) * sizeof(FILEFINDBUF4));
    704   papffbSelected = malloc((ulM + 1) * sizeof(PFILEFINDBUF4));
     694  pszFileSpec = xmalloc(CCHMAXPATH + 2,pszSrcFile,__LINE__);
     695  paffbFound = xmalloc((ulM + 1) * sizeof(FILEFINDBUF4),pszSrcFile,__LINE__);
     696  papffbSelected = xmalloc((ulM + 1) * sizeof(PFILEFINDBUF4),pszSrcFile,__LINE__);
    705697  if (paffbFound && papffbSelected && pszFileSpec) {
    706698    t = strlen(szDirBase);
    … …  
    767759                                  MPFROMLONG(ulExtraBytes),
    768760                                  MPFROMLONG(ulFileCnt));
    769             if (pciFirst)
    770             {
     761            if (!pciFirst) {
     762              Win_Error(hwndCnr,HWND_DESKTOP,pszSrcFile,__LINE__,
     763                        GetPString(IDS_FILLDIRALLOCERRTEXT));
     764              ok = FALSE;
     765              ullTotalBytes = 0;
     766            }
     767            else {
    771768              register INT   i;
    772769
    … …  
    803800                                  MPFROMP(&ri)))
    804801                  {
    805                     Win_Error(hwndCnr,HWND_DESKTOP,__FILE__,__LINE__,
    806                               GetPString(IDS_FILLDIRERR2TEXT));
     802                    Win_Error(hwndCnr,HWND_DESKTOP,pszSrcFile,__LINE__,
     803                              GetPString(IDS_FILLDIRINSERRTEXT));
    807804                    ok = FALSE;
    808805                    ullTotalBytes = 0;
    … …  
    824821              }
    825822            }
    826             else
    827             {
    828               Win_Error(hwndCnr,HWND_DESKTOP,__FILE__,__LINE__,
    829                         GetPString(IDS_FILLDIRERR3TEXT));
    830               ok = FALSE;
    831               ullTotalBytes = 0;
    832             }
    833823            if (ok)
    834824            {
    … …  
    839829          else
    840830          {
    841             paffbTemp = realloc(paffbTotal,sizeof(FILEFINDBUF4) *
    842                                      (ulFileCnt + ulTotal));
     831            paffbTemp = xrealloc(paffbTotal,
     832                                 sizeof(FILEFINDBUF4) * (ulFileCnt + ulTotal),
     833                                 pszSrcFile,__LINE__);
    843834            if (paffbTemp)
    844835            {
    … …  
    888879                              MPFROMLONG(ulExtraBytes),
    889880                              MPFROMLONG(ulTotal));
    890         if (pciFirst)
    891         {
     881        if (!pciFirst) {
     882          Win_Error(hwndCnr,HWND_DESKTOP,pszSrcFile,__LINE__,
     883                    GetPString(IDS_FILLDIRALLOCERRTEXT));
     884          ok = FALSE;
     885          ullTotalBytes = 0;
     886        }
     887        else {
    892888          register INT   i;
    893889
    … …  
    924920                             MPFROMP(pciFirst),MPFROMP(&ri)))
    925921              {
    926                 Win_Error(hwndCnr,HWND_DESKTOP,__FILE__,__LINE__,
    927                           GetPString(IDS_FILLDIRERR5TEXT));
     922                Win_Error(hwndCnr,HWND_DESKTOP,pszSrcFile,__LINE__,
     923                          GetPString(IDS_FILLDIRINSERRTEXT));
    928924                ok = FALSE;
    929925                ullTotalBytes = 0;
    … …  
    943939          }
    944940        }
    945         else
    946         {
    947           Win_Error(hwndCnr,HWND_DESKTOP,__FILE__,__LINE__,
    948                     GetPString(IDS_FILLDIRERR3TEXT));
    949           ok = FALSE;
    950           ullTotalBytes = 0;
    951         }
    952941        if (ok)
    953942        {
    … …  
    10531042              rc,
    10541043              HWND_DESKTOP,
    1055               __FILE__,
     1044              pszSrcFile,
    10561045              __LINE__,
    1057               GetPString(IDS_FILLDIRERR6TEXT));
     1046              GetPString(IDS_FILLDIRQCURERRTEXT));
    10581047    exit(0);
    10591048  }
    … …  
    10661055                          MPFROMLONG(EXTRA_RECORD_BYTES2),
    10671056                          MPFROMLONG((ULONG)numtoinsert));
    1068   if (pciFirst)
    1069   {
     1057  if (!pciFirst) {
     1058    Win_Error(hwndCnr,hwndCnr,pszSrcFile,__LINE__,
     1059              GetPString(IDS_FILLDIRALLOCERRTEXT));
     1060    exit(0);
     1061  }
     1062  else {
    10701063    pci = pciFirst;
    10711064    for(x = 0;x < 26;x++) {
    … …  
    12611254                      MPFROMP(pciFirst),
    12621255                      MPFROMP(&ri)))
    1263         Win_Error(hwndCnr,hwndCnr,__FILE__,__LINE__,
    1264                   GetPString(IDS_FILLDIRERR5TEXT));
     1256        Win_Error(hwndCnr,hwndCnr,pszSrcFile,__LINE__,
     1257                  GetPString(IDS_FILLDIRINSERRTEXT));
    12651258    }
    12661259    /* move cursor onto the default drive rather than the first drive */
    … …  
    13681361                                MPFROMP(pci),
    13691362                                MPFROMP(&ri))) {
    1370                   Win_Error(hwndCnr,hwndCnr,__FILE__,__LINE__,
    1371                             GetPString(IDS_FILLDIRERR5TEXT));
     1363                  Win_Error(hwndCnr,hwndCnr,pszSrcFile,__LINE__,
     1364                            GetPString(IDS_FILLDIRINSERRTEXT));
    13721365                  WinSendMsg(hwndCnr,
    13731366                             CM_FREERECORD,
    … …  
    14811474    }
    14821475
    1483   }
    1484   else
    1485   {
    1486     Win_Error(hwndCnr,hwndCnr,__FILE__,__LINE__,
    1487               GetPString(IDS_FILLDIRERR7TEXT));
    1488     exit(0);
    14891476  }
    14901477  if (!drivesbuilt && hwndMain)
Note: See TracChangeset for help on using the changeset viewer.