fix a potential nullptr deref bug in MeteredExecutor

Summary: ^ Reviewed By: yfeldblum Differential Revision: D32895223 fbshipit-source-id: ee4f53772410dbed4440cf4cd82df02f6629d24b
lmmMT · Dec 7, 2021 · 38dccb1 · 38dccb1
1 parent 08d9836
commit 38dccb1
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/folly/executors/MeteredExecutor.cpp b/folly/executors/MeteredExecutor.cpp
@@ -96,7 +96,7 @@ void MeteredExecutor::Consumer::executeIfNotEmpty() {
 void MeteredExecutor::Consumer::operator()(
     Task&& task, std::shared_ptr<RequestContext>&& rctx) {
   if (self_.queueObserver_) {
-    self_.queueObserver_->onDequeued(first_->getQueueObserverPayload());
+    self_.queueObserver_->onDequeued(task.getQueueObserverPayload());
   }
   if (!first_) {
     first_ = std::make_optional<Task>(std::move(task));