librbd: make sure racing flattens don't crash

The only way for a parent to disappear is a racing flatten completing,
or possibly in the future the image being forcibly removed. In either
case, continuing to flatten makes no sense, so stop early.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

1 files changed, 6 insertions, 0 deletions

diff --git a/src/librbd/internal.cc b/src/librbd/internal.cc
index e1c65f0df40..d913b314edd 100644
--- a/src/librbd/internal.cc
+++ b/src/librbd/internal.cc
@@ -2017,6 +2017,12 @@ reprotect_and_return_err:
       assert(object_overlap <= object_size);
 
       RWLock::RLocker l(ictx->parent_lock);
+      // stop early if the parent went away - it just means
+      // another flatten finished first, so this one is useless.
+      if (!ictx->parent) {
+	r = 0;
+	goto err;
+      }
       if ((r = read(ictx->parent, objectx, buf, NULL)) < 0) {
 	lderr(ictx->cct) << "reading from parent failed" << dendl;
 	goto err;