]> git.ipfire.org Git - thirdparty/squid.git/commit
Fix "address.GetPort() != 0" assertion for helpers on FreeBSD (at least).
authorAlex Rousskov <rousskov@measurement-factory.com>
Tue, 22 Jan 2013 11:06:18 +0000 (04:06 -0700)
committerAmos Jeffries <squid3@treenet.co.nz>
Tue, 22 Jan 2013 11:06:18 +0000 (04:06 -0700)
commit8534f05d464cddd265068a333a2bc5591a70156f
tree31ae0a0058cc9cf2b43bae2cfe246ccf07b40ed1
parente1a52f52a4a29cf27434a40d83734ceaec3ebecf
Fix "address.GetPort() != 0" assertion for helpers on FreeBSD (at least).

The order (or set of?) #include files used by src/ssl/helper.cc (and
probably by other helper source files) has changed recently, exposing a
defines.h dependency on sys/socket.h where that system header is
required to define AF_UNIX. With AF_UNIX incorrectly undefined,
IPC_STREAM was set to IPC_TCP_SOCKET instead of IPC_UNIX_STREAM, and
helpers that do not have a notion of a listening port, were trying to
create communication sockets using TCP streams, triggering a "must have
a port" assertion in comm_connect_addr() called from ipcCreate().

TODO: Moving IPC_* defines into an IPC-specific header file may be a
better solution then exposing all defines.h users to sys/socket.h.
src/defines.h