[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#495007: libc6: getaddrinfo_a causes SetFault in amd64 version



Hi Aurelien,

The linuxthreads was a red herring - it's with NPTL. I don't think that the Debian package patches the section of code with problems, I think it's in the upstream source. The code sample we gave definitely crashes on a vanilla lenny install so we thought you'd be interested in the patch (and it seemed like the fix would trickle through faster if we reported it to you).

Many thanks, Jon.


On Aug 13, 2008, at 7:10 PM, Aurelien Jarno wrote:

On Wed, Aug 13, 2008 at 08:23:31PM -0400, Matthew Von-Maszewski wrote:
Aurelien,

I doubt "linuxthreads" is anything but an old directory being reused.
The file gai_misc.h appears in two places. The one I mention below in
the debian and another in the ./resolve directory.  The debian tree
version appears to be someone's attempt at optimizing memory usage on
the secondary thread.  The debian version overrides the libc/resolve
version. The secondary thread is created by a conventional pthreads
call (again unrelated to "linuxthreads").

I don't understand why you are speaking about a debian version, debian
does not patch this file, so there is nothing about optimizing memory
usage.

The reduced stack size is now being pounded by what appears to be a
ton of relatively new getaddrinfo code.  __res_vinit() has a large
buffer it takes out of the stack.  It makes sense that the stack is
now to small.

I ask that you give the bug serious consideration as written.  I

I haven't found yet a time to get a closer look, but your analysis seems
to be wrong given that:
- we are not using the linuxthreads anymore
- we do not change gai_misc.h

I'll have a closer look tomorrow.

--
 .''`.  Aurelien Jarno	            | GPG: 1024D/F1BCDB73
: :' :  Debian developer           | Electrical Engineer
`. `'   aurel32@debian.org         | aurelien@aurel32.net
  `-    people.debian.org/~aurel32 | www.aurel32.net





Reply to: