ceph: -EINVAL on decoding failure in ceph_mdsc_handle_fsmap()
authorIlya Dryomov <idryomov@gmail.com>
Mon, 16 Oct 2017 08:32:50 +0000 (10:32 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Mon, 13 Nov 2017 11:11:39 +0000 (12:11 +0100)
Don't set ->mdsmap_err to -ENOENT unconditionally, and drop unneeded
return statement while at it.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
fs/ceph/mds_client.c

index 8c83612..b76506b 100644 (file)
@@ -3879,14 +3879,14 @@ void ceph_mdsc_handle_fsmap(struct ceph_mds_client *mdsc, struct ceph_msg *msg)
                goto err_out;
        }
        return;
+
 bad:
        pr_err("error decoding fsmap\n");
 err_out:
        mutex_lock(&mdsc->mutex);
-       mdsc->mdsmap_err = -ENOENT;
+       mdsc->mdsmap_err = err;
        __wake_requests(mdsc, &mdsc->waiting_for_map);
        mutex_unlock(&mdsc->mutex);
-       return;
 }
 
 /*