C++中IEnumWbemClassObject返回null，读取Win32_PhysicalMemoryArray属性失败求助

Let's work through the issues in your code that are causing the null enumerator and subsequent crash. The biggest problem right now is you're not validating any of your COM call return values—if even one step fails (like connecting to WMI or running the query), you'll end up with null pointers and crashes. Let's break this down and fix it:

Key Issues in Your Original Code #

• No HRESULT validation: Every COM function returns an HRESULT that tells you if the call succeeded. Skipping these checks means you're flying blind when something goes wrong.
• Missing WMI security setup: By default, the WMI connection might not have the right permissions to access system-level classes. You need to set a proxy blanket to enable proper authentication.
• Unreleased COM objects: You're not calling Release() on your COM pointers, which leads to memory leaks (even if the code works, this is bad practice).
• No null pointer safeguards: If enumerator is null, calling enumerator->Next() will immediately crash your program.

Fixed Code with Explanations #

#include "stdafx.h"
#include <iostream>
#include <wbemidl.h>
#include <comdef.h>

#pragma comment(lib, "wbemuuid.lib")

using namespace std;

int main() {
    HRESULT hr = CoInitializeEx(NULL, COINIT_MULTITHREADED);
    if (FAILED(hr)) {
        cerr << "CoInitializeEx failed. HRESULT: 0x" << hex << hr << endl;
        return 1;
    }

    IWbemLocator* locator = nullptr;
    hr = CoCreateInstance(CLSID_WbemLocator, NULL, CLSCTX_INPROC_SERVER, 
                          IID_IWbemLocator, reinterpret_cast<void**>(&locator));
    if (FAILED(hr) || !locator) {
        cerr << "CoCreateInstance for IWbemLocator failed. HRESULT: 0x" << hex << hr << endl;
        CoUninitialize();
        return 1;
    }

    IWbemServices* service = nullptr;
    hr = locator->ConnectServer(
        _bstr_t(L"root\\cimv2"),  // Target WMI namespace
        NULL, NULL, NULL,         // Use current user credentials
        WBEM_FLAG_CONNECT_USE_MAX_WAIT,
        NULL, NULL, &service
    );
    if (FAILED(hr) || !service) {
        cerr << "ConnectServer failed. HRESULT: 0x" << hex << hr << endl;
        locator->Release();
        CoUninitialize();
        return 1;
    }

    // Critical step: Set security context for WMI access
    hr = CoSetProxyBlanket(
        service,
        RPC_C_AUTHN_WINNT,
        RPC_C_AUTHZ_NONE,
        NULL,
        RPC_C_AUTHN_LEVEL_CALL,
        RPC_C_IMP_LEVEL_IMPERSONATE,
        NULL,
        EOAC_NONE
    );
    if (FAILED(hr)) {
        cerr << "CoSetProxyBlanket failed. HRESULT: 0x" << hex << hr << endl;
        service->Release();
        locator->Release();
        CoUninitialize();
        return 1;
    }

    IEnumWbemClassObject* enumerator = nullptr;
    hr = service->ExecQuery(
        _bstr_t(L"WQL"),
        _bstr_t(L"SELECT CreationClassName FROM Win32_PhysicalMemoryArray"),  // Only fetch needed property
        WBEM_FLAG_FORWARD_ONLY | WBEM_FLAG_RETURN_IMMEDIATELY,
        NULL,
        &enumerator
    );
    if (FAILED(hr) || !enumerator) {
        cerr << "ExecQuery failed. HRESULT: 0x" << hex << hr << endl;
        service->Release();
        locator->Release();
        CoUninitialize();
        return 1;
    }

    ULONG retcnt = 0;
    IWbemClassObject* memdevice = nullptr;
    hr = enumerator->Next(WBEM_INFINITE, 1, &memdevice, &retcnt);
    if (SUCCEEDED(hr) && retcnt > 0 && memdevice) {
        variant_t var_val;
        hr = memdevice->Get(L"CreationClassName", 0, &var_val, NULL, NULL);
        if (SUCCEEDED(hr)) {
            wcout << L"CreationClassName: " << var_val.bstrVal << endl;
            VariantClear(&var_val);  // Clean up the variant to avoid leaks
        } else {
            cerr << "Failed to get CreationClassName. HRESULT: 0x" << hex << hr << endl;
        }
        memdevice->Release();
    } else {
        cerr << "No results returned, or Next() failed. HRESULT: 0x" << hex << hr << endl;
    }

    // Clean up all COM objects properly
    enumerator->Release();
    service->Release();
    locator->Release();
    CoUninitialize();

    return 0;
}

Why This Fixes Your Problems #

1. HRESULT Validation: Every COM call is now checked for success. If any step fails, we print an error code and clean up resources instead of crashing.
2. CoSetProxyBlanket: This sets the security context for the WMI service proxy, which is often required to access system-level classes like Win32_PhysicalMemoryArray—without this, your query might silently fail.
3. Targeted Query: I modified the WQL to only select CreationClassName instead of *—this is more efficient and avoids loading unnecessary properties.
4. Proper Cleanup: All COM pointers are released with Release(), and the variant_t is cleared to prevent memory leaks.
5. Null Pointer Checks: We explicitly verify that enumerator, memdevice, etc., are non-null before using them.

Notes on CComPtr Assertion Failures #

If you want to use CComPtr later to simplify COM pointer management, the assertion failure in atlcomcli.h is likely caused by:

• Your project not having ATL enabled (go to Project Properties > Configuration Properties > General > Set "Use of ATL" to "Static Link to ATL" or "Dynamic Link to ATL").
• Mismatched compiler settings (e.g., using debug ATL libraries in a release build, or vice versa).
• Missing ATL includes—ensure #include <atlbase.h> is present before atlcomcli.h.

内容的提问来源于stack exchange，提问作者user2872935