Fix user/kernel times of GetProcessTimes()
Mike McCormack
mike at codeweavers.com
Mon Oct 23 04:55:30 CDT 2006
Sidney Fong wrote:
> From 487243c1040f7ed17bd9241fc0f63f3f4774d76d Mon Sep 17 00:00:00 2001
> From: FONG Sidney Hok Nang <wine at sidney.hk>
> Date: Sun, 22 Oct 2006 15:36:38 +0800
> Subject: Fix user/kernel times of GetProcessTimes()
>
> Changes:
> Fix the known bug[1] of GetProcessTimes() where it originally returned
> the user/kernel times of the current process instead of the designated
> one
Hi,
Thanks for the patch!
Please attach it to your mail, or use git imap-send so it doesn't get
line wrapped by Thunderbird, otherwise it won't apply properly:
mike at black:~/wine$ patch -p1 --dry-run < Fix\ user_kernel\ times\ of\
GetProcessTimes\(\).eml
patching file dlls/kernel32/time.c
patch: **** malformed patch at line 96: @@ -533,27 +530,6 @@ VOID WINAPI
GetSystemTimeAsFileTime(
> dlls/kernel32/time.c | 30 ++--------------------------
> dlls/ntdll/process.c | 2 ++
> include/wine/server_protocol.h | 2 ++
> server/process.c | 29 +++++++++++++++++++++++++++
> server/process.h | 5 +++++
> server/ptrace.c | 43
> +++++++++++++++++++++++++++++++++++++---
> 6 files changed, 80 insertions(+), 31 deletions(-)
To change the server protocol, change server/protocol.def, then run
tools/make_requests. When you submit your patch, there's no need to
include changes to include/wine/server_protocol.h or server/trace.c, so
commit the changes to those files separately.
> + /*** The following chunk is taken from suspend_for_ptrace() ***/
For this code, rather than copying, it would be nicer to integrate your
code with the existing suspect_for_ptrace() function.
Mike
More information about the wine-devel
mailing list