summaryrefslogtreecommitdiffstats
path: root/source/HTTPServer
diff options
context:
space:
mode:
authormadmaxoft <github@xoft.cz>2013-10-06 16:40:28 +0200
committermadmaxoft <github@xoft.cz>2013-10-06 16:40:28 +0200
commitf55b77a98a41ba784109842cde39ba0e1d2bc262 (patch)
tree89c1b144d1a224c24bc7ae79e9b0304766e01a8f /source/HTTPServer
parentcListenThread: Fixed thread termination. (diff)
downloadcuberite-f55b77a98a41ba784109842cde39ba0e1d2bc262.tar
cuberite-f55b77a98a41ba784109842cde39ba0e1d2bc262.tar.gz
cuberite-f55b77a98a41ba784109842cde39ba0e1d2bc262.tar.bz2
cuberite-f55b77a98a41ba784109842cde39ba0e1d2bc262.tar.lz
cuberite-f55b77a98a41ba784109842cde39ba0e1d2bc262.tar.xz
cuberite-f55b77a98a41ba784109842cde39ba0e1d2bc262.tar.zst
cuberite-f55b77a98a41ba784109842cde39ba0e1d2bc262.zip
Diffstat (limited to '')
-rw-r--r--source/HTTPServer/HTTPConnection.cpp25
-rw-r--r--source/HTTPServer/HTTPConnection.h4
-rw-r--r--source/HTTPServer/HTTPServer.cpp5
3 files changed, 32 insertions, 2 deletions
diff --git a/source/HTTPServer/HTTPConnection.cpp b/source/HTTPServer/HTTPConnection.cpp
index 2addf4cfc..68afdfc11 100644
--- a/source/HTTPServer/HTTPConnection.cpp
+++ b/source/HTTPServer/HTTPConnection.cpp
@@ -17,11 +17,22 @@ cHTTPConnection::cHTTPConnection(cHTTPServer & a_HTTPServer) :
m_State(wcsRecvHeaders),
m_CurrentRequest(NULL)
{
+ // LOGD("HTTP: New connection at %p", this);
}
+
+cHTTPConnection::~cHTTPConnection()
+{
+ // LOGD("HTTP: Del connection at %p", this);
+}
+
+
+
+
+
void cHTTPConnection::SendStatusAndReason(int a_StatusCode, const AString & a_Response)
{
AppendPrintf(m_OutgoingData, "%d %s\r\nContent-Length: 0\r\n\r\n", a_StatusCode, a_Response.c_str());
@@ -120,6 +131,19 @@ void cHTTPConnection::AwaitNextRequest(void)
+void cHTTPConnection::Terminate(void)
+{
+ if (m_CurrentRequest != NULL)
+ {
+ m_HTTPServer.RequestFinished(*this, *m_CurrentRequest);
+ }
+ m_HTTPServer.CloseConnection(*this);
+}
+
+
+
+
+
void cHTTPConnection::DataReceived(const char * a_Data, int a_Size)
{
switch (m_State)
@@ -214,6 +238,7 @@ void cHTTPConnection::SocketClosed(void)
{
m_HTTPServer.RequestFinished(*this, *m_CurrentRequest);
}
+ m_HTTPServer.CloseConnection(*this);
}
diff --git a/source/HTTPServer/HTTPConnection.h b/source/HTTPServer/HTTPConnection.h
index 941a29000..14603bb70 100644
--- a/source/HTTPServer/HTTPConnection.h
+++ b/source/HTTPServer/HTTPConnection.h
@@ -39,6 +39,7 @@ public:
} ;
cHTTPConnection(cHTTPServer & a_HTTPServer);
+ ~cHTTPConnection();
/// Sends HTTP status code together with a_Reason (used for HTTP errors)
void SendStatusAndReason(int a_StatusCode, const AString & a_Reason);
@@ -61,6 +62,9 @@ public:
/// Resets the connection for a new request. Depending on the state, this will send an "InternalServerError" status or a "ResponseEnd"
void AwaitNextRequest(void);
+ /// Terminates the connection; finishes any request being currently processed
+ void Terminate(void);
+
protected:
typedef std::map<AString, AString> cNameValueMap;
diff --git a/source/HTTPServer/HTTPServer.cpp b/source/HTTPServer/HTTPServer.cpp
index 9636eb59f..f6f5b0f8b 100644
--- a/source/HTTPServer/HTTPServer.cpp
+++ b/source/HTTPServer/HTTPServer.cpp
@@ -179,9 +179,9 @@ void cHTTPServer::Stop(void)
// Drop all current connections:
cCSLock Lock(m_CSConnections);
- for (cHTTPConnections::iterator itr = m_Connections.begin(), end = m_Connections.end(); itr != end; ++itr)
+ while (!m_Connections.empty())
{
- m_SocketThreads.RemoveClient(*itr);
+ m_Connections.front()->Terminate();
} // for itr - m_Connections[]
}
@@ -213,6 +213,7 @@ void cHTTPServer::CloseConnection(cHTTPConnection & a_Connection)
break;
}
}
+ delete &a_Connection;
}