responders: remove checks that are handled inside cache_req

Those checks were redundant, since they are already part of cache_req.

Resolves:
https://fedorahosted.org/sssd/ticket/3151

Reviewed-by: Lukáš Slebodník <lslebodn@redhat.com>

3 files changed, 1 insertions, 19 deletions

diff --git a/src/responder/nss/nsssrv_cmd.c b/src/responder/nss/nsssrv_cmd.c
index 0352ff348..062a1f339 100644
--- a/src/responder/nss/nsssrv_cmd.c
+++ b/src/responder/nss/nsssrv_cmd.c
@@ -5418,10 +5418,7 @@ static void users_find_by_cert_done(struct tevent_req *req)
 
     ret = cache_req_user_by_cert_recv(cctx, req, &result);
     talloc_zfree(req);
-    if (ret == ENOENT || result->count == 0) {
-        ret = ENOENT;
-        goto done;
-    } else if (ret != EOK) {
+    if (ret != EOK) {
         goto done;
     }
 
diff --git a/src/responder/pac/pacsrv_cmd.c b/src/responder/pac/pacsrv_cmd.c
index ddb8c0c90..e3aab8886 100644
--- a/src/responder/pac/pacsrv_cmd.c
+++ b/src/responder/pac/pacsrv_cmd.c
@@ -238,14 +238,6 @@ static void pac_resolve_user_sid_done(struct tevent_req *req)
         return;
     }
 
-    if (result->count != 1) {
-        DEBUG(SSSDBG_CRIT_FAILURE,
-              "Expected only 1 result for SID lookup, got [%ud].\n",
-              result->count);
-        ret = EINVAL;
-        goto done;
-    }
-
     user_attrs = sysdb_new_attrs(pr_ctx);
     if (user_attrs == NULL) {
         DEBUG(SSSDBG_OP_FAILURE, "sysdb_new_attrs failed.\n");
diff --git a/src/responder/pam/pamsrv_cmd.c b/src/responder/pam/pamsrv_cmd.c
index ca4c37605..e73a81906 100644
--- a/src/responder/pam/pamsrv_cmd.c
+++ b/src/responder/pam/pamsrv_cmd.c
@@ -1431,13 +1431,6 @@ static void pam_forwarder_lookup_by_cert_done(struct tevent_req *req)
         goto done;
     }
 
-    if (ret == EOK && result->count > 1) {
-        DEBUG(SSSDBG_CRIT_FAILURE,
-              "Search by certificate returned more than one result.\n");
-        ret = EINVAL;
-        goto done;
-    }
-
     if (ret == EOK) {
         if (preq->domain == NULL) {
             preq->domain = result->domain;