highperfocused/bitcoin
1
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2025-04-08 03:59:18 +02:00
Code Issues Packages Projects Releases Wiki Activity

http: Release work queue after event base finish

Browse Source 
This fixes a race between http_request_cb and StopHTTPServer where
the work queue is used after release.
This commit is contained in:
João Barbosa 2020-05-21 18:12:49 +01:00
parent 2aaff4813c
commit 4e353cb618
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/httpserver.cpp
View File

@ -91,7 +91,7 @@ public:
bool Enqueue(WorkItem* item)
{
LOCK(cs);
if (queue.size() >= maxDepth) {
if (!running || queue.size() >= maxDepth) {
return false;
}
queue.emplace_back(std::unique_ptr<WorkItem>(item));
@ -107,7 +107,7 @@ public:
WAIT_LOCK(cs, lock);
while (running && queue.empty())
cond.wait(lock);
if (!running)
if (!running && queue.empty())
break;
i = std::move(queue.front());
queue.pop_front();
@ -456,8 +456,6 @@ void StopHTTPServer()
thread.join();
}
g_thread_http_workers.clear();
delete workQueue;
workQueue = nullptr;
}
// Unlisten sockets, these are what make the event loop running, which means
// that after this and all connections are closed the event loop will quit.
@ -477,6 +475,10 @@ void StopHTTPServer()
event_base_free(eventBase);
eventBase = nullptr;
}
if (workQueue) {
delete workQueue;
workQueue = nullptr;
}
LogPrint(BCLog::HTTP, "Stopped HTTP server\n");
}