This is the mail archive of the libc-hacker@sourceware.cygnus.com mailing list for the glibc project.

Note that libc-hacker is a closed list. You may look at the archives of this list, but subscription and posting are not open.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Patch for linuxthreads_db/td_thr_getgregs.c


Ulrich Drepper <drepper@cygnus.com> writes:

|> Andreas Jaeger <aj@suse.de> writes:
|> 
|> > Uli, if those function declarations are indeed correct (which I
|> > doubt), you need to change td_thr_getgregs.c accordingly (memset
|> > (&greg,...)).
|> 
|> Well, the form I have in the moment is acutally what Solaris has.  It
|> was strange when I saw it but I haven't really spend time looking at
|> it.  But you are right, it cannot be correct.

The problem is that gregset_t is sometimes an array (i386), sometimes a
struct (mips).  The array decays to a pointer, so you won't see the
different types, but the sizeof in td_thr_getregs will return the wrong
value (size of a pointer instead of the whole array).

Andreas.

-- 
Andreas Schwab                                  "And now for something
SuSE Labs                                        completely different."
schwab@suse.de
SuSE GmbH, Schanzäckerstr. 10, D-90443 Nürnberg

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]