Jim Meyering <[EMAIL PROTECTED]> wrote:
> Ron Yorston <[EMAIL PROTECTED]> wrote:
>> "Daniel P. Berrange" <[EMAIL PROTECTED]> wrote:
>>> static int umlDomainShutdown(virDomainPtr dom) {
> ...
>>>+    char *info;
> ...
>>>     VIR_FREE(info);
> ...
>> info should be initialised to NULL, otherwise the VIR_FREE will fail.
>> With the umlMonitorCommand if'ed out this is also broken before the
>> patch, which is how I came to notice it.
>>
>> Ron
>
> Thanks for reporting that!
> I'll commit the following shortly:
>
> Mon Nov 30 10:28:21 CET 2008 Jim Meyering <[EMAIL PROTECTED]>
>
>        uml: don't free an uninitialized variable
>        * src/uml_driver.c (umlDomainShutdown): Initialize "info".
>        Patch and bug report from Ron Yorston <[EMAIL PROTECTED]>.

Committed.

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

Reply via email to