From c0f82142185d866011b78f67f488089875ed99f9 Mon Sep 17 00:00:00 2001 From: rubidium Date: Wed, 8 Apr 2009 00:03:05 +0000 Subject: (svn r15972) -Codechange: remove unneeded parameter --- src/network/core/address.cpp | 14 +++++++++----- src/network/core/address.h | 3 +-- src/network/core/udp.cpp | 6 +----- src/network/network.cpp | 2 +- 4 files changed, 12 insertions(+), 13 deletions(-) diff --git a/src/network/core/address.cpp b/src/network/core/address.cpp index 5d889c0ad..14b52faa8 100644 --- a/src/network/core/address.cpp +++ b/src/network/core/address.cpp @@ -162,13 +162,13 @@ SOCKET NetworkAddress::Resolve(int family, int socktype, int flags, SocketList * /* Setting both hostname to NULL and port to 0 is not allowed. * As port 0 means bind to any port, the other must mean that * we want to bind to 'all' IPs. */ - if (this->address_length == 0 && StrEmpty(this->hostname)) { + if (this->address_length == 0 && this->GetPort() == 0) { strecpy(this->hostname, this->address.ss_family == AF_INET ? "0.0.0.0" : "::", lastof(this->hostname)); } - int e = getaddrinfo(this->GetHostname(), port_name, &hints, &ai); + int e = getaddrinfo(StrEmpty(this->hostname) ? NULL : this->hostname, port_name, &hints, &ai); if (e != 0) { - DEBUG(net, 0, "getaddrinfo(%s, %s) failed: %s", this->GetHostname(), port_name, FS2OTTD(gai_strerror(e))); + DEBUG(net, 0, "getaddrinfo(%s, %s) failed: %s", this->hostname, port_name, FS2OTTD(gai_strerror(e))); return INVALID_SOCKET; } @@ -270,12 +270,16 @@ static SOCKET ListenLoopProc(addrinfo *runp) /* Connection succeeded */ if (!SetNonBlocking(sock)) DEBUG(net, 0, "Setting non-blocking mode failed"); + DEBUG(net, 1, "[%s] Listening on port %s", + runp->ai_socktype == SOCK_STREAM ? "tcp" : "udp", + NetworkAddress(runp->ai_addr, runp->ai_addrlen).GetAddressAsString()); + return sock; } -SOCKET NetworkAddress::Listen(int family, int socktype, SocketList *sockets) +SOCKET NetworkAddress::Listen(int socktype, SocketList *sockets) { - return this->Resolve(family, socktype, AI_ADDRCONFIG | AI_PASSIVE, sockets, ListenLoopProc); + return this->Resolve(AF_UNSPEC, socktype, AI_ADDRCONFIG | AI_PASSIVE, sockets, ListenLoopProc); } #endif /* ENABLE_NETWORK */ diff --git a/src/network/core/address.h b/src/network/core/address.h index 4d263b3c0..4c0abdf63 100644 --- a/src/network/core/address.h +++ b/src/network/core/address.h @@ -217,12 +217,11 @@ public: /** * Make the given socket listen. - * @param family the type of 'protocol' (IPv4, IPv6) * @param socktype the type of socket (TCP, UDP, etc) * @param sockets the list of sockets to add the sockets to * @return the socket (if sockets != NULL) */ - SOCKET Listen(int family, int socktype, SocketList *sockets = NULL); + SOCKET Listen(int socktype, SocketList *sockets = NULL); }; #endif /* ENABLE_NETWORK */ diff --git a/src/network/core/udp.cpp b/src/network/core/udp.cpp index 96ece0ace..4a5570002 100644 --- a/src/network/core/udp.cpp +++ b/src/network/core/udp.cpp @@ -41,11 +41,7 @@ bool NetworkUDPSocketHandler::Listen() this->Close(); for (NetworkAddress *addr = this->bind.Begin(); addr != this->bind.End(); addr++) { - addr->Listen(AF_UNSPEC, SOCK_DGRAM, &this->sockets); - } - - for (SocketList::iterator s = this->sockets.Begin(); s != this->sockets.End(); s++) { - DEBUG(net, 1, "[udp] listening on port %s", s->first.GetAddressAsString()); + addr->Listen(SOCK_DGRAM, &this->sockets); } return this->sockets.Length() != 0; diff --git a/src/network/network.cpp b/src/network/network.cpp index e1d3ad2a8..a603dd3f9 100644 --- a/src/network/network.cpp +++ b/src/network/network.cpp @@ -539,7 +539,7 @@ static bool NetworkListen() DEBUG(net, 1, "Listening on %s", address.GetAddressAsString()); - SOCKET ls = address.Listen(AF_INET, SOCK_STREAM); + SOCKET ls = address.Listen(SOCK_STREAM); if (ls == INVALID_SOCKET) { ServerStartError("Could not create listening socket"); return false; -- cgit v1.2.3-70-g09d2