<div dir="ltr"><div dir="ltr"><div>This patch breaks the error return value for:</div><div>virDomainControllerDefFormat()</div><div>virDomainSoundDefFormat()<br></div><div>virDomainMemballoonDefFormat()<br></div><div><br></div><div>Patch adds the "ret" variable but in error exit it use "return 0" statement.</div><div>Actually this breaks compilation. <br>Was this code compiled ?</div><div><br></div><div><div>conf/domain_conf.c: In function 'virDomainControllerDefFormat':</div><div>conf/domain_conf.c:24368:9: error: variable 'ret' set but not used [-Werror=unused-but-set-variable]</div><div>     int ret = -1;</div><div>         ^~~</div><div>  CC       test/libvirt_driver_test_la-test_driver.lo</div><div>  CC       vmx/libvirt_vmx_la-vmx.lo</div><div>  CC       vmware/libvirt_driver_vmware_la-vmware_driver.lo</div><div>  CC       vmware/libvirt_driver_vmware_la-vmware_conf.lo</div><div>conf/domain_conf.c: In function 'virDomainSoundDefFormat':</div><div>conf/domain_conf.c:25882:9: error: variable 'ret' set but not used [-Werror=unused-but-set-variable]</div><div>     int ret = -1;</div><div>         ^~~</div><div>conf/domain_conf.c: In function 'virDomainMemballoonDefFormat':</div><div>conf/domain_conf.c:25926:9: error: variable 'ret' set but not used [-Werror=unused-but-set-variable]</div><div>     int ret = -1;</div><div>         ^~~</div></div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, 16 Nov 2018 at 19:56, Pavel Hrdina <<a href="mailto:phrdina@redhat.com">phrdina@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Nov 16, 2018 at 05:21:30PM +0100, Andrea Bolognani wrote:<br>
> In many cases, an early exit from a function would cause<br>
> memory allocated by local virBuffer instances not to be<br>
> released.<br>
> <br>
> Provide proper cleanup paths to solve the issue.<br>
> <br>
> Signed-off-by: Andrea Bolognani <<a href="mailto:abologna@redhat.com" target="_blank">abologna@redhat.com</a>><br>
> ---<br>
>  src/conf/domain_conf.c | 137 ++++++++++++++++++++++++++++++-----------<br>
>  1 file changed, 100 insertions(+), 37 deletions(-)<br>
<br>
Reviewed-by: Pavel Hrdina <<a href="mailto:phrdina@redhat.com" target="_blank">phrdina@redhat.com</a>><br>
--<br>
libvir-list mailing list<br>
<a href="mailto:libvir-list@redhat.com" target="_blank">libvir-list@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/libvir-list" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/libvir-list</a></blockquote></div>