[Home]

Summary:ASTERISK-10024: wrong configure.ac check for gethostbyname_r on some platforms
Reporter:Luigi Rizzo (rizzo)Labels:
Date Opened:2007-08-05 11:06:47Date Closed:2007-08-06 09:02:32
Priority:MinorRegression?No
Status:Closed/CompleteComponents:General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:A recent addition to configure.ac http://svn.digium.com/view/asterisk?view=rev&revision=77864
to check for gethostbyname_r fails on some platforms (e.g
FreeBSD) as the test uses NULL without including some required header.
This affects both trunk and 1.4

The following change (in two places in configure.ac) should fix the problem,
i am not committing it because cannot test it on other platforms.
I am not sure what is the proper header to include to make sure NULL
is defined, though.

-        AC_LANG_PROGRAM([#include <netdb.h>],
+        AC_LANG_PROGRAM([#include <stdlib.h>
+                       #include <netdb.h>],
Comments:By: Digium Subversion (svnbot) 2007-08-06 09:00:47

Repository: asterisk
Revision: 78166

------------------------------------------------------------------------
r78166 | file | 2007-08-06 09:00:47 -0500 (Mon, 06 Aug 2007) | 4 lines

(closes issue ASTERISK-10024)
Reported by: rizzo
Include stdlib.h so NULL gets defined for gethostbyname_r checks.

------------------------------------------------------------------------
By: Digium Subversion (svnbot) 2007-08-06 09:02:32

Repository: asterisk
Revision: 78167

------------------------------------------------------------------------
r78167 | file | 2007-08-06 09:02:31 -0500 (Mon, 06 Aug 2007) | 12 lines

Merged revisions 78166 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r78166 | file | 2007-08-06 11:18:20 -0300 (Mon, 06 Aug 2007) | 4 lines

(closes issue ASTERISK-10024)
Reported by: rizzo
Include stdlib.h so NULL gets defined for gethostbyname_r checks.

........

------------------------------------------------------------------------