On Wed, Aug 27, 2008 at 12:55:48AM +0000, simo wrote: > Jeremy, > I think this one should be instead: > > if (!expired && ret && is_null_sid(psid)) { > > Otherwise you do not try when the negative cache is expired. > > Analogous for the other cases. Got it - thanks for the review. Jeremy.