变量“temp”周围的堆栈已损坏

Question

我正在尝试将包含由空格分隔的多个值的 std::string 转换为向量对象。我让它工作（意味着调试器在算法结束时在向量中显示正确的值），但是我遇到了一个我以前从未见过的异常​​。例外情况如下：

我以前从未见过这种情况，我已经研究了一个小时来研究导致这种情况的原因，但我仍然不明白。

导致异常的算法如下：

vector<BYTE> ConvertStringOfValuesToVectorOfBytes(std::string valStr)
{
    vector<string> vectorOfValues = split(valStr, ' '); //split string into vector of strings (works)
    vector<BYTE> Hex;
    vector<const char*> vectOfCStrings;

    for(int i = 0; i < vectorOfValues.size(); i++)
    {
        const char* temp = vectorOfValues[i].c_str();
        vectOfCStrings.push_back(temp);
    }

    //now we have a vector of c strings...

    for(int i = 0; i < vectOfCStrings.size(); i++)
    {       
        char temp = 0;
        sscanf(vectOfCStrings[i], "%x", &temp);
        Hex.push_back(temp);
    }

    return Hex;

} //<-- debugger gets to here and on the next step causes the exception

我不知道如何解决这个问题。正如我所说，算法在我得到正确结果的意义上“有效”，我只需要清除异常。我该如何解决这个问题？

Answer 1

这行不对：

sscanf(vectOfCStrings[i], "%x", &temp);

%x 格式用于int，而不是char。

因为sscanf 需要int 的地址，所以您会遇到堆栈损坏。它尝试使用您传递的地址作为能够持有int 的地址。它显然使用了比有效内存更多的内存。

你需要的是：

for(int i = 0; i < vectOfCStrings.size(); i++)
{       
    int temp = 0;
    sscanf(vectOfCStrings[i], "%x", &temp);
    Hex.push_back(static_cast<char>(temp));
}

Answer 2

std::vector<unsigned char> func(std::string valStr) {
    vector<string> vectorOfValues = split(valStr); //split string into vector of strings (works)
    vector<BYTE> Hex;
    vector<const char*> vectOfCStrings;

    for(int i = 0; i < vectorOfValues.size(); i++) {
        const char* temp = vectorOfValues[i].c_str();
        vectOfCStrings.push_back(temp);
    }

    //now we have a vector of c strings...

    for(int i = 0; i < vectOfCStrings.size(); i++) {
        int temp = 0;
        sscanf(vectOfCStrings[i], "%x", &temp);
        Hex.push_back(char(temp));
    }

    return Hex;
}

所以是的，我刚刚将第二个 tmp 设为 int，因为 sscanf 和类似的东西使用 int 作为类型，并且由于 byte 小于 int，这会导致您的溢出。

不确定它是否有效，但它不再给我错误。

Answer 3

罪魁祸首似乎在这两行：

    char temp = 0;
    sscanf(vectOfCStrings[i], "%x", &temp);

这正是 C++ 中不使用 printf/scanf 系列函数并被std::iostream 机制取代的原因。当您指定 %x 时，函数 sscanf 期望获得指向 unsigned int 的指针，但您将指针传递给 char，而 sscanf 本身无法验证这一点。尽管一些现代编译器可以检查传递给 printf 或 scanf 的值是否与格式说明符匹配并生成警告，否则通常不起作用（例如传递给函数的格式，而不是使用字符串文字）。所以这里 sscanf 取消引用指针 &temp 作为指向 unsigned int 的指针，并将临时变量周围的内存覆盖为 char == 1 的 sizeof。

Answer 4

你的算法到底想完成什么？

从外观上看，您正在将 std::string 分解为“单词”的 std::vector，然后将这些单词转换为 c 风格的字符串（为什么？），然后组装一个 std:vector 包含每个单词的第一个字母被解释为一个 int。使用 std:stringstream 代替：

for ( auto& word : vectorOfValues )
{
    for ( auto& letter : word ) 
    {
        unsigned int val;
        std::stringstream ss;
        ss << std::hex << word[0];
        ss >> val;
        Hex.push_back( static_cast<BYTE>( val ) );
    }
}

通常，如果您不直接与 C API 交互，则不应在 c++ 中使用 c 风格的字符串。