Re: Continuum patch

[Dan Kegel]

On 7/20/06, [e-mail]  wrote:
> There is a patch in the AppDB entry for the game Continuum. (sorry I
> don't have AppDB playing nice with DNS here yet so no link.) What needs
> to be done to have this considered?

Here's the patch:

diff --git a/dlls/kernel/process.c b/dlls/kernel/process.c
index 33f9ee1..d50cb7d 100644
--- a/dlls/kernel/process.c
+++ b/dlls/kernel/process.c
@@ -2460,6 +2464,7 @@ HANDLE WINAPI OpenProcess( DWORD access,
     OBJECT_ATTRIBUTES   attr;
     CLIENT_ID           cid;

+if (access & PROCESS_VM_WRITE) return NULL;
     cid.UniqueProcess = (HANDLE)id;
     cid.UniqueThread = 0; /* FIXME ? */

That patch as written doesn't seem quite right.
I think what needs to happen is somebody needs to
write a conformance test for OpenProcess.
That will help everyone figure out what the right patch is.
- Dan