GetModuleFileNameEx，访问被拒绝错误

Question

我正在尝试获取所有打开进程的名称。 这就是我所拥有的：

   #include "stdafx.h"
#include <Psapi.h>

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, PSTR pCmdLine, int iCmdShow)
{
    bool _result;
    DWORD *pProcessIds = new DWORD[1000];
    DWORD cb;
    DWORD pBytesReturned;

    _result =  EnumProcesses(pProcessIds, 1000, &pBytesReturned);

    HANDLE hndProccesse;


    for (int i = 0; i < pBytesReturned / sizeof(DWORD); ++i)
    {
        hndProccesse =   OpenProcess(STANDARD_RIGHTS_ALL,  false, *pProcessIds);
        DWORD _len;

        DWORD _len2 =0;
        LPWSTR lpFilename = new WCHAR[100];
        _len =GetModuleFileNameEx(hndProccesse,NULL, lpFilename, _len2);
        DWORD _errr;
        _errr =  GetLastError();
        MessageBox(NULL, lpFilename, NULL, 0);
        CloseHandle(hndProccesse);

        pProcessIds ++;



    }

    return 0;
}

一切正常，直到GetModuleFileNameEx给出访问拒绝错误（5）。

这也是消息框中显示的内容：

有任何想法吗？

Answer 1

GetModuleFileNameEx()的第四个参数必须是在第三个参数中传递的数组的大小。 您传入_len2 。 但是您将_len2设置为零，而不是lpFilename数组的大小（100）。 因此， GetModuleFileNameEx()认为它无关紧要，甚至都没有触及lpFilename数组。 堆数据不一定要初始化为零，因此lpFilename仍然包含随机垃圾，因此是随机消息框的内容。

我将猜测GetModuleFileNameEx()返回零是因为它不需要编写任何内容，但是由于没有任何失败而没有设置最后的错误代码，因此该程序的较早部分遗留下了拒绝访问错误。

更多注意事项：

请记住，乔纳森·波特（Jonathan Potter）所说的有关检查Windows API函数返回的错误的正确方法。 您从_len GetModuleFileNameEx()返回值。 MSDN说， GetModuleFileNameEx()在出错时返回零。 因此，您需要在获取最后一个错误值之前检查_len以查看其是否等于零，否则它将没有意义。 如前所述，如果成功， GetModuleFileNameEx()不必清除最后一个错误值。

HANDLE hndProccesse = new HANDLE; 肯定是错误的，但不是程序中的错误（尽管这是内存泄漏！）。 HANDLE本身是一个指针，这就是允许new运行的原因。 但是这样做是没有意义的，因为HANDLE是由操作系统返回的，通常不应用作指针。 可以考虑将它们视为不透明值。

关于内存泄漏，您永远不要delete[]在循环中创建的每个lpFilename ，也不要delete[] pProcessIds 。 对于您上面发布的小程序来说，这可能并不重要，但是如果您的程序不断增长，那么您肯定会解决该问题。

通常，使用MAX_PATH作为文件名缓冲区的标称长度而不是100。这就是各种Shell函数使用的名称。 （更长的时间也不会伤害，但是会更短。）

Answer 2

您需要使用FormatMessage获得有用的错误描述。 下面显示了一个示例。此示例与您的示例完全相同，但所做的只是合并了错误检查。 它不能解决问题！

#include <windows.h>
#include <Psapi.h>

void ShowError(DWORD err)
{
    LPTSTR lpMsgBuf = nullptr;
    FormatMessageA(FORMAT_MESSAGE_ALLOCATE_BUFFER | FORMAT_MESSAGE_FROM_SYSTEM | FORMAT_MESSAGE_IGNORE_INSERTS, nullptr, err, MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT), reinterpret_cast<LPTSTR>(&lpMsgBuf), 0, nullptr);
    MessageBoxA(NULL, lpMsgBuf, "ERROR", 0);
    LocalFree(lpMsgBuf);
}

int main()
{
    DWORD* pProcessIds = new DWORD[1000];
    DWORD pBytesReturned = 0;

    bool res =  EnumProcesses(&pProcessIds[0], 1000, &pBytesReturned);

    if (!res)
    {
        DWORD err = GetLastError();
        MessageBoxW(NULL, L"ENUMPROCESSES FAILED", L"Error", 0);
        ShowError(err);
        delete[] pProcessIds;
        return EXIT_FAILURE;
    }

    for (unsigned int i = 0; i < pBytesReturned / sizeof(DWORD); ++i)
    {
        if (pProcessIds[i] == 0) //error.. process id is 0..
            continue;

        wchar_t lpFilename[256] = {0};
        HANDLE hndProccess =   OpenProcess(STANDARD_RIGHTS_ALL,  false, pProcessIds[i]);

        if (hndProccess == NULL || hndProccess == INVALID_HANDLE_VALUE)
        {
            DWORD err = GetLastError();
            MessageBoxW(NULL, L"FAILED TO OPEN PROCESS", L"ERROR", 0);
            ShowError(err);
            delete[] pProcessIds;
            return EXIT_FAILURE;
        }

        int len = GetModuleFileNameExW(hndProccess, NULL, lpFilename, sizeof(lpFilename) / sizeof(lpFilename[0]));
        if (len <= 0)
        {
            DWORD err = GetLastError();
            if (err)
            {
                MessageBoxW(NULL, L"FAILED TO GET MODULEFILENAME", L"ERROR", 0);
                ShowError(err);
                delete[] pProcessIds;
                return EXIT_FAILURE;
            }
        }          

        CloseHandle(hndProccess);

        MessageBoxW(NULL, lpFilename, L"NAME", 0);
    }

    delete[] pProcessIds;
    return 0;
}