Add missing lock in CScheduler::AreThreadsServicingQueue()

TheBlueMatt · TheBlueMatt · commit a56f8b0be370 · 2017-07-23T14:51:22.000-04:00
Not an actual bug as this is only used in asserts right now, but
nice to not have a missing lock.
diff --git a/src/scheduler.cpp b/src/scheduler.cpp
@@ -141,6 +141,7 @@ size_t CScheduler::getQueueInfo(boost::chrono::system_clock::time_point &first,
 }
 
 bool CScheduler::AreThreadsServicingQueue() const {
+    boost::unique_lock<boost::mutex> lock(newTaskMutex);
     return nThreadsServicingQueue;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -141,6 +141,7 @@ size_t CScheduler::getQueueInfo(boost::chrono::system_clock::time_point &first,`
`141`	`141`	`}`
`142`	`142`
`143`	`143`	`bool CScheduler::AreThreadsServicingQueue() const {`
	`144`	`+ boost::unique_lock<boost::mutex> lock(newTaskMutex);`
`144`	`145`	`return nThreadsServicingQueue;`
`145`	`146`	`}`
`146`	`147`