为什么在将 LPBYTE 复制到 LPTSTR（剪贴板）期间使用 memcpy 抛出异常？

Question

我有一个 LPBYTE 数组（从文件中获取），我需要将其复制到 LPTSRT （实际上是复制到剪贴板中）。问题是复制工作但不稳定，有时会抛出异常（并非总是如此），我不明白为什么。代码是：

     FILE *fConnect = _wfopen(connectFilePath, _T("rb"));
  if (!fConnect)
   return;
  fseek(fConnect, 0, SEEK_END);
  lSize = ftell(fConnect);
  rewind(fConnect);

  LPBYTE lpByte = (LPBYTE) malloc(lSize);  
  fread(lpByte, 1, lSize, fConnect); 
  lpByte[lSize] = 0;
  fclose(fConnect);

  //Copy into clipboard
  BOOL openRes = OpenClipboard(NULL);
  if (!openRes)
   return;
  DWORD err = GetLastError();

  EmptyClipboard(); 
  HGLOBAL hText;
  hText = GlobalAlloc(GMEM_MOVEABLE, (lSize+ sizeof(TCHAR)));

  LPTSTR sMem = (TCHAR*)GlobalLock(hText); 
  memcpy(sMem, lpByte, (lSize + sizeof(TCHAR)));

最后一个字符串就是抛出异常的地方。 多谢

Answer 1

我并不是说，这是你问题的原因，但它可能是或可能是将来其他问题的原因。

如果您像这样分配内存

LPBYTE lpByte = (LPBYTE) malloc(lSize);  

这是对已分配内存块之外的访问：

lpByte[lSize] = 0;

分配的内存大小为lSize，并且包含&lpByte[0]形式的位置&lpByte[lSize - 1] 包括在内。

编辑：

正如 Hans 注意到的，memcpy 还访问分配块之外的内存。如果 sizeof(TCHAR) 为 1，则最后读取的字节为 lpByte[lSize]，如果 sizeof(TCHAR) 大于 1，则为 bytes过去的 lpByte[lSize] 也会被读取或至少尝试被读取。

Answer 2

我不确定是什么导致您的代码出现问题，但是以下代码可以工作，并且所有内容都锁定/复制，很好（请注意，您的剪贴板操作可以轻松注释掉，并且不会影响问题的根源）：

   LPBYTE lpByte = (LPBYTE)malloc(512);  
   lpByte[0] = 'A';
   lpByte[1] = 'B';
   lpByte[2] = '0';

   // OpenClipboard(NULL);
   // EmptyClipboard(); 

   HGLOBAL hText;
   hText = GlobalAlloc(GMEM_MOVEABLE, 512);

   LPTSTR sMem = (TCHAR*)GlobalLock(hText); 
   memcpy(sMem, lpByte, 512);

您可以尝试在异常发生之前在代码中设置断点（实际上可能有不同的原因）。

Answer 3

GlobalAlloc 或 GlobalLock 有效吗？输入一些错误检查代码并查看，两者都应该返回非 NULL 值。

Answer 4

_wfopen 是 fopen 的宽字符版本 - 您应该传递 L"..."，而不是 TCHAR。 TCHAR 版本是 _tfopen （将归结为 fopen 或 _wfopen 之一） - 请参阅：http://msdn.microsoft.com/en-us/library/yeby3zcb%28VS.80 %29.aspx

LPBYTE lpByte = (LPBYTE) malloc(lSize);

如果这是 C，则实际上不需要转换 malloc 的结果。就我个人而言，MS LP* 类型给我留下了不好的印象 - 我觉得匈牙利语掩盖了精通 C 语言的人的代码可读性。因此，我更喜欢 BYTE * 优于 LPBYTE，但它不会创建或破坏代码。

fread(lpByte, 1, lSize, fConnect);

检查返回值。

lpByte[lSize] = 0;

正如其他人提到的，此内存访问超出了数组范围。

if (!openRes)
    return;
DWORD err = GetLastError();

• 你泄漏了 lpByte
• 你调用了 GetLastError() ...成功了吗？

接下来，

LPTSTR sMem = (TCHAR*)GlobalLock(hText);

虽然我更喜欢非 LP 的东西，但也许选择一个？ （也许可以进行转换LPTSTR？）同样，最终这并不重要。 （这可能属于“它是一个 malloc，也不需要强制转换”。）

memcpy(sMem, lpByte, (lSize + sizeof(TCHAR)));

正如其他人提到的，这个 memcpy 也访问无效内存。具体来说，lpByte 的长度为 lSize，但您在执行此操作时还加上了额外的 sizeof(TCHAR)。