On Mon, 12 Feb 2007, Simon Kelley wrote:
For giggles, the faulty code was:

   if (crecp->flags & F_IMMORTAL)
       while (*up && (!(*up)->flags & F_IMMORTAL))
         up = &((*up)->hash_next);

which should, (of course) have been:

   if (crecp->flags & F_IMMORTAL)
       while (*up && !((*up)->flags & F_IMMORTAL))
         up = &((*up)->hash_next);

 This is why I leave the programming up to the programmers.
 Rolling this update out to Fedora...

     Jima

Reply via email to