Re: [libvirt] [PATCH] Make int-overflow test case pass

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Stefan Berger wrote:
> Jim Meyering <jim@xxxxxxxxxxxx> wrote on 03/29/2010 02:34:06 PM:
>> Stefan Berger wrote:
>> > This fix makes the int-overflow test case pass.
>> >
>> > Signed-off-by: Stefan Berger <stefanb@xxxxxxxxxx>
>> >
>> > Index: libvirt/tests/int-overflow
>> > ===================================================================
>> > --- libvirt.orig/tests/int-overflow
>> > +++ libvirt/tests/int-overflow
>> > @@ -14,6 +14,7 @@ fi
>> >  . "$srcdir/test-lib.sh"
>> >
>> >  echo "error: failed to get domain '4294967298'" > exp || fail=1
>> > +echo "error: Failed to start the nwfilter driver: Is the daemon
>> running ?" >> exp || fail=1
>> >  echo domname 4294967298 | $abs_top_builddir/tools/virsh --quiet \
>> >      --connect test://$abs_top_srcdir/examples/xml/test/testnode.xml \
>> >      > /dev/null 2> err || fail=1
>>
>> If we've already failed to get a domain, does it even
>> make sense to attempt to start the nwfilter driver?
>
> This would be a temporary fix. The root problem I think stems from there not
> being a test driver with name "nwfilter'.

As a temporary fix, that's fine.  Thanks.
It'd be nice to indicate that by adding a "FIXME" comment.

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]