Current-Users archive

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

Re: ntpq -p coredumps with today's sources



This helps in the way - one down more to go.
Seems that there are some more issues with getaddrinfo.c.

I'll commit the SIGSEGV fix, but the name resolution issue still needs to be resolved (christos?)

Frank

On 03/20/12 10:47, Ryo ONODERA wrote:
From: Frank Kardel<kardel%netbsd.org@localhost>, Date: Tue, 20 Mar 2012 
10:13:27 +0100

On 03/20/12 09:59, Martin Husemann wrote:
On Tue, Mar 20, 2012 at 08:39:43AM +0100, Frank Kardel wrote:
Could you check this patch.

ntpq.c exercises the new AI_ADDRCONFIG option in getaddrinfo(). There
seems to
be a PNPE (programmed null pointer exception).
See also PR pkg/46206 (likely the same problem)

Martin
Looks like it from the PR. I can't test right now but the patch should
be very much
the correct fix as the current code is definitely a *0 when
AI_ADDRCONFIG is set.

Can anyone test ?
I have reproduced the problem of firefox.

With your patch, firefox runs (without your patch core dumps).
But firefox cannot resolve hostname.
For example, on firefox, http://www.yahoo.co.jp/ is "Server not found".
http://203.216.227.245/ is displayed properly.

This will help?




Home | Main Index | Thread Index | Old Index