On Fri, Jun 10, 2011 at 01:06:15PM -0700, Andi Kleen wrote: > From: Andi Kleen <[email protected]> > > The kernel returns a integer, not a pointer for the timer id > in the memory pointed to by timer_id. > strace used a pointer, which added random junk on a 64bit system. > Fix it to be an integer. > --- > time.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/time.c b/time.c > index 0c6deaf..ac366fb 100644 > --- a/time.c > +++ b/time.c > @@ -838,12 +838,12 @@ struct tcb *tcp; > printsigevent(tcp, tcp->u_arg[1]); > tprintf(", "); > } else { > - void *p; > + int p; > > if (syserror(tcp) || umove(tcp, tcp->u_arg[2], &p) < 0) > tprintf("%#lx", tcp->u_arg[2]); > else > - tprintf("{%p}", p); > + tprintf("{%u}", p); > } > return 0; > }
Thanks, changed "int p" to less confusing "int timer_id" and applied. -- ldv
pgpEP9SQLvTnr.pgp
Description: PGP signature
------------------------------------------------------------------------------ EditLive Enterprise is the world's most technically advanced content authoring tool. Experience the power of Track Changes, Inline Image Editing and ensure content is compliant with Accessibility Checking. http://p.sf.net/sfu/ephox-dev2dev
_______________________________________________ Strace-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/strace-devel
