Brad Nicholes wrote:
Do we even know that there is a problem with this code? I haven't seen any memory leaks so far. I would hate to go to all of the work to redesign and rewrite the ldap_cache manager for little to no gain.
It does not seem to handle the "we ran out of memory while trying to add to the cache" case very gracefully. It doesn't crash any more, but I'm experiencing false negatives still, which is the problem I was trying to fix when I started trying to fix the code.
Regards, Graham --
