C++ 类到 Void 指针，数据丢失？

Question

(这是在 GameDev 中错误发布的。感谢那里的版主在正确的位置提出问题!)

我正在将一个简单的结构转换为一个空指针，这样它就可以在网络上发送出去，但是由于我不知道的原因，它切断了我的第二个变量。

我发现了一些类似的主题，特别是 this one ，这是一个很大的帮助。我有理由相信这是一个“新手错误”，或者我忽略了一些非常简单的事情。

欢迎任何意见，提前致谢!

/* struct provided */
struct Login_Struct
{
/*000*/ char username[32];
/*032*/ char password[64];
/*096*/
};


void NetWrapper::sendLogin(std::string user, std::string pass)
{
    // ensure size constraints are met
    if( user.length() > sizeof(Login_Struct::username)
    ||  pass.length() > sizeof(Login_Struct::password)) { return; }

    std::string hashPass = Security::strSHA256(pass); // returns hashed password

    Login_Struct login; // empty struct
    //strncpy(login.username, user.c_str(), user.length()); // i.e 'bob'
    //strncpy(login.password, hashPass.c_str(), hashPass.length()); // hashed password (64 char)
    memcpy(login.username, user.c_str(), user.length()); 
    memcpy(login.password, hashPass.c_str(), hashPass.length());

    char* pData = new char[sizeof(Login_Struct)];
    //memcpy(static_cast<void*>(pData), static_cast<void*>(&login), sizeof(Login_Struct)); // serialize data for sending
    memcpy(pData, &login, sizeof(Login_Struct)); // serialize data for sending

    // if we have data, send it and cleanup
    if(pData) { m_network->send(pData, OP_Login); delete pData; } // Network::send(char* d, short opCode)
    pData = nullptr;
}

此外，调试输出看起来像这样(为了便于阅读从代码中省略，如果需要/请求可以提供):

NetWrapper::sendLogin: Login_Struct Dump [bob] (bob), [5e884898da28047151d0e56f8dc6292773603d0d6aabbdd62a11ef721d1542d8] (password)
DEBUG: username maxlength [32], user length [3]
DEBUG: password maxlength [64], pass length [64]
DEBUG: pData [x=�i=], [0x7ffc5e4fd0e0], sizeof(Login_Struct) [96], sizeof(login) [96]
NetWrapper::sendLogin: New pData! pData [x=�i=]
NetWrapper::sendLogin: Post memcpy! pData [bob]
Network::send : Sending [d0c8504000100bob] ## ignore characters preceding "bob"

@wondra 在我位置不正确的帖子中提到，这是空终止符为我杀死它的问题。我已进行调整以复制没有空终止符的字符串，但我要么误解了它的放置方式，要么这不是问题所在。

---

编辑:添加带有调试废话的代码以帮助理解这一点。感谢到目前为止所有回复的人! (是的，我对使用 cout spam 感到尴尬。愿意接受关于更好/更清洁方法的建议!)

void NetWrapper::sendLogin(std::string user, std::string pass)
{
    if( user.length() > sizeof(Login_Struct::username)
    ||  pass.length() > sizeof(Login_Struct::password)) { return; }

    std::string hashPass = Security::strSHA256(pass);

    Login_Struct login;
    memset(&login, 0, sizeof(Login_Struct)); // clear buffer
    strncpy(login.username, user.c_str(), user.length());
    strncpy(login.password, hashPass.c_str(), hashPass.length());
    //memcpy(login.username, user.c_str(), user.length());
    //memcpy(login.password, hashPass.c_str(), hashPass.length());

    std::cout<<"NetWrapper::sendLogin: Login_Struct Dump ["<<login.username<<"] ("<<user<<"), ["<<login.password<<"] ("<<pass<<")\n";
    std::cout<<"DEBUG: username maxlength ["<<sizeof(login.username)<<"], user length ["<<user.length()<<"], user size ["<<sizeof(user)<<"]\n";
    std::cout<<"DEBUG: password maxlength ["<<sizeof(login.password)<<"], pass length ["<<hashPass.length()<<"], pass size ["<<sizeof(hashPass)<<"]\n";

    char* pData = new char[sizeof(Login_Struct)];
    std::cout<<"DEBUG: pData ["<<pData<<"], sizeof(pData) ["<<sizeof(pData)<<"], sizeof(&pData) ["<<sizeof(&pData)<<"], ["<<static_cast<void*>(&login)<<"], sizeof(Login_Struct) ["<<sizeof(Login_Struct)<<"], sizeof(login) ["<<sizeof(login)<<"]\n";
    std::cout<<"NetWrapper::sendLogin: New pData! pData ["<<pData<<"], sizeof(pData) ["<<sizeof(pData)<<"], sizeof(&pData) ["<<sizeof(&pData)<<"]\n";
    /// ------------------
    //memcpy(static_cast<void*>(pData), static_cast<void*>(&login), sizeof(Login_Struct)); // serialize data for sending
    memcpy(pData, &login, sizeof(Login_Struct)); // serialize data for sending
    /// ------------------
    std::cout<<"NetWrapper::sendLogin: Post memcpy! pData ["<<pData<<"], sizeof(pData) ["<<sizeof(pData)<<"], sizeof(&pData) ["<<sizeof(&pData)<<"], sizeof(*pData) ["<<sizeof(*pData)<<"]\n";

    // if we have data, send it and cleanup
    if(pData) { m_network->send(pData, OP_Login); delete pData; }
    pData = nullptr;
}

Answer 1

从问题中不清楚您的问题到底是什么。不过，我发现有几个潜在的问题。

您查看代码是否正常的方法将不起作用。

std::cout<<"NetWrapper::sendSerialized: Post memcpy! pData ["<<pData<<"], sizeof(pData) ["<<sizeof(pData)<<"], sizeof(&pData) ["<<sizeof(&pData)<<"], sizeof(*pData) ["<<sizeof(*pData)<<"]\n";

char 数组 pData 将包含两个包含两个字符串的 char 缓冲区。如果您的测试名称是“bob”，那么后面会有一个空字符，这就是它将使用 cout 显示的全部内容。您需要一个函数来转储缓冲区的内容，或者可能执行类似 printf("password='%.*s'", 64, pData + 32) 的操作。那是你要弄明白的。

另一个问题是您正在使用 memcpy() 但之前没有清除缓冲区的内存。在您复制的字符串之后缓冲区中的内容是未定义的。

您通过网络发送它，这样您的字符串就不会以空值终止。那可能没问题，也可能不行。这取决于预期的结果。如果接收端的某些东西期望有效的空终止 C 字符串，那肯定会导致问题。或者也尝试对它们使用类似 printf() 的东西。

这将解决不首先清除内存的问题，假设它发送 not 作为空终止字符串:

memset(&login, 0, sizeof(login));
strncpy(login.username, user.c_str(), sizeof(login.username)); 
strncpy(login.password, hashPass.c_str(), sizeof(login.password));

第一个 memset() 将所有字节清除为 0，而 strncpy() 确保复制的字符数不超过目标缓冲区中可用的字符数。

如果问题是它确实需要一个空终止的 C 字符串，那么您需要将 +1 添加到 char 数组的大小以容纳空终止符，然后将它们复制为:

memset(&login, 0, sizeof(login));
strncpy(login.username, user.c_str(), sizeof(login.username)-1); 
strncpy(login.password, hashPass.c_str(), sizeof(login.password)-1);