From 81de672a6742268a42c62628cfddd058616fd7e4 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Mon, 28 Nov 2016 13:45:35 -0500 Subject: [PATCH] os/bluestore/BlueFS: check flush result CID 1394759 (#1 of 1): Unchecked return value (CHECKED_RETURN) 20. check_return: Calling _flush without checking return value (as is done elsewhere 4 out of 5 times). Signed-off-by: Sage Weil --- src/os/bluestore/BlueFS.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/os/bluestore/BlueFS.cc b/src/os/bluestore/BlueFS.cc index 9d32697343f..8df7e712903 100644 --- a/src/os/bluestore/BlueFS.cc +++ b/src/os/bluestore/BlueFS.cc @@ -1178,7 +1178,8 @@ void BlueFS::_compact_log_async(std::unique_lock& l) new_log_writer->append(bl); // 3. flush - _flush(new_log_writer, true); + r = _flush(new_log_writer, true); + assert(r == 0); lock.unlock(); // 4. wait -- 2.47.3