sssd.git/src/responder/common, branch prompting

RESPONDER: Removing of redudant function

2016-05-11T11:43:32+00:00

There is redudant function responder_get_neg_timeout_from_confdb().
This patch removes it.

Reviewed-by: Pavel Březina

RESPONDERS: Negcache in resp_ctx preparing

2016-05-11T11:42:58+00:00

Preparation for initialization of negative cache in common responder.

Reviewed-by: Pavel Březina

RESPONDER: Removing neg_timeout from nss responder

2016-05-11T10:52:13+00:00

Timeout of negative cahce is handled by internal negative cache
context. This patch removes neg_timeout from struct nss_ctx.

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

Reviewed-by: Pavel Březina

RESPONDER: Removing neg_timeout from pam responder

2016-05-11T10:50:25+00:00

It removes neg_timeout parameter from struct pam_ctx. Timeout is
handled by context of negative cache internally.

This patch additioanlly removes neg_timeout from struct cache_req_state.

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

Reviewed-by: Pavel Březina

NEGCACHE: Adding getter for timeout

2016-05-11T10:50:11+00:00

It adds new function to negative cache API:
* int sss_ncache_get_timeout(struct sss_nc_ctx *ctx);

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

Reviewed-by: Pavel Březina

NEGCACHE: Removing timeout from sss_ncache_check_*

2016-05-11T10:49:51+00:00

It removes timeout parameter from check functions of negative cache.
Timeout is set ny init function and it is handled internally.

API change:
 * int sss_ncache_check_...(struct sss_nc_ctx *ctx,
                            int ttl,      <----- timeout vanished
                            struct sss_domain_info *dom,
                            ...);

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

Reviewed-by: Pavel Březina

NEGCACHE: Adding timeout to struct sss_nc_ctx

2016-05-11T10:49:45+00:00

It adds timeout of negative cache to handling
struct sss_nc_ctx.

There is one change in API of negatice cache:
 * int sss_ncache_init(TALLOC_CTX *memctx,
                       uint32_t timeout,  <----- new
                       struct sss_nc_ctx **_ctx);

There is also one new function in common/responder:
*  errno_t responder_get_neg_timeout_from_confdb(struct confdb_ctx *cdb,
                                                 uint32_t *ncache_timeout);

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

Reviewed-by: Pavel Březina

NEGCACHE: Removing of condition for ttl = -1

2016-04-20T18:07:21+00:00

If ttl = -1 then function sss_ncache_check_str() returns EEXIST without
checking negcache. This behaviour is out of logic.

We use ttl = 0 for permanent caching.

Reviewed-by: Lukáš Slebodník

Responders: Fix client destructor

2016-03-17T08:18:26+00:00

To close a socket associated to an fd event we must set the close
function of the event and not associate a destructor to a parent context.

Otherwise the destructor will close() the socket before the fd event is
freed, and this may cause invalid calls on a closed file descriptor to
poll/epoll/etc.

Discovered by looking at strace output.

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

Reviewed-by: Pavel Březina

cache_req: remove old comment

2016-03-14T16:34:51+00:00

Reviewed-by: Jakub Hrozek