Windows: Buffer too small

[LFX3 (Gast)]

Guten Tag,

ich habe aus dem Buch "C - Einführung und professionelle Anwendung" 
folgendes Beispiel in Visual C++ 2008 Express eingegeben:

#include <stdio.h>
#include <windows.h>

HINSTANCE hInst = 0;
char szAppName[] = "Basis";
char szTitle[100] = "Willys Titelleiste";

BOOL InitApplication ();
BOOL InitInstance (int);
LRESULT CALLBACK WndProc (HWND, UINT, WPARAM, LPARAM);

void onPaint (HWND);

int WINAPI WinMain (  HINSTANCE hInstance,
            HINSTANCE hPrevInstance,
            LPSTR    lpCmdLine,
            int       nCmdShow)
{

  MSG msg;
  hInst = hInstance;

  if (!InitApplication () ) return FALSE;
  if (!InitInstance (nCmdShow) ) return FALSE;

  while (GetMessage (&msg, NULL, 0, 0))
  {
    TranslateMessage (&msg);
    DispatchMessage (&msg);
  }



  return (msg.wParam);
}

BOOL InitApplication ()
{
  WNDCLASSEX wc;

  wc.cbSize = sizeof(WNDCLASSEX);
  wc.style = CS_HREDRAW | CS_VREDRAW;
  wc.lpfnWndProc = (WNDPROC)WndProc;
  wc.cbClsExtra = 0;
  wc.cbWndExtra = 0;
  wc.hInstance = hInst;
  wc.hIcon = LoadIcon ( NULL, IDI_QUESTION);
  wc.hCursor = LoadCursor ( NULL, IDC_WAIT);
  wc.hbrBackground = (HBRUSH)GetStockObject(GRAY_BRUSH);
  wc.lpszMenuName = NULL;
  wc.lpszClassName = szAppName;
  wc.hIconSm = LoadIcon (NULL, IDI_WINLOGO);

  return RegisterClassEx(&wc);
}


BOOL InitInstance (int nCmdShow)
{
  HWND hWnd;
  hWnd = CreateWindow(
      szAppName,
      szTitle,
      WS_OVERLAPPEDWINDOW,
      50,//CW_USEDEFAULT,
      100,
      500,
      250,
      NULL,
      NULL,
      hInst,
      NULL);
  if (!hWnd) return (FALSE);
  ShowWindow (hWnd, nCmdShow);
  UpdateWindow (hWnd);

  return (TRUE);
}


LRESULT CALLBACK WndProc (HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
{
  
  switch (message)
  {
  case WM_PAINT:
    onPaint(hWnd);
    break;
  case WM_DESTROY:
    PostQuitMessage(0);
    break;
  default:
    return DefWindowProc (hWnd, message, wParam, lParam);
  }

  return 0;
}



void onPaint (HWND hWnd)
{
  PAINTSTRUCT ps;
  HDC hdc;
  TEXTMETRIC tm;
  char szZahl[10];

  int i = 0;
  int nXStart1, nXStart2, nYStart;
  int nAveCharWidth, nLineHeight;

  char* strMetrik[] = {  "Zeichenhoehe: ",
              "Hoehe oberhalb der Basislinie:",
              "Hoehe unterhalb der Basislinie:",
              "Hoehe des Bereichs fuer Umlaute:",
              "Abstand zwischen den Zeilen:",
              "Mittlere Zeichenbreite:",
              "Maximale Zeichenbreite:"
            };

  hdc = BeginPaint (hWnd, &ps);

  GetTextMetrics(hdc, &tm);
  nLineHeight = (int)(tm.tmHeight + tm.tmExternalLeading)+2;
  nAveCharWidth = (int)tm.tmAveCharWidth;

  nXStart1 = 5*nAveCharWidth;
  nXStart2 = nXStart1 + 40*nAveCharWidth;

  nYStart = 1*nLineHeight;

  for (i=0; i < 7; i++)
  {
    TextOut (hdc, nXStart1, nYStart, strMetrik[i], strlen(strMetrik[i]) );

    switch(i)
    {
    case 0: sprintf_s(szZahl, "%31d", tm.tmHeight);
        break;
    case 1: sprintf_s(szZahl, "%31d", tm.tmAscent);
        break;
    case 2: sprintf_s(szZahl, "%31d", tm.tmDescent);
        break;
    case 3: sprintf_s(szZahl, "%31d", tm.tmInternalLeading);
        break;
    case 4: sprintf_s(szZahl, "%31d", tm.tmExternalLeading);
        break;
    case 5: sprintf_s(szZahl, "%31d", tm.tmAveCharWidth);
        break;
    case 6: sprintf_s(szZahl, "%31d", tm.tmMaxCharWidth);
        break;
    };

    TextOut(hdc, nXStart2, nYStart, szZahl, strlen(szZahl) );
    nYStart += nLineHeight;

  }

  EndPaint (hWnd, &ps);
}

Ich erhalte folgende Fehlermeldung:

Debug Assertion Failed!
Program
...ges\textmetric\Debug\textmetric.exe
File f:\dd\vctools\crt_bld\self_x86\crt\src\vsprintf.c
Line 244
Expression:("Buffer too small", 0)



Kann mir jemand sagen, woran das liegt?


LFX3

[Karl H. (kbuchegg)]

>  char szZahl[10];

>    case 0: sprintf_s(szZahl, "%31d", tm.tmHeight);


Wie soll denn eine Zahl, die du mit 31 Dezimalstellen als Text 
dargestellt haben willst (und der Text daher aus 31 Zeichen bestehen 
wird), in einen String passen, der gerade mal Platz für 9 Zeichen hat?

[Peter II (Gast)]

irgendwie stimmen die parameter nicht mit der doku überein

sprintf_s(szZahl, "%31d", tm.tmHeight);

http://msdn.microsoft.com/de-de/library/ce3zzk1k%28v=vs.80%29.aspx

int sprintf_s(
   char *buffer,
   size_t sizeOfBuffer,
   const char *format [,
      argument] ...
);

[LFX3 (Gast)]

Aua, danke für den Hinweis:


Statt

case 0: sprintf_s(szZahl, "%31d", tm.tmHeight);

muss es heißen:

case 0: sprintf_s(szZahl, "%3ld", tm.tmHeight);

VIELEN DANK
LFX3

[Karl H. (kbuchegg)]

LFX3 schrieb:

>

> case 0: sprintf_s(szZahl, "%3ld", tm.tmHeight);
>

Es muss vor allen Dingen

sprintf_s( szZahl, sizeof(szZahl), "%3ld", tm.tmHeight);

heissen.
Auch wenn Microsoft mittels Template ...

template <size_t size>
int sprintf_s(
   char (&buffer)[size],
   const char *format [,
      argument] ... 
); // C++ only

... eine Automatik eingebaut hat, die dann greift, wenn der Compiler das 
Array selber sehen kann. Denn irgendwann hast du dann diesen Fall nicht 
mehr und dann ist das Kopfkratzen groß, was denn das Problem ist. Nichts 
gegen Komfort, aber zuviel Komfort macht nachlässig.

Und ob die "ld" richtig sind, muss man in der TEXTMETRIC Struktur 
nachsehen. Das steht

typedef struct tagTEXTMETRIC {  /* tm */
    int  tmHeight;
    int  tmAscent;
    int  tmDescent;
    int  tmInternalLeading;
    int  tmExternalLeading;
    int  tmAveCharWidth;
    int  tmMaxCharWidth;
    int  tmWeight;
    BYTE tmItalic;
    BYTE tmUnderlined;
    BYTE tmStruckOut;
    BYTE tmFirstChar;
    BYTE tmLastChar;
    BYTE tmDefaultChar;
    BYTE tmBreakChar;
    BYTE tmPitchAndFamily;
    BYTE tmCharSet;
    int  tmOverhang;
    int  tmDigitizedAspectX;
    int  tmDigitizedAspectY;
} TEXTMETRIC;

Nö. ld ist definitiv falsch. tmHeight ist vom Datentyp int. Also muss 
der Formatspecifier ein "d" sein und kein "ld".


Leg das Buch zur Seite und kauf dir erst mal ein vernünftiges 
Einsteigerbuch für C. Der Klassiker "Kernighan&Ritchie" wird immer 
wieder gerne genommen.
Wenn du den durch hast und auch verstehst, dann fallen dir die Fehler in 
diesem Buch dann auch auf. Alleine der Titel muss schon stutzig machen. 
Zwischen "Einführung" und "professioneller Anwendung" liegen im 
günstigen Fall so ca. 3 bis 4 Jahre und eine halbes Bücherregal an 
Literatur.