From 5ad936b13f317cb277bf03c19e9734d931b1bd00 Mon Sep 17 00:00:00 2001 From: danieldg Date: Tue, 1 Sep 2009 15:08:00 +0000 Subject: Remove unneeded paramaters from OnAcceptReady git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@11581 e03df62e-2008-0410-955e-edbf42e46eb7 --- src/listensocket.cpp | 10 ++-------- src/modules/m_httpd.cpp | 9 ++++++--- src/modules/m_spanningtree/utils.cpp | 7 +++++-- src/modules/m_spanningtree/utils.h | 2 +- src/threadengines/threadengine_win32.cpp | 7 +++---- 5 files changed, 17 insertions(+), 18 deletions(-) (limited to 'src') diff --git a/src/listensocket.cpp b/src/listensocket.cpp index f40afd123..ed99c2787 100644 --- a/src/listensocket.cpp +++ b/src/listensocket.cpp @@ -125,15 +125,9 @@ void ListenSocketBase::AcceptInternal() } } - std::string server_addr; - std::string client_addr; - int dummy_port; - irc::sockets::satoap(&server, server_addr, dummy_port); - irc::sockets::satoap(&client, client_addr, dummy_port); - ServerInstance->SE->NonBlocking(incomingSockfd); ServerInstance->stats->statsAccept++; - this->OnAcceptReady(server_addr, incomingSockfd, client_addr); + this->OnAcceptReady(incomingSockfd); } void ListenSocketBase::HandleEvent(EventType e, int err) @@ -152,7 +146,7 @@ void ListenSocketBase::HandleEvent(EventType e, int err) } } -void ClientListenSocket::OnAcceptReady(const std::string &ipconnectedto, int nfd, const std::string &incomingip) +void ClientListenSocket::OnAcceptReady(int nfd) { ServerInstance->Users->AddUser(ServerInstance, nfd, false, &client, &server); } diff --git a/src/modules/m_httpd.cpp b/src/modules/m_httpd.cpp index c40f6055a..bdf4e424e 100644 --- a/src/modules/m_httpd.cpp +++ b/src/modules/m_httpd.cpp @@ -48,7 +48,7 @@ class HttpServerSocket : public BufferedSocket public: - HttpServerSocket(InspIRCd* SI, int newfd, char* ip, FileReader* ind) : BufferedSocket(SI, newfd, ip), index(ind), postsize(0) + HttpServerSocket(InspIRCd* SI, int newfd, const char* ip, FileReader* ind) : BufferedSocket(SI, newfd, ip), index(ind), postsize(0) { InternalState = HTTP_SERVE_WAIT_REQUEST; } @@ -353,9 +353,12 @@ class HttpListener : public ListenSocketBase this->index = idx; } - virtual void OnAcceptReady(const std::string &ipconnectedto, int nfd, const std::string &incomingip) + virtual void OnAcceptReady(int nfd) { - new HttpServerSocket(ServerInstance, nfd, (char *)incomingip.c_str(), index); // ugly cast courtesy of bufferedsocket + int port; + std::string incomingip; + irc::sockets::satoap(&client, incomingip, port); + new HttpServerSocket(ServerInstance, nfd, incomingip.c_str(), index); } }; diff --git a/src/modules/m_spanningtree/utils.cpp b/src/modules/m_spanningtree/utils.cpp index 8a17baf84..ee12712b8 100644 --- a/src/modules/m_spanningtree/utils.cpp +++ b/src/modules/m_spanningtree/utils.cpp @@ -29,10 +29,13 @@ /* $ModDep: m_spanningtree/resolvers.h m_spanningtree/main.h m_spanningtree/utils.h m_spanningtree/treeserver.h m_spanningtree/link.h m_spanningtree/treesocket.h */ /* Create server sockets off a listener. */ -void ServerSocketListener::OnAcceptReady(const std::string &ipconnectedto, int newsock, const std::string &incomingip) +void ServerSocketListener::OnAcceptReady(int newsock) { bool found = false; - char *ip = (char *)incomingip.c_str(); // XXX ugly cast + int port; + std::string incomingip; + irc::sockets::satoap(&client, incomingip, port); + char *ip = const_cast(incomingip.c_str()); found = (std::find(Utils->ValidIPs.begin(), Utils->ValidIPs.end(), ip) != Utils->ValidIPs.end()); if (!found) diff --git a/src/modules/m_spanningtree/utils.h b/src/modules/m_spanningtree/utils.h index a32b5a445..81c47267d 100644 --- a/src/modules/m_spanningtree/utils.h +++ b/src/modules/m_spanningtree/utils.h @@ -49,7 +49,7 @@ class ServerSocketListener : public ListenSocketBase this->Utils = u; } - virtual void OnAcceptReady(const std::string &ipconnectedto, int nfd, const std::string &incomingip); + virtual void OnAcceptReady(int nfd); }; typedef std::map TreeServerList; diff --git a/src/threadengines/threadengine_win32.cpp b/src/threadengines/threadengine_win32.cpp index eb710bbde..361d25cc4 100644 --- a/src/threadengines/threadengine_win32.cpp +++ b/src/threadengines/threadengine_win32.cpp @@ -55,7 +55,7 @@ class ThreadSignalSocket : public BufferedSocket { SocketThread* parent; public: - ThreadSignalSocket(SocketThread* t, InspIRCd* SI, int newfd, char* ip) + ThreadSignalSocket(SocketThread* t, InspIRCd* SI, int newfd, const char* ip) : BufferedSocket(SI, newfd, ip), parent(t) { } @@ -86,11 +86,10 @@ class ThreadSignalListener : public ListenSocketBase } } - virtual void OnAcceptReady(const std::string &ipconnectedto, int nfd, const std::string &incomingip) + virtual void OnAcceptReady(int nfd) { - new ThreadSignalSocket(parent, ServerInstance, nfd, const_cast(ipconnectedto.c_str())); + new ThreadSignalSocket(parent, ServerInstance, nfd, ""); ServerInstance->SE->DelFd(this); - // XXX unsafe casts suck } /* Using getsockname and ntohs, we can determine which port number we were allocated */ int GetPort() -- cgit v1.2.3