edg-l/ddnet
1
0
Fork 0
mirror of https://github.com/ddnet/ddnet.git synced 2024-09-20 01:24:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Cleanup HTTP request cleanup

Browse source 
Ensure with assertion that file is closed when `CHttpRequest` is destructed.

Remove unnecessary assignments in destructor.
This commit is contained in:
Robert Müller 2024-02-04 16:23:35 +01:00
parent c09f1e133f
commit 6c6f058a8c
1 changed files with 4 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/engine/shared/http.cpp
View file

@ -72,21 +72,10 @@ CHttpRequest::CHttpRequest(const char *pUrl)
CHttpRequest::~CHttpRequest()
{
m_ResponseLength = 0;
if(!m_WriteToFile)
{
m_BufferSize = 0;
free(m_pBuffer);
m_pBuffer = nullptr;
}
dbg_assert(m_File == nullptr, "HTTP request file was not closed");
free(m_pBuffer);
curl_slist_free_all((curl_slist *)m_pHeaders);
m_pHeaders = nullptr;
if(m_pBody)
{
m_BodyLength = 0;
free(m_pBody);
m_pBody = nullptr;
}
free(m_pBody);
}
bool CHttpRequest::BeforeInit()
@ -317,6 +306,7 @@ void CHttpRequest::OnCompletionInternal(std::optional<unsigned int> Result)
dbg_msg("http", "i/o error, cannot close file: %s", m_aDest);
State = EHttpState::ERROR;
}
m_File = nullptr;
if(State == EHttpState::ERROR || State == EHttpState::ABORTED)
{