rados_bench: wait for completion callbacks before returning

If we don't wait for the callback, the finisher may cleanup the callback
context before the callback is actually invoked, causing a
use-after-free error.

This fixes #3048.

Signed-off-by: Mike Ryan <mike.ryan@inktank.com>

diff --git a/src/rados.cc b/src/rados.cc
index 8f8489a526686e5c00c8aa7d4e20c51adee9e35b..50131e32f94c9fff0916bfba06d9e958d6d9b6ce 100644 (file)
--- a/src/rados.cc
+++ b/src/rados.cc
@@ -807,7 +807,7 @@ protected:
   }
 
   int completion_wait(int slot) {
-    return completions[slot]->wait_for_safe();
+    return completions[slot]->wait_for_safe_and_cb();
   }
   int completion_ret(int slot) {
     return completions[slot]->get_return_value();