On Wed, May 23, 2012 at 09:03:16AM -0400, Stephen Gallagher wrote: > If the mmap cache cannot be initialized (such as insufficient > permissions or SELinux/AppArmor denial), we are supposed to fall back to > our 1.8 behavior of LDB cache only. However, we weren't properly > checking that the cache had been set up and we were always attempting to > dereference the mmap context in fill_pwent() and fill_grent(). > > Fixes https://fedorahosted.org/sssd/ticket/1346
I think we should do the same for sss_mmap_cache_gr_store, too. And perhaps do the check inside the functions themselves and not the callers so that we don't forget to add the check when we add a new call of these functions. _______________________________________________ sssd-devel mailing list [email protected] https://fedorahosted.org/mailman/listinfo/sssd-devel
