Changeset 613


Ignore:
Timestamp:
Apr 19, 2007, 2:06:38 AM (18 years ago)
Author:
Steven Levine
Message:

Correct selective logging checks

File:
1 edited

Legend:

Unmodified
Added
Removed

  • TabularUnified trunk/dll/error.c

    r552 r613  
    77
    88  Copyright (c) 1993-98 M. Kimes
    9   Copyright (c) 2004, 2006 Steven H. Levine
     9  Copyright (c) 2004, 2007 Steven H. Levine
    1010
    1111  12 Aug 04 SHL Comments
    … …  
    2121  16 Aug 06 SHL Tweak message formatting
    2222  07 Jan 07 GKY Move error strings etc. to string file
     23  18 Apr 07 SHL showMsg: correct selective logging checks
    2324
    2425***********************************************************************/
    … …  
    4041#pragma alloc_text(FMINPUT,Win_Error,Dos_Error,saymsg,showMsg)
    4142
    42 static APIRET showMsg(ULONG mb_type, HWND hwnd, PCSZ pszTitle, PCSZ pszMsg);
     43static APIRET showMsg(ULONG mb_type, HWND hwnd, PCSZ pszTitle, PCSZ pszMsg, BOOL wantLog);
    4344
    4445//== Win_Error: report Win...() error using passed message string ===
    … …  
    6364  // Format callers message
    6465  va_start(va, pszFmt);
    65   vsprintf(szMsg, pszFmt, va);
    66   va_end(va);
     66  szMsg[sizeof(szMsg) - 1] = 0;
     67  vsprintf(szMsg, pszFmt, va);
     68  va_end(va);
     69
     70  if (szMsg[sizeof(szMsg) - 1]) {
     71    fprintf(stderr, "Buffer overflow in Win_Error - need %u bytes\n", strlen(szMsg) + 1);
     72    fflush(stderr);
     73  }
    6774
    6875  if (strchr(szMsg, ' ') == NULL)
    … …  
    7784  if (!pErrInfoBlk) {
    7885    psz = szMsg + strlen(szMsg);
    79     strcpy(psz, " WinGetError failed.");
     86    strcpy(psz, " WinGetErrorInfo failed.");
    8087  }
    8188  else {
    … …  
    98105  }
    99106
    100   showMsg(MB_ENTER | MB_ICONEXCLAMATION, hwndOwner, GetPString(IDS_GENERR2TEXT),        // Titlebar message
    101           szMsg);                       // Formatted message
     107  showMsg(MB_ENTER | MB_ICONEXCLAMATION, hwndOwner, GetPString(IDS_GENERR2TEXT),
     108          szMsg, TRUE);
    102109
    103110}                                       // Win_Error
    … …  
    131138  // Format caller's message
    132139  va_start(va, pszFmt);
    133   vsprintf(szMsg, pszFmt, va);
    134   va_end(va);
     140  szMsg[sizeof(szMsg) - 1] = 0;
     141  vsprintf(szMsg, pszFmt, va);
     142  va_end(va);
     143
     144  if (szMsg[sizeof(szMsg) - 1]) {
     145    fprintf(stderr, "Buffer overflow in Dos_Error - need %u bytes\n", strlen(szMsg) + 1);
     146    fflush(stderr);
     147  }
    135148
    136149  if (strchr(szMsg, ' ') == NULL)
    … …  
    179192  }
    180193
    181   return showMsg(mb_type | MB_ICONEXCLAMATION, hwndOwner, GetPString(IDS_DOSERR2TEXT),  // Title
    182                  szMsg);
     194  return showMsg(mb_type | MB_ICONEXCLAMATION, hwndOwner, GetPString(IDS_DOSERR2TEXT),
     195                 szMsg, TRUE);
    183196
    184197}                                       // Dos_Error
    … …  
    203216  // Format caller's message
    204217  va_start(va, pszFmt);
    205   vsprintf(szMsg, pszFmt, va);
    206   va_end(va);
     218  szMsg[sizeof(szMsg) - 1] = 0;
     219  vsprintf(szMsg, pszFmt, va);
     220  va_end(va);
     221
     222  if (szMsg[sizeof(szMsg) - 1]) {
     223    fprintf(stderr, "Buffer overflow in Runtime_Error - need %u bytes\n", strlen(szMsg) + 1);
     224    fflush(stderr);
     225  }
    207226
    208227  if (strchr(szMsg, ' ') == NULL)
    … …  
    210229
    211230  sprintf(szMsg + strlen(szMsg),
    212           GetPString(IDS_GENERR1TEXT), pszSrcFile, uSrcLineNo);
    213 
    214   showMsg(MB_ICONEXCLAMATION, HWND_DESKTOP, DEBUG_STRING, szMsg);
     231          GetPString(IDS_GENERR1TEXT), pszSrcFile, uSrcLineNo);
     232
     233  showMsg(MB_ICONEXCLAMATION, HWND_DESKTOP, DEBUG_STRING, szMsg, TRUE);
    215234
    216235}                                       // Runtime_Error
    … …  
    234253
    235254  va_start(va, pszFmt);
    236   vsprintf(szMsg, pszFmt, va);
    237   va_end(va);
    238 
    239   return showMsg(mb_type, hwnd, pszTitle, szMsg);
     255  szMsg[sizeof(szMsg) - 1] = 0;
     256  vsprintf(szMsg, pszFmt, va);
     257  va_end(va);
     258
     259  if (szMsg[sizeof(szMsg) - 1]) {
     260    fprintf(stderr, "Buffer overflow in saymsg - need %u bytes\n", strlen(szMsg) + 1);
     261    fflush(stderr);
     262  }
     263
     264  return showMsg(mb_type, hwnd, pszTitle, szMsg, FALSE);
    240265
    241266}                                       // saymsg
    … …  
    243268//=== showMsg: display error popup ===
    244269
    245 static APIRET showMsg(ULONG mb_type, HWND hwnd, PCSZ pszTitle, PCSZ pszMsg)
    246 {
    247   if ((mb_type & (MB_YESNO | MB_YESNOCANCEL)) == 0) {
     270static APIRET showMsg(ULONG mb_type, HWND hwnd, PCSZ pszTitle, PCSZ pszMsg, BOOL wantLog)
     271{
     272  if (wantLog) {
    248273    fputs(pszMsg, stderr);
    249274    fputc('\n', stderr);
    … …  
    260285                       (PSZ) pszMsg, (PSZ) pszTitle, 0, // help id
    261286                       mb_type | MB_MOVEABLE);
    262 }                                       // showMsg
     287} // showMsg
Note: See TracChangeset for help on using the changeset viewer.