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

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

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 の前に行う

次ページ

続きを表示

1を表示