[libvirt] [PATCH] qemu: fix uninitialized variable in qemuParseCommandLine

Osier Yang jyang at redhat.com
Fri Sep 14 13:14:10 UTC 2012


On 2012年09月14日 20:55, Osier Yang wrote:
> On 2012年09月14日 20:50, Ján Tomko wrote:
>> Newly added if branch for kvm_pv_eoi did not set the ret variable.
>> ---
>> src/qemu/qemu_command.c | 3 +--
>> 1 files changed, 1 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
>> index ec825bc..d0c8e22 100644
>> --- a/src/qemu/qemu_command.c
>> +++ b/src/qemu/qemu_command.c
>> @@ -7655,7 +7655,7 @@ qemuParseCommandLineCPU(virDomainDefPtr dom,
>> else if (*p == '+' || *p == '-') {
>> char *feature;
>> int policy;
>> - int ret;
>> + int ret = 0;
>>
>> if (*p == '+')
>> policy = VIR_CPU_FEATURE_REQUIRE;
>> @@ -7702,7 +7702,6 @@ qemuParseCommandLineCPU(virDomainDefPtr dom,
>> goto error;
>> }
>> dom->clock.timers[i]->present = present;
>> - ret = 0;
>> } else if (STREQ(feature, "kvm_pv_eoi")) {
>> if (policy == VIR_CPU_FEATURE_REQUIRE)
>> dom->apic_eoi = VIR_DOMAIN_APIC_EOI_ON;
>
>
> ACK,I get the build failure too.
>

Now pushed.

Regards,
Osier




More information about the libvir-list mailing list