James M Leddy <james.leddy at redhat.com> writes: > [...] > Shouldn't it be this ? > > - while ((vginfo2 = primary_vginfo->next)); > + while ((vginfo2 = vginfo2->next)); Isn't that whole loop then equivalent to "vginfo2 = NULL;" ? - FChE