diff options
| author | Sage Weil | 2010-11-08 16:28:52 +0100 |
|---|---|---|
| committer | Sage Weil | 2010-11-08 16:29:05 +0100 |
| commit | cb4276cca4695670916a82e359f2e3776f0a9138 (patch) | |
| tree | 09695ba622771c933e045bafe63c1da1139266f3 /fs/ceph/mds_client.c | |
| parent | ceph: fix rdcache_gen usage and invalidate (diff) | |
| download | kernel-qcow2-linux-cb4276cca4695670916a82e359f2e3776f0a9138.tar.gz kernel-qcow2-linux-cb4276cca4695670916a82e359f2e3776f0a9138.tar.xz kernel-qcow2-linux-cb4276cca4695670916a82e359f2e3776f0a9138.zip | |
ceph: fix uid/gid on resent mds requests
MDS requests can be rebuilt and resent in non-process context, but were
filling in uid/gid from current_fsuid/gid. Put that information in the
request struct on request setup.
This fixes incorrect (and root) uid/gid getting set for requests that
are forwarded between MDSs, usually due to metadata migrations.
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'fs/ceph/mds_client.c')
| -rw-r--r-- | fs/ceph/mds_client.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/fs/ceph/mds_client.c b/fs/ceph/mds_client.c index 3142b15940c2..d22fbbef1959 100644 --- a/fs/ceph/mds_client.c +++ b/fs/ceph/mds_client.c @@ -529,6 +529,9 @@ static void __register_request(struct ceph_mds_client *mdsc, ceph_mdsc_get_request(req); __insert_request(mdsc, req); + req->r_uid = current_fsuid(); + req->r_gid = current_fsgid(); + if (dir) { struct ceph_inode_info *ci = ceph_inode(dir); @@ -1588,8 +1591,8 @@ static struct ceph_msg *create_request_message(struct ceph_mds_client *mdsc, head->mdsmap_epoch = cpu_to_le32(mdsc->mdsmap->m_epoch); head->op = cpu_to_le32(req->r_op); - head->caller_uid = cpu_to_le32(current_fsuid()); - head->caller_gid = cpu_to_le32(current_fsgid()); + head->caller_uid = cpu_to_le32(req->r_uid); + head->caller_gid = cpu_to_le32(req->r_gid); head->args = req->r_args; ceph_encode_filepath(&p, end, ino1, path1); |