From: Sam Lang <sam.lang@inktank.com>
Date: Tue, 29 Jan 2013 16:18:29 +0000 (-0600)
Subject: client: Don't use geteuid/gid for fuse ll_create
X-Git-Tag: v0.57~82^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=cf7c3f7d3fc7b8dc3a08a4fbe4ca1c10f2cb6054;p=ceph.git

client: Don't use geteuid/gid for fuse ll_create

Fixes a bug in ll_create where files that already exist at the MDS
don't get the created flag set on reply.  This causes a permissions
check, which fails because geteuid/getegid are 0/0 for ll_create.

Signed-off-by: Sam Lang <sam.lang@inktank.com>
---

diff --git a/src/client/Client.cc b/src/client/Client.cc
index b3d3f6c2596d..feb3beb8a312 100644
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -7243,8 +7243,6 @@ int Client::ll_create(vinodeno_t parent, const char *name, mode_t mode, int flag
   _ll_get(in);
 
   if (!created) {
-    uid_t uid = geteuid();
-    gid_t gid = getegid();
     r = check_permissions(in, flags, uid, gid);
     if (r < 0)
       goto out;