Question

（这在GameDev中被错误地发布。感谢主持人在正确的位置询问！）

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

我找到了一些类似的主题，特别是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垃圾邮件感到尴尬。请接受有关更好/更清洁方法的建议！）

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'，那么之后将会有一个null char，这就是它将使用cout显示的所有内容。您需要一个函数来转储缓冲区的内容，否则可能会执行printf("password='%.*s'", 64, pData + 32)之类的操作。那是你要弄明白的。

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

您是通过网络发送的，因此您的字符串最终不会被终止。那可能没问题，或者可能没有。这取决于预期的内容。如果接收端的某些东西期望有效的空终止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()确保不复制比目标缓冲区中可用的字符更多的字符。

如果问题是确实期望空终止的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:

答案：这是我没有清除缓冲区然后我的问题被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]

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

2 个答案: