Changeset 1565 for trunk/dll/select.c


Ignore:
Timestamp:
May 31, 2011, 8:37:01 PM (14 years ago)
Author:
Steven Levine
Message:

Ensure mask->pszMasks[1] initialize to NULL if not used

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/dll/select.c

    r1544 r1565  
    77
    88  Copyright (c) 1993-98 M. Kimes
    9   Copyright (c) 2004, 2008 Steven H. Levine
     9  Copyright (c) 2004, 2011 Steven H. Levine
    1010
    1111  01 Aug 04 SHL Rework lstrip/rstrip usage
     
    3030  12 Jan 08 SHL Localize SpecialSelect in comp.c
    3131  29 Feb 08 GKY Use xfree where appropriate
     32  31 May 11 SHL Ensure mask->pszMasks[1] initialize to NULL if not used
    3233
    3334***********************************************************************/
     
    151152            *p = 0;
    152153        }
    153         /* input now contains name of file to select */
     154        // Input now contains name of file to select
    154155        pci = FindCnrRecord(hwndCnr,
    155156                            input, pciParent, partial, partial, TRUE);
    156         if (pci)                        /* found it? */
     157        if (pci)                        // Found it?
    157158          WinSendMsg(hwndCnr,
    158159                     CM_SETRECORDEMPHASIS,
     
    162163        else
    163164          errs++;
    164         if (errs > 50) {                /* prevent runaway on bad file */
     165        if (errs > 50) {                // Prevent runaway on bad file
    165166
    166167          APIRET ret;
     
    230231              pszToMatch = file;
    231232            if (*Mask.pszMasks[x] != '/') {
    232               if (wildcard(pszToMatch, Mask.pszMasks[x], TRUE)) {
     233              if (wildcard(pszToMatch, Mask.pszMasks[x], TRUE)) {
    233234                markit = TRUE;
    234235              }
     
    253254          ULONG pos;
    254255          LONG len;
    255           FILE *inputFile;
    256           CHAR *moderb = "rb";
    257 
    258           if ((inputFile = xfsopen(pci->pszFileName, moderb, SH_DENYNO,
    259                                    pszSrcFile, __LINE__, TRUE)) != NULL) {
     256          FILE *inputFile;
     257          CHAR *moderb = "rb";
     258
     259          if ((inputFile = xfsopen(pci->pszFileName, moderb, SH_DENYNO,
     260                                   pszSrcFile, __LINE__, TRUE)) != NULL) {
    260261            pos = ftell(inputFile);
    261262            while (!feof(inputFile)) {
     
    359360          LONG len;
    360361          FILE *inputFile;
    361           CHAR *moderb = "rb";
    362 
    363           if ((inputFile = xfsopen(pci->pszFileName, moderb, SH_DENYNO,
    364                                    pszSrcFile, __LINE__, TRUE)) != NULL) {
     362          CHAR *moderb = "rb";
     363
     364          if ((inputFile = xfsopen(pci->pszFileName, moderb, SH_DENYNO,
     365                                   pszSrcFile, __LINE__, TRUE)) != NULL) {
    365366            pos = ftell(inputFile);
    366367            while (!feof(inputFile)) {
     
    529530//== SetMask() Convert mask string to array of pointers to masks ==
    530531
    531 VOID SetMask(PSZ maskstr, MASK * mask)
     532VOID SetMask(PSZ maskstr, MASK *mask)
    532533{
    533534  UINT x;
     
    546547      p++;                              // Find separator
    547548    if (*p) {
    548       *p = 0;                           // Replace ;
     549      *p = 0;                           // Replace ; will nul to terminate string
    549550      p++;
    550551    }
    551552  }                                     // for
    552553  mask->pszMasks[x] = NULL;             // Mark end
     554  if (!x)
     555    mask->pszMasks[1] = NULL;           // Need 1 more for multiple mask detect 2011-05-31 SHL
    553556  DosExitCritSec();
    554557}
     
    655658    return;
    656659
    657   /* count directory containers, build array of hwnds */
     660  // Count directory containers, build array of hwnds
    658661  henum = WinBeginEnumWindows(hwndParent);
    659662  while ((hwnd = WinGetNextWindow(henum)) != NULLHANDLE) {
     
    684687               hwndCnr,
    685688               UM_NOTIFY, MPFROMP(GetPString(IDS_BUILDINGLISTSTEXT)), MPVOID);
    686     DosSleep(0); //26 Aug 07 GKY 1
    687   }
    688 
    689   /* count records, build array of pointers to records */
     689    DosSleep(0);                        // Allow other windows to update
     690  }
     691
     692  // Count records, build array of pointers to records
    690693  for (z = 0; z < numwindows; z++) {
    691694    pci = (PCNRITEM) WinSendMsg(Cnrs[z].hwndCnr,
     
    714717                                  MPFROM2SHORT(CMA_NEXT, CMA_ITEMORDER));
    715718    }
    716     DosSleep(0);  //26 Aug 07 GKY 1
     719    DosSleep(0);                        // Allow other windows to update
    717720    Cnrs[z].numfiles = x;
    718721    if (Cnrs[z].numfiles)
     
    739742              Cnrs[z].ss[x].largest = FALSE;
    740743            cmp = TestCDates(&bsres->pci->date, &bsres->pci->time,
    741                              &Cnrs[z].ss[x].pci->date, &Cnrs[z].ss[x].pci->time);
     744                             &Cnrs[z].ss[x].pci->date, &Cnrs[z].ss[x].pci->time);
    742745              /*(Cnrs[z].ss[x].pci->date.year >
    743746               bsres->pci->date.year) ? TRUE : (Cnrs[z].ss[x].pci->date.year <
     
    837840                     MPFROM2SHORT(TRUE, CRA_SELECTED));
    838841      }
    839       DosSleep(0); //26 Aug 07 GKY 1
     842      DosSleep(0);                      // Allow other windows to update
    840843    }
    841844    break;
     
    848851                     MPFROM2SHORT(TRUE, CRA_SELECTED));
    849852      }
    850       DosSleep(0);  //26 Aug 07 GKY 1
     853      DosSleep(0);                      // Allow other windows to update
    851854    }
    852855    break;
     
    859862                     MPFROM2SHORT(TRUE, CRA_SELECTED));
    860863      }
    861       DosSleep(0);  //26 Aug 07 GKY 1
     864      DosSleep(0);                      // Allow other windows to update
    862865    }
    863866    break;
     
    870873                     MPFROM2SHORT(TRUE, CRA_SELECTED));
    871874      }
    872       DosSleep(0); //26 Aug 07 GKY 1
     875      DosSleep(0);                      // Allow other windows to update
    873876    }
    874877    break;
     
    881884                     MPFROM2SHORT(TRUE, CRA_SELECTED));
    882885      }
    883       DosSleep(0); //26 Aug 07 GKY 1
     886      DosSleep(0);                      // Allow other windows to update
    884887    }
    885888    break;
     
    892895                     MPFROM2SHORT(TRUE, CRA_SELECTED));
    893896      }
    894       DosSleep(0); //26 Aug 07 GKY 1
     897      DosSleep(0);                      // Allow other windows to update
    895898    }
    896899    break;
     
    903906                     MPFROM2SHORT(TRUE, CRA_SELECTED));
    904907      }
    905       DosSleep(0); //26 Aug 07 GKY 1
     908      DosSleep(0);                      // Allow other windows to update
    906909    }
    907910    break;
     
    915918                     MPFROM2SHORT(FALSE, CRA_SELECTED));
    916919      }
    917       DosSleep(0); //26 Aug 07 GKY 1
     920      DosSleep(0);                      // Allow other windows to update
    918921    }
    919922    break;
     
    926929                     MPFROM2SHORT(FALSE, CRA_SELECTED));
    927930      }
    928       DosSleep(0); //26 Aug 07 GKY 1
     931      DosSleep(0);                      // Allow other windows to update
    929932    }
    930933    break;
     
    937940                     MPFROM2SHORT(FALSE, CRA_SELECTED));
    938941      }
    939       DosSleep(0); //26 Aug 07 GKY 1
     942      DosSleep(0);                      // Allow other windows to update
    940943    }
    941944    break;
     
    948951                     MPFROM2SHORT(FALSE, CRA_SELECTED));
    949952      }
    950       DosSleep(0); //26 Aug 07 GKY 1
     953      DosSleep(0);                      // Allow other windows to update
    951954    }
    952955    break;
     
    959962                     MPFROM2SHORT(FALSE, CRA_SELECTED));
    960963      }
    961       DosSleep(0); //26 Aug 07 GKY 1
     964      DosSleep(0);                      // Allow other windows to update
    962965    }
    963966    break;
     
    970973                     MPFROM2SHORT(FALSE, CRA_SELECTED));
    971974      }
    972       DosSleep(0); //26 Aug 07 GKY 1
     975      DosSleep(0);                      // Allow other windows to update
    973976    }
    974977    break;
     
    981984                     MPFROM2SHORT(FALSE, CRA_SELECTED));
    982985      }
    983       DosSleep(0); //26 Aug 07 GKY 1
     986      DosSleep(0);                      // Allow other windows to update
    984987    }
    985988    break;
Note: See TracChangeset for help on using the changeset viewer.