Changeset 1628


Ignore:
Timestamp:
Aug 27, 2011, 9:35:39 PM (14 years ago)
Author:
Gregg Young
Message:

Hard coded the flags for the xDosAlloc* wrappers and added a description for each of them.

Location:
trunk/dll
Files:
14 edited

Legend:

Unmodified
Added
Removed

  • trunk/dll/autoview.c

    r1627 r1628  
    146146  }
    147147
    148   if (xDosAllocMem((PPVOID) &pfealist, ealen, PAG_COMMIT | PAG_READ |
    149                    PAG_WRITE, pszSrcFile, __LINE__))
     148  if (xDosAllocMem((PPVOID) &pfealist, ealen, pszSrcFile, __LINE__))
    150149    ret = FALSE;
    151150  else {

  • trunk/dll/copyf.c

    r1627 r1628  
    328328  else
    329329    ealen = sizeof(FEALIST) + 10;
    330   if (xDosAllocMem((PPVOID) &pfealist,
    331                    ealen + 32L, PAG_COMMIT | PAG_READ | PAG_WRITE,
    332                    pszSrcFile, __LINE__))
     330  if (xDosAllocMem((PPVOID) &pfealist, ealen + 32L, pszSrcFile, __LINE__))
    333331    return FALSE;
    334332  else {

  • trunk/dll/datamin.c

    r1627 r1628  
    856856        ULONG numprocs = 0, numthreads = 0;
    857857
    858         if (!xDosAllocMem((PVOID)&pbh, USHRT_MAX + 4096,
    859                           PAG_COMMIT | PAG_READ | PAG_WRITE,
    860                           pszSrcFile, __LINE__)) {
     858        if (!xDosAllocMem((PVOID)&pbh, USHRT_MAX + 4096, pszSrcFile, __LINE__)) {
    861859          if (DosQProcStatus((ULONG *)pbh, USHRT_MAX))
    862860            noqproc = TRUE;
    … …  
    891889        ULONG numprocs = 0, numthreads = 0;
    892890
    893         if (!xDosAllocMem((PVOID) & pbh, USHRT_MAX + 4096,
    894                           PAG_COMMIT | PAG_READ | PAG_WRITE,
    895                           pszSrcFile, __LINE__)) {
     891        if (!xDosAllocMem((PVOID) & pbh, USHRT_MAX + 4096, pszSrcFile, __LINE__)) {
    896892          //2 Sep 07 GKY 0x05 = process & Mod data only
    897893          if (DosQuerySysState(QS_PROCESS | QS_MTE, 0, 0, 0, pbh, USHRT_MAX))

  • trunk/dll/eas.c

    r1627 r1628  
    204204
    205205            ealen = sizeof(FEA2LIST) + strlen(s) + 64;
    206             if (!xDosAllocMem((PPVOID) & pfealist, ealen + 1,
    207                               PAG_COMMIT | PAG_READ | PAG_WRITE,
    208                               pszSrcFile, __LINE__)) {
     206            if (!xDosAllocMem((PPVOID) & pfealist, ealen + 1, pszSrcFile, __LINE__)) {
    209207              memset(pfealist, 0, ealen + 1);
    210208              pfealist->cbList = ealen;
    … …  
    11051103  }
    11061104
    1107   if (!xDosAllocMem((PPVOID) & pfealist, ealen,
    1108                     PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__)) {
     1105  if (!xDosAllocMem((PPVOID) & pfealist, ealen, pszSrcFile, __LINE__)) {
    11091106    memset(pfealist, 0, ealen);
    11101107    pfealist->list[0].oNextEntryOffset = 0;

  • trunk/dll/instant.c

    r1627 r1628  
    107107        mem = MLEgetlen(hwndMLE);
    108108        if (mem) {
    109           if (xDosAllocMem((PVOID) & bat, mem,
    110                            PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__)) {
     109          if (xDosAllocMem((PVOID) & bat, mem, pszSrcFile, __LINE__)) {
    111110            WinDismissDlg(hwnd, 0);
    112111            break;

  • trunk/dll/killproc.c

    r1627 r1628  
    129129
    130130  WinSendDlgItemMsg(hwnd, KILL_LISTBOX, LM_DELETEALL, MPVOID, MPVOID);
    131   if (!xDosAllocMem((PVOID) & pbh, USHRT_MAX + 4096,
    132                     PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__)) {
     131  if (!xDosAllocMem((PVOID) & pbh, USHRT_MAX + 4096, pszSrcFile, __LINE__)) {
    133132    rc = DosQProcStatus((ULONG *)pbh, USHRT_MAX);
    134133    if (!rc) {
    … …  
    194193
    195194  WinSendDlgItemMsg(hwnd, KILL_LISTBOX, LM_DELETEALL, MPVOID, MPVOID);
    196   if (!xDosAllocMem((PVOID) & pbh, USHRT_MAX + 4096,
    197                     PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__)) {
     195  if (!xDosAllocMem((PVOID) & pbh, USHRT_MAX + 4096, pszSrcFile, __LINE__)) {
    198196    rc = DosQuerySysState(QS_PROCESS | QS_MTE, 0, 0, 0, pbh, USHRT_MAX);
    199197    if (!rc) {

  • trunk/dll/misc.c

    r1627 r1628  
    884884          else
    885885            ealen = sizeof(FEALIST) + 9;
    886           if (!xDosAllocMem((PPVOID) & pfealist, ealen + 64,
    887                             PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__)) {
     886          if (!xDosAllocMem((PPVOID) & pfealist, ealen + 64, pszSrcFile, __LINE__)) {
    888887            memset(pfealist, 0, ealen + 1);
    889888            pfealist->cbList = ealen;

  • trunk/dll/mle.c

    r1627 r1628  
    183183  if (len) {
    184184    len++;
    185     if (!xDosAllocMem((PVOID) & temp, 4096,
    186                       PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__)) {
     185    if (!xDosAllocMem((PVOID) & temp, 4096, pszSrcFile, __LINE__)) {
    187186      ancpos = MLEancpos(h);
    188187      curpos = MLEcurpos(h);
    … …  
    261260  if (!sel)
    262261    return FALSE;
    263   if (xDosAllocMem((PVOID) & temp, 32768L,
    264                    PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__)) {
     262  if (xDosAllocMem((PVOID) & temp, 32768L, pszSrcFile, __LINE__)) {
    265263    free(sel);
    266264#   ifdef FORTIFY
    … …  
    533531    DosChgFilePtr(handle, 0, FILE_BEGIN, &action);
    534532    if (len) {
    535       if (xDosAllocMem((PVOID) & hexbuff, 50001,
    536                        PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__))
     533      if (xDosAllocMem((PVOID) & hexbuff, 50001, pszSrcFile, __LINE__))
    537534        ret = FALSE;
    538535      else {
    … …  
    659656    fseek(fp, 0L, SEEK_SET);
    660657    if (len && len != -1) {
    661       if (xDosAllocMem((PVOID) & buffer,
    662                         50000L, PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__))
     658      if (xDosAllocMem((PVOID) & buffer, 50000L, pszSrcFile, __LINE__))
    663659        ret = FALSE;
    664660      else {
    … …  
    907903  }
    908904
    909   if (xDosAllocMem((PVOID) & buffer, 4096L,
    910                     PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__))
     905  if (xDosAllocMem((PVOID) & buffer, 4096L, pszSrcFile, __LINE__))
    911906    ok = FALSE;
    912907  else {

  • trunk/dll/saveclip.c

    r1627 r1628  
    104104        if (clip)
    105105          len += strlen(clip) + 1;
    106         if (!xDosAllocSharedMem((PPVOID) &hold, (PSZ) NULL, len, PAG_COMMIT |
    107                                 OBJ_GIVEABLE | PAG_READ | PAG_WRITE, __FILE__, __LINE__)) {
     106        if (!xDosAllocSharedMem((PPVOID) &hold, (PSZ) NULL, len, __FILE__, __LINE__)) {
    108107          *hold = 0;
    109108          if (clip)

  • trunk/dll/subj.c

    r1627 r1628  
    128128      else
    129129        ealen = sizeof(FEALIST) + 9;
    130       if (!xDosAllocMem((PPVOID) & pfealist, ealen + 1L,
    131                         PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__)) {
     130      if (!xDosAllocMem((PPVOID) & pfealist, ealen + 1L, pszSrcFile, __LINE__)) {
    132131        memset(pfealist, 0, ealen + 1);
    133132        pfealist->cbList = ealen;

  • trunk/dll/systemf.c

    r1627 r1628  
    770770    hwnd = HWND_DESKTOP;
    771771
    772   if (xDosAllocMemLow((PVOID)&pszPgm,
    773                        MaxComLineStrg,
    774                        PAG_COMMIT | PAG_READ | PAG_WRITE,
    775                        pszSrcFile,__LINE__))
     772  if (xDosAllocMemLow((PVOID)&pszPgm, MaxComLineStrg, pszSrcFile,__LINE__))
    776773    return -1; //already complained
    777774  *szSavedir = 0;
    … …  
    815812      temp = *p;
    816813      if (temp) {
    817         if (xDosAllocMemLow((PVOID)&pszArgs, MaxComLineStrg * 2,
    818                              PAG_COMMIT | PAG_READ | PAG_WRITE,
    819                             pszSrcFile, __LINE__)) {
     814        if (xDosAllocMemLow((PVOID)&pszArgs, MaxComLineStrg * 2, pszSrcFile, __LINE__)) {
    820815          DosFreeMem(pszPgm);
    821816          return -1;   //already complained
    … …  
    958953      if (~type & FULLSCREEN)
    959954        type |= WINDOWED;
    960       if (xDosAllocMemLow((PVOID) &pszArgs, MaxComLineStrg * 2,
    961                            PAG_COMMIT | PAG_READ | PAG_WRITE,
    962                            pszSrcFile, __LINE__)) {
     955      if (xDosAllocMemLow((PVOID) &pszArgs, MaxComLineStrg * 2, pszSrcFile, __LINE__)) {
    963956        DosFreeMem(pszPgm);
    964957        return -1;   //already complained

  • trunk/dll/valid.c

    r1627 r1628  
    309309
    310310# define BUFFER_BYTES 8192
    311   if (xDosAllocMem(&pvBuffer, BUFFER_BYTES,
    312                     PAG_COMMIT | PAG_READ | PAG_WRITE, pszSrcFile, __LINE__))
     311  if (xDosAllocMem(&pvBuffer, BUFFER_BYTES, pszSrcFile, __LINE__))
    313312    return -1;                          // Say failed
    314313  szPath[0] = chDrive;

  • trunk/dll/wrappers.c

    r1627 r1628  
    6767}
    6868
     69/**
     70 * xDosAllocSharedMem uses OBJ_ANY on systems that support high memory use
     71 * and falls back to low memory allocation where it is not supported.
     72 * Flags are hard coded PAG_COMMIT | OBJ_GIVEABLE | PAG_READ | PAG_WRITE
     73 * The wrapper provides error checking.
     74 */
     75
    6976APIRET xDosAllocSharedMem(PPVOID ppb,
    7077                          PSZ pszName,
    7178                          ULONG cb,
    72                           ULONG flag,
    7379                          PCSZ pszSrcFile,
    7480                          UINT uiLineNumber)
    … …  
    7682  APIRET rc; ;
    7783
    78   rc = DosAllocSharedMem(ppb, pszName, cb, flag | OBJ_ANY);
     84  rc = DosAllocSharedMem(ppb, pszName, cb,
     85                         PAG_COMMIT | OBJ_GIVEABLE | PAG_READ | PAG_WRITE | OBJ_ANY);
    7986  //DbgMsg(pszSrcFile, __LINE__, "ppb %p", *ppb);
    8087  if (rc)
    81     rc = DosAllocSharedMem(ppb, pszName, cb, flag);
     88    rc = DosAllocSharedMem(ppb, pszName, cb, PAG_COMMIT | OBJ_GIVEABLE | PAG_READ | PAG_WRITE);
    8289  if (rc)
    8390    Runtime_Error(pszSrcFile, uiLineNumber, GetPString(IDS_OUTOFMEMORY));
    … …  
    8592}
    8693
     94/**
     95 * xDosAllocMem uses OBJ_ANY on systems that support high memory use
     96 * and falls back to low memory allocation where it is not supported.
     97 * Flags are hard coded PAG_COMMIT | PAG_READ | PAG_WRITE.
     98 * The wrapper provides error checking.
     99 */
     100
    87101APIRET xDosAllocMem(PPVOID ppb,
    88102                    ULONG cb,
    89                     ULONG flag,
    90103                    PCSZ pszSrcFile,
    91104                    UINT uiLineNumber)
    … …  
    93106  APIRET rc;
    94107
    95   rc = DosAllocMem(ppb, cb, flag | OBJ_ANY);
     108  rc = DosAllocMem(ppb, cb, PAG_COMMIT | PAG_READ | PAG_WRITE | OBJ_ANY);
    96109  //DbgMsg(pszSrcFile, uiLineNumber, "ppb %p %x", *ppb, rc);
    97110  if (rc)
    98     rc = DosAllocMem(ppb, cb, flag);
     111    rc = DosAllocMem(ppb, cb, PAG_COMMIT | PAG_READ | PAG_WRITE);
    99112  if (rc)
    100113    Runtime_Error(pszSrcFile, uiLineNumber, GetPString(IDS_OUTOFMEMORY));
    … …  
    103116}
    104117
     118/**
     119 * xDosAllocMemLow doesn't use OBJ_ANY. It should be used when the buffer
     120 * is going to be used by 16 functions that fail to thunk high memory addresses properly
     121 * such as DosQueryAppType, DosOpenL, DosGetMessage  and DosReadQueue (probably others)
     122 * Flags are hard coded PAG_COMMIT | PAG_READ | PAG_WRITE.
     123 * The wrapper provides error checking.
     124 */
     125
    105126APIRET xDosAllocMemLow(PPVOID ppb,
    106127                       ULONG cb,
    107                        ULONG flag,
    108128                       PCSZ pszSrcFile,
    109129                       UINT uiLineNumber)
    … …  
    111131  APIRET rc;
    112132
    113   rc = DosAllocMem(ppb, cb, flag);
     133  rc = DosAllocMem(ppb, cb, PAG_COMMIT | PAG_READ | PAG_WRITE);
    114134  if (rc)
    115135    Runtime_Error(pszSrcFile, uiLineNumber, GetPString(IDS_OUTOFMEMORY));

  • trunk/dll/wrappers.h

    r1627 r1628  
    2828                          PSZ pszName,
    2929                          ULONG cb,
    30                           ULONG flag,
    3130                          PCSZ pszSrcFile,
    3231                          UINT uiLineNumber);
    … …  
    3433APIRET xDosAllocMem(PPVOID ppb,
    3534                    ULONG cb,
    36                     ULONG flag,
    3735                    PCSZ pszSrcFile,
    3836                    UINT uiLineNumber);
    … …  
    4038APIRET xDosAllocMemLow(PPVOID ppb,
    4139                       ULONG cb,
    42                        ULONG flag,
    4340                       PCSZ pszSrcFile,
    4441                       UINT uiLineNumber);
Note: See TracChangeset for help on using the changeset viewer.