Re: [PATCH v2] conf: Report sensible error for invalid disk name

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

 



On 11/22/2012 02:16 PM, Peter Krempa wrote:
> On 11/20/12 16:20, Martin Kletzander wrote:
>> The error "... but the cause is unknown" appeared for XMLs similar to
>> this:
>>
>>   <disk type='file' device='cdrom'>
>>     <driver name='qemu' type='raw'/>
>>     <source file='/dev/zero'/>
>>     <target dev='sr0'/>
>>   </disk>
>>
>> Notice unsupported disk type (for the driver), but also no address
>> specified. The first part is not a problem and we should not abort
>> immediately because of that, but the combination with the address
>> unknown was causing an unspecified error.
>>
>> While fixing this, I added an error to one place where this return
>> value was not managed properly.
>> ---
>> v2:
>>   - Error moved from virDiskNameToIndex@util/util.c to
>>     virDomainDiskDefAssignAddress@conf/domain_conf.c
>>   - One more error added into qemuParseCommandLine@qemu/qemu_command.c
>>
>>   src/conf/domain_conf.c  | 6 +++++-
>>   src/qemu/qemu_command.c | 6 +++++-
>>   2 files changed, 10 insertions(+), 2 deletions(-)
>>
> 
> ACK.
> 
> Peter
> 

Thanks, pushed.

Martin

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