【初心者歓迎】C/C++室 Ver.59【環境依存OK】

【初心者歓迎】C/C++� ..

396:デフォルトの名無しさん
08/04/05 14:33:43
>>390
lpszBuf = (TCHAR*)GlobalLock(hMem);
ReadFile(hFile, lpszBuf, dwSize, &dwAccBytes, NULL);
CloseHandle(hFile);
lstrcat(lpszBuf, szTime);
lpszBuf[dwSize] = '\0';

このあたりか。
何か無茶苦茶危険なコードに見えるんだけど。

hMem = GlobalAlloc(GHND, sizeof(TCHAR) * dwSize + 1);　　← dwSize 文字分のメモリを確保してるつもりのようだが、+1 の位置がおかしい（TCHAR が２バイトの時に領域が足りない）
lpszBuf = (TCHAR*)GlobalLock(hMem);
ReadFile(hFile, lpszBuf, dwSize, &dwAccBytes, NULL);　　← dwSize 読み込む
CloseHandle(hFile);
lstrcat(lpszBuf, szTime);　← '\0' を付けないまま lstrcat は危険というか、szTime の文字数分領域がさらに必要なんだけどその領域を確保すらしていない
lpszBuf[dwSize] = '\0';　← lstrcat の前に行う

次ページ