[SSSD] [PATCH] Don't use negative cache in netgroup lookup

Jan Zelený jzeleny at redhat.com
Thu Apr 21 09:32:51 UTC 2011


> On Fri, 2011-04-01 at 09:31 +0200, Jan Zelený wrote:
> > In responder a negative cache is used to indicate that the record has
> > not been found by previous lookup. This approach is however not
> > applicable for netgroup lookup because the design of their lookup is a
> > little different.
> > 
> > This patch removes some pieces of code working with negative cache,
> > because they didn't fuction well. Instead a new flag has been added to
> > the positive cache. This flag indicates if the record in the cache
> > is a record of existing netgroup or it's just a placeholder.
> > 
> > https://fedorahosted.org/sssd/ticket/820
> 
> Nack.
> 
> This patch throws an error on subsequent cached lookups of the netgroup:
> "Fatal error calling setnetgrent_send"
> 
> Find attached a patch to fix this. If you agree, please squash it into
> your patch and I will ack.

Sqashed patch in attachment. Thanks for your correction.

Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jzeleny-011-2-netgroup-ncache.patch
Type: text/x-patch
Size: 5402 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110421/6bb15525/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110421/6bb15525/attachment.sig>


More information about the sssd-devel mailing list