[libvirt] PATCH: 11/28: Reduce return points for UML driver

[Jim Meyering]

Jim Meyering <jim at meyering.net> wrote:
> Ron Yorston <rmy at tigress.co.uk> wrote:
>> "Daniel P. Berrange" <berrange at redhat.com> 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 <meyering at redhat.com>
>
>        uml: don't free an uninitialized variable
>        * src/uml_driver.c (umlDomainShutdown): Initialize "info".
>        Patch and bug report from Ron Yorston <rmy at tigress.co.uk>.

Committed.