Re: [PATCHv2] virBitmap: Place virBitmapIsAllClear check after virBitmapParse calls

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

 




On 04/13/2015 02:18 PM, Peter Krempa wrote:
> On Mon, Apr 13, 2015 at 14:01:27 +0200, Erik Skultety wrote:
>> This patch adds checks for empty bitmaps right after the calls of
>> virBitmapParse. These only include spots where set API's are called and
>> where domain's XML is parsed.
>> Also, it partially reverts commit 983f5a which added a check for
>> invalid nodeset "0,^0" into virBitmapParse function. This change broke
>> the logic, as an empty bitmap should not cause an error.
>>
>> https://bugzilla.redhat.com/show_bug.cgi?id=1210545
>> ---
>>  src/conf/domain_conf.c   | 35 +++++++++++++++++++++++++++++++----
>>  src/conf/numa_conf.c     | 23 +++++++++++++++++++----
>>  src/qemu/qemu_driver.c   |  5 +++--
>>  src/util/virbitmap.c     |  3 ---
>>  src/xenconfig/xen_sxpr.c |  7 +++++++
>>  tests/virbitmaptest.c    | 13 ++++++++++---
>>  6 files changed, 70 insertions(+), 16 deletions(-)
> 
> ACK, thanks for adding the test.
> 
> Peter
> 
Thank you, now pushed.
Erik

--
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]