On Wed, Feb 20, 2008 at 03:23:52PM +0100, Jim Meyering wrote: > Jim Meyering <[EMAIL PROTECTED]> 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/ [EMAIL PROTECTED] | libxml GNOME XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/ -- Libvir-list mailing list Libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list