From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Eric W. Biederman" Subject: [PATCH 05/14] xfs: Update xfs_ioctl_setattr to handle projids in any user namespace Date: Wed, 13 Mar 2013 15:23:06 -0700 Message-ID: <1363213395-10988-5-git-send-email-ebiederm@xmission.com> References: <87boan3prc.fsf@xmission.com> <1363213395-10988-1-git-send-email-ebiederm@xmission.com> Cc: Linux Containers , "Serge E. Hallyn" , xfs@oss.sgi.com, Ben Myers , Alex Elder , Dave Chinner , "Eric W. Biederman" To: Return-path: Received: from out01.mta.xmission.com ([166.70.13.231]:57402 "EHLO out01.mta.xmission.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933273Ab3CMWZ0 (ORCPT ); Wed, 13 Mar 2013 18:25:26 -0400 In-Reply-To: <1363213395-10988-1-git-send-email-ebiederm@xmission.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: From: "Eric W. Biederman" - Convert the userspace value in fa->fsx_projid into a kprojid and store it in the variable projid. - Verify that xfs can store the projid after it is converted into xfs's user namespace. - Replace uses of fa->fsx_projid with projid throughout xfs_ioctl_setattr. Cc: Ben Myers Cc: Alex Elder Cc: Dave Chinner Signed-off-by: "Eric W. Biederman" --- fs/xfs/xfs_ioctl.c | 26 ++++++++++++++++++-------- 1 files changed, 18 insertions(+), 8 deletions(-) diff --git a/fs/xfs/xfs_ioctl.c b/fs/xfs/xfs_ioctl.c index 1dd72d9..9aeecfb 100644 --- a/fs/xfs/xfs_ioctl.c +++ b/fs/xfs/xfs_ioctl.c @@ -925,6 +925,7 @@ xfs_ioctl_setattr( struct xfs_dquot *gdqp = NULL; struct xfs_dquot *olddquot = NULL; int code; + kprojid_t projid = INVALID_PROJID; trace_xfs_ioctl_setattr(ip); @@ -934,11 +935,20 @@ xfs_ioctl_setattr( return XFS_ERROR(EIO); /* - * Disallow 32bit project ids when projid32bit feature is not enabled. + * Verify the specifid project id is valid. */ - if ((mask & FSX_PROJID) && (fa->fsx_projid > (__uint16_t)-1) && - !xfs_sb_version_hasprojid32bit(&ip->i_mount->m_sb)) - return XFS_ERROR(EINVAL); + if (mask & FSX_PROJID) { + projid = make_kprojid(current_user_ns(), fa->fsx_projid); + if (!projid_valid(projid)) + return XFS_ERROR(EINVAL); + + /* + * Disallow 32bit project ids when projid32bit feature is not enabled. + */ + if ((from_kprojid(&init_user_ns, projid) > (__uint16_t)-1) && + !xfs_sb_version_hasprojid32bit(&ip->i_mount->m_sb)) + return XFS_ERROR(EINVAL); + } /* * If disk quotas is on, we make sure that the dquots do exist on disk, @@ -950,7 +960,7 @@ xfs_ioctl_setattr( */ if (XFS_IS_QUOTA_ON(mp) && (mask & FSX_PROJID)) { code = xfs_qm_vop_dqalloc(ip, ip->i_d.di_uid, - ip->i_d.di_gid, fa->fsx_projid, + ip->i_d.di_gid, projid, XFS_QMOPT_PQUOTA, &udqp, &gdqp); if (code) return code; @@ -986,7 +996,7 @@ xfs_ioctl_setattr( if (mask & FSX_PROJID) { if (XFS_IS_QUOTA_RUNNING(mp) && XFS_IS_PQUOTA_ON(mp) && - !projid_eq(ip->i_d.di_projid, fa->fsx_projid)) { + !projid_eq(ip->i_d.di_projid, projid)) { ASSERT(tp); code = xfs_qm_vop_chown_reserve(tp, ip, udqp, gdqp, capable(CAP_FOWNER) ? @@ -1104,12 +1114,12 @@ xfs_ioctl_setattr( * Change the ownerships and register quota modifications * in the transaction. */ - if (!projid_eq(ip->i_d.di_projid, fa->fsx_projid)) { + if (!projid_eq(ip->i_d.di_projid, projid)) { if (XFS_IS_QUOTA_RUNNING(mp) && XFS_IS_PQUOTA_ON(mp)) { olddquot = xfs_qm_vop_chown(tp, ip, &ip->i_gdquot, gdqp); } - ip->i_d.di_projid = fa->fsx_projid; + ip->i_d.di_projid = projid; /* * We may have to rev the inode as well as -- 1.7.5.4