Re: [PATCH v2 0/2] MR6734: wbemprox: Add property "ExecutablePath" to Win32_Process

[Hans Leidekker (＠hans)]

Hans Leidekker (@hans) commented about dlls/wbemprox/builtin.c:
>          goto done;
>      }
>
> -
>    process = OpenProcess(PROCESS_QUERY_INFORMATION | PROCESS_VM_READ, FALSE, entry.th32ProcessID);
> -
>    if (!process)
> -
>    {
> -
>        ERR("Failed to open process\n");
> -
>        status = FILL_STATUS_FAILED;
> -
>        goto done;
> -
>    }
> -
>    size = ARRAY_SIZE(executable_path);
> -
>    QueryFullProcessImageNameW(process, 0, executable_path, &size);

Please add a get_executablepath() helper and allocate the path dynamically.