client: switch _lookup() implementation to UserPerm

Signed-off-by: Greg Farnum <gfarnum@redhat.com>

diff --git a/src/client/Client.cc b/src/client/Client.cc
index 26aee3432811477026460bffbd5221ae548fb984..5f6e43b1dfe6346759a43d02848925e76a78f84f 100644 (file)
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -5860,12 +5860,11 @@ int Client::_do_lookup(Inode *dir, const string& name, int mask,
   return r;
 }
 
-int Client::_lookup(Inode *dir, const string& dname, int mask,
-                   InodeRef *target, int uid, int gid)
+int Client::_lookup(Inode *dir, const string& dname, int mask, InodeRef *target,
+                   const UserPerm& perms)
 {
   int r = 0;
   Dentry *dn = NULL;
-  UserPerm perms(uid, gid);
 
   if (!dir->is_dir()) {
     r = -ENOTDIR;

diff --git a/src/client/Client.h b/src/client/Client.h
index 695dc108436d547a09007fdbec41f185f41895e7..60da3a639062bafbc22d6292356f65e62c533d9f 100644 (file)
--- a/src/client/Client.h
+++ b/src/client/Client.h
@@ -779,10 +779,13 @@ private:
   //   call these with client_lock held!
   int _do_lookup(Inode *dir, const string& name, int mask, InodeRef *target,
                 const UserPerm& perms);
-  int _lookup(Inode *dir, const string& dname, int mask, InodeRef *target, int uid, int gid);
+
+  int _lookup(Inode *dir, const string& dname, int mask, InodeRef *target,
+             const UserPerm& perm);
   int _lookup(Inode *dir, const string& dname, int mask, InodeRef *target,
-             const UserPerm& perm) {
-    return _lookup(dir, dname, mask, target, perm.uid(), perm.gid());
+             int uid, int gid) {
+    UserPerm perms(uid, gid);
+    return _lookup(dir, dname, mask, target, perms);
   }
 
   int _link(Inode *in, Inode *dir, const char *name, const UserPerm& perm,