RegSetValueEx - 路径错误，64 位，以管理员身份运行

Posted 2023-02-22

【中文标题】RegSetValueEx - 路径错误，64 位，以管理员身份运行

【英文标题】：RegSetValueEx - wrong path, 64bit, running as admin

【发布时间】：2015-03-31 20:35:44

【问题描述】：

我有一些关于注册表相关功能的问题。 我使用 RegSetValueEx 函数来创建注册表数据，但不幸的是我肯定做错了，但我不确定错误在哪里。我想在 HKEY_LOCAL_MACHINE - SOFTWARE\Microsoft\Windows\CurrentVersion\Run 中创建一个注册表，但它只在 HKEY_LOCAL_MACHINE 键中创建数据，仅此而已。

TCHAR name[UNLEN + 1];
DWORD size = UNLEN + 1;
GetUserName((TCHAR*)name, &size);
string namep = name;
string path = "C:\\Users\\" + namep + "\\AppData\\Roaming\\MyProgram\\MyProgram.exe";
if (RegOpenKeyEx(HKEY_LOCAL_MACHINE, "SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Run", 0, KEY_ALL_ACCESS, &hKey) == ERROR_SUCCESS)

    cout << "OPENED!";
    if (RegQueryValueEx(HKEY_LOCAL_MACHINE, "MyRegistry", 0, NULL, NULL, NULL) == ERROR_FILE_NOT_FOUND)
    
        cout << "NOT FOUND!"; // If registry not found, then create it?
        if (RegSetValueEx(HKEY_LOCAL_MACHINE, "MyRegistry", 0, REG_SZ, (const BYTE*)path.c_str(), path.size()) == ERROR_SUCCESS)
            cout << "REGISTRY WRITTEN";
        RegCloseKey(hKey);
    

好的，这就是代码，有什么问题？ 另外我想问一下怎么做，所以我不必以管理员身份运行它来创建注册表，因为如果我不以管理员身份运行它，它就不会创建注册表。 以及 64 位版本的操作系统如何？我在 32 位上编译，所以我必须做些什么才能在 64 位机器上运行？

谢谢。 =)

【问题讨论】：

我建议使用宽字符串而不是 TCHAR 来传递给 winapi 函数。您在调用函数时也假设TCHAR 是char，这违背了目的。

文档说 cbData 必须包含终止空字符的大小。 对于RegSetValueEx。

数据保存正确 o.0 但问题是它的路径不正确。我要试试宽弦的东西。

如果您传递 ANSI 数据，宽字符串不会产生影响，但您基本上有三种选择，TCHAR 表示项目的构建数量是两倍，而 ANSI 不支持绝大多数的字符。我认为TCHAR 在代码中也有更多的工作，而且 ANSI 与 Unicode 在那里是相等的，所以为什么不选择更好的呢？

【参考方案1】：

您的代码有几个问题。

首先，不要对文件路径进行硬编码，尤其是包含系统文件夹的路径。在这种情况下，您应该使用SHGetFolderPath(CSIDL_APPDATA) 或SHGetKnownFolderPath(FOLDERID_RoamingAppData) 来发现当前用户的AppData\Roaming 文件夹的位置。或者，如果 MyProgram.exe 是当前正在运行的应用程序，您可以改用 GetModuleFileName(0)。

其次，非管理员用户没有对HKEY_LOCAL_MACHINE 的写访问权，只有对HKEY_CURRENT_USER 的写访问权。如果您需要写信给HKEY_LOCAL_MACHINE，您的应用需要以提升的管理员权限运行。

第三，您没有正确使用RegQueryValueEx() 和RegSetValueEx()。具体来说：

您在第一个参数中传递了错误的HKEY。您需要将RegOpenKeyEx()返回给您的HKEY传递给您。

对于RegSetValueEx()，REG_SZ 值必须包含空终止符，但path.size() 不计算空终止符，因此您需要改用size()+1。

第四，你泄露了RegOpenKeyEx()返回给你的HKEY，因为只有当RegQueryValueEx()成功时你才调用RegCloseKey()。

说了这么多，试试这样的：

if (RegOpenKeyEx(HKEY_CURRENT_USER, TEXT("SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Run"), 0, KEY_ALL_ACCESS, &hKey) == ERROR_SUCCESS)

    cout << "OPENED! ";

    LONG lRet = RegQueryValueEx(hKey, TEXT("MyRegistry"), 0, NULL, NULL, NULL);
    if (lRet == ERROR_FILE_NOT_FOUND)
    
        cout << "NOT FOUND! ";

        // assuming you want to store the calling process's
        // filename, otherwise adjust this as needed ...
        TCHAR filename[MAX_PATH + 1];
        DWORD len = GetModuleFileName(NULL, filename, MAX_PATH);
        filename[len] = 0;

        if (RegSetValueEx(hKey, TEXT("MyRegistry"), 0, REG_SZ, (const BYTE*)filename, (len+1) * sizeof(TCHAR)) == ERROR_SUCCESS)
            cout << "WRITTEN!";
        else
            cout << "NOT WRITTEN!";
    
    else if (lRet == 0)
    
        cout << "FOUND!";
    
    else
        cout << "NOT QUERIED!";

    RegCloseKey(hKey);

else
    cout << "NOT OPENED!";

如果您只想确保注册表包含最新的文件名，您可以省略 RegQueryValueEx() 并让 RegSetValueEx() 覆盖现有值（如果存在）：

if (RegOpenKeyEx(HKEY_CURRENT_USER, TEXT("SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Run"), 0, KEY_ALL_ACCESS, &hKey) == ERROR_SUCCESS)

    cout << "OPENED! ";

    // assuming you want to store the calling process's
    // filename, otherwise adjust this as needed ...
    TCHAR filename[MAX_PATH + 1];
    DWORD len = GetModuleFileName(NULL, filename, MAX_PATH);
    filename[len] = 0;

    if (RegSetValueEx(hKey, TEXT("MyRegistry"), 0, REG_SZ, (const BYTE*)filename, (len+1) * sizeof(TCHAR)) == ERROR_SUCCESS)
        cout << "WRITTEN!";
    else
        cout << "NOT WRITTEN!";

    RegCloseKey(hKey);

else
    cout << "NOT OPENED!";

【讨论】：

好吧，我想你回答了我的问题。谢谢！ =)哦，还有一个问题没有回答。 64位机器怎么样？当我在 32 位机器上编译时，有什么需要避免或做的事情使它在 64 位机器上工作？

在 32 位机器上编译它和在 64 位机器上运行它没有任何区别。重要的是它是编译为 32 位还是 64 位可执行文件（32 位机器可以编译 64 位可执行文件）。当通过 WOW64 模拟器在 64 位机器上运行 32 位可执行文件时，这确实会影响注册表某些区域的访问方式。请参阅 MSDN：Registry Redirector。

@RemyLebeau，非常好。谢谢）