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

Question

（这在 GameDev 中被错误地 post。感谢那里的版主在正确的位置提出问题！）

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

我找到了一些类似的话题，尤其是 this one，这对我帮助很大。我相当确定这是一个 "newbie mistake" 或者我忽略了一些非常简单的事情。

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

/* 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;
}

此外，调试输出看起来像这样（为了便于阅读从代码中省略，如果 needed/requested 可以提供）：

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 在我的错误定位 post 中提到，这是空终止符为我杀死它的问题。我已经进行了调整以复制没有空终止符的字符串，但我要么误解了它的放置方式，要么这不是问题所在。

编辑：添加带有调试废话的代码以帮助理解这一点。感谢到目前为止所有回复的人！（是的，我对使用 cout spam 感到尴尬。愿意接受关于 better/cleaner 方法的建议！）

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 结尾。那可能没问题，也可能不行。这取决于预期的结果。如果接收端的某些东西期望有效的空终止 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);

Answer 2

Solution by the original asker:

ANSWER: This was an issue of me not clearing the buffer and then me getting caught by cout and a null terminator. The bit below is working (and debug output below). Huge thanks to MrJLP for the time/effort!

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

And debug output:

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]

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

C++ Class to Void Pointer, Data Lost?

c++

serialization

network-programming

void-pointers