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）。

这也是在消息框中显示的内容： enter image description here

任何想法？

Answer 1

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

我猜测GetModuleFileNameEx()返回零，因为它不需要写任何东西，但没有设置最后一个错误代码，因为没有任何失败，所以留下了访问被拒绝的错误来自该计划的早期部分。

更多说明：

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

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

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

通常，使用{{1}}作为文件名缓冲区的标称长度而不是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;
}

GetModuleFileNameEx，访问被拒绝错误

2 个答案: