client: remove redundant check

This condition:

    (!dn->inode || dn->inode->caps_issued_mask(mask, true))

is already checked in the surrounding `if`.

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
(cherry picked from commit 41a3f099516ae609a0dbb9c2c020f9523adbc26a)

diff --git a/src/client/Client.cc b/src/client/Client.cc
index 2c4cd72cbe4a34be3d2438a7a5c04aea3881814a..0796aa994cc3335d37b8b9acb30c6236eb302718 100644 (file)
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -7395,9 +7395,9 @@ relookup:
       }
       // dir shared caps?
       if (dir->caps_issued_mask(CEPH_CAP_FILE_SHARED, true)) {
-       if (dn->cap_shared_gen == dir->shared_gen &&
-           (!dn->inode || dn->inode->caps_issued_mask(mask, true)))
-             goto hit_dn;
+       if (dn->cap_shared_gen == dir->shared_gen) {
+         goto hit_dn;
+        }
        if (!dn->inode && (dir->flags & I_COMPLETE)) {
          ldout(cct, 10) << __func__ << " concluded ENOENT locally for "
                         << *dir << " dn '" << dname << "'" << dendl;