Доступ к нарушению исключения перед входом в функцию

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-ness из значения указателя, возвращенного из 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;
}