Question

我有这个函数只是加密一个字符串（这个函数工作正常，并经过测试）。

DWORD SomeObj::Encrypt(string * To_Enc) {
    DWORD text_len = (To_Enc->length());
    if (!CryptEncrypt(this->hKey,
        NULL,  // hHash = no hash
        1,  // Final
        0,     // dwFlags
       (PBYTE)(*To_Enc).c_str(), //*pbData
       &text_len,  //*pdwDataLen
       128)) {      //dwBufLen
       return SERVER_ERROR;
    }
    return SERVER_SUCCESS;
}

我有这段代码：

string s= "stringTest";

Encrypt(&s);

简单地调用传递字符串指针的函数。

该程序在调用函数Encrypt(&s)时正在导致访问冲突异常，我猜这是关于参数&s正在传递的内容，但我无法弄清楚这一点。根据您的经验有什么想法？

Answer 1

这个答案将重申已经在评论中提出的重要观点，包括示例代码。

您当前的代码：

DWORD SomeObj::Encrypt(string * To_Enc) {
    DWORD text_len = (To_Enc->length());
    if (!CryptEncrypt(this->hKey,
        NULL,  // hHash = no hash
        1,  // Final
        0,     // dwFlags
       (PBYTE)(*To_Enc).c_str(), //*pbData
       &text_len,  //*pdwDataLen
       128)) {      //dwBufLen
       return SERVER_ERROR;
    }
    return SERVER_SUCCESS;
}

在线：

(PBYTE)(*To_Enc).c_str(), //*pbData

请注意，您正在从const（）方法调用返回的指针值中丢弃c_str - 。

这应该立即成为一面红旗;有时可能会抛弃const - ness是一个有效的用例，但它不是规则的例外。

未经测试，但使用临时的可变缓冲区可以解决您的问题，例如：

#include <cstddef>
#include <vector>
...
DWORD SomeObj::Encrypt(string * To_Enc) {
    std::vector<std::string::value_type> vecBuffer(To_Enc->length() * 3, 0);  // use the maximum value that could be output, possibly some multiple of the length of 'To_Enc'
    std::size_t nIndex = 0; 
    for (auto it = To_Enc->cbegin(); it != To_End->cend(); ++it)
    {
        vecBuffer[nIndex++] = *it;
    }
    DWORD text_len = (To_Enc->length());
    if (!CryptEncrypt(this->hKey,
        NULL,  // hHash = no hash
        1,  // Final
        0,     // dwFlags
       reinterpret_cast<PBYTE>(&vecBuffer[0]), //*pbData
       &text_len,  //*pdwDataLen
       vecBuffer.size())) {      //dwBufLen
       return SERVER_ERROR;
    }
    To_Enc->assign(&vecBuffer[0], text_len);  // assumes 'text_len' is returned with the new length of the buffer
    return SERVER_SUCCESS;
}

进入函数前访问冲突异常

1 个答案: