responder_get_domain(): remove timeout calculation

The current timout calculation code in responder_get_domain() is flawed
and I think it always was. I removed the related code because
- it currently has no effect, a match is returned even if it is expired
- that callers do not have any code to handle expired domains.

1 files changed, 1 insertions, 10 deletions

diff --git a/src/responder/common/responder_common.c b/src/responder/common/responder_common.c
index 480b0461a..008d622de 100644
--- a/src/responder/common/responder_common.c
+++ b/src/responder/common/responder_common.c
@@ -899,8 +899,6 @@ int sss_dp_get_domain_conn(struct resp_ctx *rctx, const char *domain,
 struct sss_domain_info *
 responder_get_domain(struct resp_ctx *rctx, const char *name)
 {
-    time_t now = time(NULL);
-    time_t time_diff = 0;
     struct sss_domain_info *dom;
     struct sss_domain_info *ret_dom = NULL;
 
@@ -909,19 +907,12 @@ responder_get_domain(struct resp_ctx *rctx, const char *name)
             continue;
         }
 
-        if (!IS_SUBDOMAIN(dom)) {
-            time_diff = now - dom->subdomains_last_checked.tv_sec;
-        }
         if (strcasecmp(dom->name, name) == 0 ||
             (dom->flat_name != NULL &&
              strcasecmp(dom->flat_name, name) == 0)) {
             ret_dom = dom;
-            if (!IS_SUBDOMAIN(dom) ||
-                (IS_SUBDOMAIN(dom) && time_diff < rctx->domains_timeout)) {
-                break;
-            }
+            break;
         }
-
     }
 
     if (!ret_dom) {