Re: kernel32: Add basic OpenProcess test, make it pass under Wine

[Robert Shearman]

Dmitry Timoshkov wrote:
> Hello,
>
> Changelog:
>     kernel32: Add basic OpenProcess test, make it pass under Wine.
>
> @@ -1960,7 +1961,7 @@ NTSTATUS WINAPI NtFlushVirtualMemory( HA
>      NTSTATUS status = STATUS_SUCCESS;
>      void *addr = ROUND_ADDR( *addr_ptr, page_mask );
>
> -    if (!is_current_process( process ))
> +    if (!is_current_process( process, PROCESS_VM_OPERATION ))
>      {
>          ERR("Unsupported on other process\n");
>          return STATUS_ACCESS_DENIED;

Does this fix a real application or is it something that could just as 
easily be deferred until these remote operations are supported?

-- 
Rob Shearman

kernel32: Add basic OpenProcess test, make it pass under Wine

[Dmitry Timoshkov]

"Robert Shearman" <rob(a)codeweavers.com> wrote:
> Does this fix a real application or is it something that could just as 
> easily be deferred until these remote operations are supported?

No, this patch is not supposed to fix any real app, that's just for
a test case I've written (was curious how it works in Wine after reading
microsoft.public.win32.programmer.kernel). And no, it has nothing to do
with remote operations, it's purely for the current process.

-- 
Dmitry.