ceph: -EINVAL on decoding failure in ceph_mdsc_handle_fsmap()
Don't set ->mdsmap_err to -ENOENT unconditionally, and drop unneeded return statement while at it. Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
This commit is contained in:
parent
933ad2c9c8
commit
76bd6ec498
|
@ -3879,14 +3879,14 @@ void ceph_mdsc_handle_fsmap(struct ceph_mds_client *mdsc, struct ceph_msg *msg)
|
||||||
goto err_out;
|
goto err_out;
|
||||||
}
|
}
|
||||||
return;
|
return;
|
||||||
|
|
||||||
bad:
|
bad:
|
||||||
pr_err("error decoding fsmap\n");
|
pr_err("error decoding fsmap\n");
|
||||||
err_out:
|
err_out:
|
||||||
mutex_lock(&mdsc->mutex);
|
mutex_lock(&mdsc->mutex);
|
||||||
mdsc->mdsmap_err = -ENOENT;
|
mdsc->mdsmap_err = err;
|
||||||
__wake_requests(mdsc, &mdsc->waiting_for_map);
|
__wake_requests(mdsc, &mdsc->waiting_for_map);
|
||||||
mutex_unlock(&mdsc->mutex);
|
mutex_unlock(&mdsc->mutex);
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
|
Loading…
Reference in New Issue