On Thu, Apr 23, 2009 at 01:32:18PM +0300, Elad Efrat wrote: > + for (mp = CIRCLEQ_FIRST(&mountlist); mp != NULL; mp = nmp) { Actually, it should be something like "mp != (void *)&mountlist". Have a look at the existing loops.