From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:54781) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1R0bsl-0005bw-Q1 for qemu-devel@nongnu.org; Mon, 05 Sep 2011 12:18:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1R0bsk-0005FD-OM for qemu-devel@nongnu.org; Mon, 05 Sep 2011 12:18:47 -0400 Received: from e28smtp01.in.ibm.com ([122.248.162.1]:51684) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1R0bsk-0005DQ-4l for qemu-devel@nongnu.org; Mon, 05 Sep 2011 12:18:46 -0400 Received: from d28relay05.in.ibm.com (d28relay05.in.ibm.com [9.184.220.62]) by e28smtp01.in.ibm.com (8.14.4/8.13.1) with ESMTP id p85GIewJ013079 for ; Mon, 5 Sep 2011 21:48:40 +0530 Received: from d28av05.in.ibm.com (d28av05.in.ibm.com [9.184.220.67]) by d28relay05.in.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id p85GIdlG3051598 for ; Mon, 5 Sep 2011 21:48:39 +0530 Received: from d28av05.in.ibm.com (loopback [127.0.0.1]) by d28av05.in.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id p85GIdDX029209 for ; Tue, 6 Sep 2011 02:18:39 +1000 From: "M. Mohan Kumar" Date: Mon, 5 Sep 2011 21:48:31 +0530 Message-Id: <1315239516-4451-11-git-send-email-mohan@in.ibm.com> In-Reply-To: <1315239516-4451-1-git-send-email-mohan@in.ibm.com> References: <1315239516-4451-1-git-send-email-mohan@in.ibm.com> Subject: [Qemu-devel] [PATCH V12 10/15] hw/9pfs: Move file post creation changes to none security model List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org, Anthony Liguori , Stefan Hajnoczi After creating a file object, its permission and ownership details are updated as per 9p client's request for both passthrough and none security model. But with chrooted environment its not required for passthrough security model. Move all post file creation changes to none security model. Signed-off-by: M. Mohan Kumar --- hw/9pfs/virtio-9p-local.c | 25 +++++++++---------------- 1 files changed, 9 insertions(+), 16 deletions(-) diff --git a/hw/9pfs/virtio-9p-local.c b/hw/9pfs/virtio-9p-local.c index 3b97f51..68551e2 100644 --- a/hw/9pfs/virtio-9p-local.c +++ b/hw/9pfs/virtio-9p-local.c @@ -155,23 +155,16 @@ static int local_set_xattr(const char *path, FsCred *credp) return 0; } -static int local_post_create_passthrough(FsContext *fs_ctx, const char *path, - FsCred *credp) +static int local_post_create_none(const char *path, FsCred *credp) { - char buffer[PATH_MAX]; + int retval; - if (chmod(rpath(fs_ctx, path, buffer), credp->fc_mode & 07777) < 0) { + if (chmod(path, credp->fc_mode & 07777) < 0) { return -1; } - if (lchown(rpath(fs_ctx, path, buffer), credp->fc_uid, - credp->fc_gid) < 0) { - /* - * If we fail to change ownership and if we are - * using security model none. Ignore the error - */ - if (fs_ctx->fs_sm != SM_NONE) { - return -1; - } + retval = lchown(path, credp->fc_uid, credp->fc_gid); + if (retval < 0) { + /* Ignore return value for none security model */ } return 0; } @@ -347,7 +340,7 @@ static int local_mknod(FsContext *fs_ctx, V9fsPath *dir_path, if (err == -1) { goto out; } - err = local_post_create_passthrough(fs_ctx, path, credp); + err = local_post_create_none(buffer, credp); if (err == -1) { serrno = errno; goto err_end; @@ -395,7 +388,7 @@ static int local_mkdir(FsContext *fs_ctx, V9fsPath *dir_path, if (err == -1) { goto out; } - err = local_post_create_passthrough(fs_ctx, path, credp); + err = local_post_create_none(buffer, credp); if (err == -1) { serrno = errno; goto err_end; @@ -482,7 +475,7 @@ static int local_open2(FsContext *fs_ctx, V9fsPath *dir_path, const char *name, err = fd; goto out; } - err = local_post_create_passthrough(fs_ctx, path, credp); + err = local_post_create_none(buffer, credp); if (err == -1) { serrno = errno; goto err_end; -- 1.7.6