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

我有这段代码：

string s= "stringTest";

Encrypt(&s);

它只是调用传递字符串指针的函数。

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

4

1 回答 1

0

该答案将通过示例代码重申评论中已经提出的要点。

您当前的代码：

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() 方法调用返回的指针值中抛弃-ness。

这应该立即成为一个危险信号；有时，抛弃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;
}
于 2018-03-16T11:00:03.740 回答