C ++类到空指针，数据丢失了吗？

Question

（这是在GameDev中的错误发布。感谢那里的主持人在正确的位置发问！）

我正在将简单的结构强制转换为void指针，以便可以在网络上将其发送出去，但是由于我未知的原因，它切断了我的第二个变量。

我发现了一些类似的线程，尤其是这个线程，它提供了很大的帮助。 我可以肯定地说这是一个“新手错误”，或者我忽略了一个非常简单的事情。

任何输入表示赞赏，所以在此先感谢您！

/* 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在我的位置不正确的帖子中提到，这是空终止符为我杀死它的问题。 我已经进行了调整，以复制不带null终止符的字符串，但是我误解了它的放置方式，或者这不是问题。

---

编辑：添加带有调试废话的代码可帮助实现此目的。 感谢到目前为止的所有答复！ （是的，我很尴尬地诉诸垃圾邮件。欢迎接受有关更好/更清洁方法的建议！）

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()但之前并未清除缓冲区的内存。 待复制字符串之后的缓冲区中的内容是不确定的。

您正在通过网络发送此消息，因此您的字符串可能最终不会被null终止。 也许可以，也可以不。 这取决于期望的结果。 如果接收端的某些对象期望一个有效的以null终止的C字符串，肯定会引起问题。 或者也尝试在其上使用诸如printf()的东西。

这将解决不首先清除内存（假设它不以空终止字符串发送）的问题：

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()确保复制的字符数不超过目标缓冲区中可用的字符数。

如果问题在于它确实期望以null结尾的C字符串，那么您需要在char数组的大小上加上+1以容纳null终止符，然后将其复制为：

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);

Answer 2

原始提问者的解决方案：

解答：这是我不清除缓冲区，然后被cout和null终止符捕获的问题。 下面的位正在工作（下面是调试输出）。 非常感谢MrJLP的时间/精力！

 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()); 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)]; memcpy(pData, &login, sizeof(Login_Struct)); // serialize data for sending Login_Struct dslogin; memset(&dslogin, 0, sizeof(Login_Struct)); // clear buffer memcpy(&dslogin, pData, sizeof(Login_Struct)); std::cout<<"NetWrapper::sendLogin: Test Dump ["<<dslogin.username<<"] ["<<dslogin.password<<"]\\n"; // if we have data, send it and cleanup if(pData) { m_network->send(pData, OP_Login); delete pData; } pData = nullptr; } 

并调试输出：

 NetWrapper::sendLogin: Login_Struct Dump [bob] (bob), [5e884898da28047151d0e56f8dc6292773603d0d6aabbdd62a11ef721d1542d8  lY ] (password) DEBUG: username maxlength [32], user length [3], user size [32] DEBUG: password maxlength [64], pass length [64], pass size [32] NetWrapper::sendLogin: Test Dump [bob] [5e884898da28047151d0e56f8dc6292773603d0d6aabbdd62a11ef721d1542d8]