On 05/05/2016 12:08 PM, Petr Cech wrote:
Hello,

this patch set moves negative cache from particular context of given
responder to common context resp_ctx.

It is reaction on Fix taloc context for negative cache [1]. And it
replaces it.

It is applicable after [PATCH SET] Make the negcache timeout part of
nc_ctx [2]

For clarity, there is branch with all negative cache's patches [3].


Links:

[1]
https://www.mail-archive.com/sssd-devel@lists.fedorahosted.org/msg26529.html


[2]
https://www.mail-archive.com/sssd-devel@lists.fedorahosted.org/msg26515.html


[3] https://github.com/celestian/sssd/commits/ncache_v2

Sorry,

I recognized that attached patch could be last patch of this patch set.

Regards

--
Petr^4 Čech
>From d2de3f7cfdd3851df70cb74d281f97df45467a48 Mon Sep 17 00:00:00 2001
From: Petr Cech <pc...@redhat.com>
Date: Thu, 5 May 2016 10:28:30 -0400
Subject: [PATCH 7/7] RESPONDER: Removing of redudant function

There is redudant function responder_get_neg_timeout_from_confdb().
This patch removes it.
---
 src/responder/common/responder.h        |  3 ---
 src/responder/common/responder_common.c | 28 ----------------------------
 2 files changed, 31 deletions(-)

diff --git a/src/responder/common/responder.h b/src/responder/common/responder.h
index d3f5c8d94000a7b0ca8f41fb3cb40b32186d391e..7fc2770243dadcd7180736594ee51ea67b168416 100644
--- a/src/responder/common/responder.h
+++ b/src/responder/common/responder.h
@@ -347,7 +347,4 @@ errno_t sss_parse_inp_recv(struct tevent_req *req, TALLOC_CTX *mem_ctx,
 const char **parse_attr_list_ex(TALLOC_CTX *mem_ctx, const char *conf_str,
                                 const char **defaults);
 
-errno_t responder_get_neg_timeout_from_confdb(struct confdb_ctx *cdb,
-                                              uint32_t *ncache_timeout);
-
 #endif /* __SSS_RESPONDER_H__ */
diff --git a/src/responder/common/responder_common.c b/src/responder/common/responder_common.c
index 4f620f7d9bc47db374144f008d1855fbe1178f50..883b22a6548c29738f23a5ccf6747969f4e7146c 100644
--- a/src/responder/common/responder_common.c
+++ b/src/responder/common/responder_common.c
@@ -1129,31 +1129,3 @@ void responder_set_fd_limit(rlim_t fd_limit)
                "Proceeding with system values\n");
     }
 }
-
-errno_t responder_get_neg_timeout_from_confdb(struct confdb_ctx *cdb,
-                                              uint32_t *ncache_timeout)
-{
-    int value;
-    int ret;
-
-    ret = confdb_get_int(cdb, CONFDB_NSS_CONF_ENTRY,
-                         CONFDB_NSS_ENTRY_NEG_TIMEOUT, 15,
-                         &value);
-    if (ret != EOK) {
-        DEBUG(SSSDBG_FATAL_FAILURE,
-              "Fatal failure of setup negative cache timeout.\n");
-        ret = ENOENT;
-        goto done;
-    }
-
-    if (value < 0) {
-        ret = EINVAL;
-        goto done;
-    }
-
-    *ncache_timeout = value;
-    ret = EOK;
-
-done:
-    return ret;
-}
-- 
2.5.5

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org

Reply via email to