From e9cbed2c4d2a885e851350438609843873839f78 Mon Sep 17 00:00:00 2001
From: Yingxin Cheng <yingxin.cheng@intel.com>
Date: Mon, 15 Aug 2022 11:31:41 +0800
Subject: [PATCH] crimson/os/seastore/transaction_manager: don't return invalid
 extent in get_extents_if_live()

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
---
 src/crimson/os/seastore/transaction_manager.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/crimson/os/seastore/transaction_manager.cc b/src/crimson/os/seastore/transaction_manager.cc
index f7a560829b8..5f8fce47c7c 100644
--- a/src/crimson/os/seastore/transaction_manager.cc
+++ b/src/crimson/os/seastore/transaction_manager.cc
@@ -598,15 +598,15 @@ TransactionManager::get_extents_if_live(
 	laddr,
 	len
       ).si_then([=, &t](auto ret) {
+        std::list<CachedExtentRef> res;
         if (ret) {
           DEBUGT("{} {}~{} {} is live as physical extent -- {}",
                  t, type, laddr, len, paddr, *ret);
+          res.emplace_back(std::move(ret));
         } else {
           DEBUGT("{} {}~{} {} is not live as physical extent",
                  t, type, laddr, len, paddr);
         }
-	std::list<CachedExtentRef> res;
-	res.emplace_back(std::move(ret));
         return get_extents_if_live_ret(
 	  interruptible::ready_future_marker{},
 	  std::move(res));
-- 
2.39.5