On Wed, Feb 20, 2008 at 03:23:52PM +0100, Jim Meyering wrote:
Jim Meyering <jim(a)meyering.net> 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(a)redhat.com | libxml GNOME XML XSLT toolkit
http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine
http://rpmfind.net/