c++ - 读取 REG_BINARY 会损坏方法的参数

Question

我正在尝试使用以下代码读取 REG_BINARY。 value 以正确的字符串结尾，但在 return 语句后出现访问冲突错误。经过一些调试，我发现方法的参数在执行RegQueryValueExW后被乱码覆盖(例如，name以“test”开头，变成了“ε\0εειôvv¦ &...”在 RegQueryValueExW 之后)。

string GetREG_BINARYValue(HKEY hkey, DWORD cbData, wstring name)
{
    DWORD* data = new DWORD[cbData]; 
    DWORD size = cbData;

    if( RegQueryValueExW(hkey,name.c_str(), NULL, NULL, reinterpret_cast<LPBYTE>(&data), &size)  != ERROR_SUCCESS)
    {
        RegCloseKey(hkey);
        return "Could not read REG_BINARY registry value";
    }    

    string value =  ByteArrayToString((BYTE*)&data, size);
    return value;
}

函数是从这个方法调用的:

string RegistryReader::ReadRegValue(HKEY root, string key, string name)
{
    HKEY hkey;
    DWORD type;
    DWORD cbData;
    string result;
    wstring wkey, wname;
    std::wstring_convert<std::codecvt_utf8_utf16<wchar_t>> converter;
    wkey=converter.from_bytes(key);
    wname=converter.from_bytes(name);

    if (RegOpenKeyExW(root, wkey.c_str(), 0, KEY_ACCESS_DEF64 , &hkey) != ERROR_SUCCESS)
    {
        if (RegOpenKeyExW(root, wkey.c_str(), 0,  KEY_ACCESS_DEF32, &hkey) != ERROR_SUCCESS)
        {
            return "Could not find registry key";      
        }  
    }

    if (RegQueryValueExW(hkey, wname.c_str(), NULL, &type, NULL, &cbData) != ERROR_SUCCESS)
    {
        RegCloseKey(hkey);
        return "Could not open registry value";
    }

    switch (type)
    {
    case REG_SZ:
        result=GetREG_SZValue(hkey, cbData, wname);
        break;
    case REG_DWORD:
        result=GetREG_DWORDValue(hkey, cbData, wname);
        break;
    case REG_BINARY:
        result=GetREG_BINARYValue(hkey, cbData, wname); //<--Here
        break;
    default:
        return "Invalid data type";
        break;
    }
    return result;
}

起初我以为这是一个数组大小问题，但我最终得到了正确的结果，而且 cbData 始终包含我期望的大小。我可能在使用 RegQueryValueExW 时做错了什么，但我很难找到明确的文档。这段代码有什么明显的错误吗？

Answer 1

您的代码有很多问题。我在对问题的评论中列出了很多。根本问题是您必须向 RegQueryValueExW 传递一个指向长度为 size 的数组的指针。但你没有。

现在，data 是一个指向 DWORD 数组的指针，长度为 size。所以它包含的字节数是你需要的四倍。这本身并没有那么糟糕。但不是传递 data，而是传递 &data。这是指针的地址而不是数组的地址。然后，当 API 函数覆盖指针而不是写入它指向的缓冲区时，您将调用未定义的行为。

您必须施放的事实应该是第一个警告信号。尽量不要投。我再说一遍，尽量不要施法。当你进行强制转换时，你会阻止编译器发现你的错误。当编译器告诉你你犯了一个错误时，不要用强制转换忽略它。听听。

如果我们必须使用这些全部大写的 Windows 类型，则需要分配一个 BYTE 数组:

BYTE* data = new BYTE[cbData]; 

现在您可以将data 传递给API 函数而不是&data。显然也将 data 传递给 ByteArrayToString。并记得在用完数组后将其删除。

正如我在对问题的评论中所说，还有很多其他错误，我真的不想调试和纠正所有这些错误。希望我在该评论中给了您一个很好的指导，剩下的就交给您了。