On Wed, Feb 20, 2008 at 03:23:52PM +0100, Jim Meyering wrote: > Jim Meyering <jim@xxxxxxxxxxxx> wrote: > ... > >> + if (conf_dir != NULL) > >> + return -1 > > > > Whoa. > > Chris Lalance pointed out another obvious flaw. > > I meant to add *this*: > > > >> + if (conf_dir == NULL) > >> + return -1 > > > > Will repost shortly. > > Thanks, Chris! > > Here's the corrected patch: Okay, curiously I didn't spotted those problems :-\ Daniel -- Red Hat Virtualization group http://redhat.com/virtualization/ Daniel Veillard | virtualization library http://libvirt.org/ veillard@xxxxxxxxxx | libxml GNOME XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/ -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list