-1

我在使用 memcpy 时遇到了一些麻烦，因为当执行 memcpy 操作时，我得到：

"ÍÍWF03-021913.datýýý««««««««þ"

当我应该得到：

“WF03-021913.datýýý«««««««««þ”

我不知道这些领先的“ÍÍ”是从哪里来的。

代码：

注意：lpszFileName = "WF03-021913.dat"

typedef struct {
    BYTE  cbRequestType;
    BYTE  cbFileName;   
    char* szFileName;      
} UFTP_GET_FILE_INFO_REQUEST;

BOOL Uftp_BuildFileInfoRequest(PUFTP_REQUEST request, LPCTSTR lpszFileName)
{
    UFTP_GET_FILE_INFO_REQUEST *fileInfo;
    int fileNameLen;

    if (lpszFileName == NULL) {
        ASSERT( 0 );
        return FALSE;
    }

    fileNameLen = strlen( lpszFileName );
    if (fileNameLen == 0)
        return FALSE;

    request->dwRequestSize = sizeof(UFTP_GET_FILE_INFO_REQUEST) - 
                             sizeof(void*) + fileNameLen;
    request->RequestBuffer = malloc( request->dwRequestSize );
    if ( !request->RequestBuffer ) {
        TRACE0("Failed to allocate RequestBuffer");
        return FALSE;
    }

    fileInfo = (UFTP_GET_FILE_INFO_REQUEST*) request->RequestBuffer;
    fileInfo->cbRequestType = UFTP_GET_FILE_INFO;
    fileInfo->cbFileName = fileNameLen;

    memcpy(&fileInfo->szFileName, lpszFileName, fileNameLen);
    return TRUE;
}
4

1 回答 1

1

我只是在这里猜测，但我的猜测是它fileInfo->szFileName是一个指针。这意味着它&fileInfo->szFileName是一个指向指针的指针，所以你复制到一个完整的其他内存区域。

此外，您不会复制'\0'所需的终止字符。fileNameLen + 1在分配和复制时都需要这样做。

如果您真的希望它全部在连续内存中，您可能应该将结构更改为以大小为零的字符数组结尾（您的编译器可能不支持，然后使用大小为 1 的数组）并sizeof(UFTP_GET_FILE_INFO_REQUEST) + fileNameLen + 1用作要分配的大小. 然后您可以将该数组用作普通字符串数组。

如果你解决了这些问题，你还有另一个问题：你没有初始化指针以指向分配的内存。这意味着它将指向一些随机内存。

所有这些错误都会导致未定义的行为，我会说你很幸运它没有崩溃。

于 2013-04-12T18:14:33.093 回答