From: "Yan, Zheng" <[email protected]>

req->r_locked_dir is NULL if the request was created by
open_root_dentry(). ceph_fill_trace() lacks check for this case.
It causes oops when mounting cephfs with non-existing path.

Signed-off-by: Yan, Zheng <[email protected]>
---
 fs/ceph/inode.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/fs/ceph/inode.c b/fs/ceph/inode.c
index 9fff9f3..40e2ef1 100644
--- a/fs/ceph/inode.c
+++ b/fs/ceph/inode.c
@@ -992,6 +992,13 @@ int ceph_fill_trace(struct super_block *sb, struct 
ceph_mds_request *req,
        if (rinfo->head->is_dentry) {
                struct inode *dir = req->r_locked_dir;
 
+               /*
+                * req->r_locked_dir is null if the request was created
+                * by open_root_dentry()
+                */
+               if (!dir)
+                       return le32_to_cpu(rinfo->head->result);
+
                err = fill_inode(dir, &rinfo->diri, rinfo->dirfrag,
                                 session, req->r_request_started, -1,
                                 &req->r_caps_reservation);
-- 
1.7.11.2

--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to