[libvirt] [PATCH] qemu: Free priv->machineName
Commit c3bd0019c0e3 forgot to cleanup after itself. Signed-off-by: Martin Kletzander <mkletzan@redhat.com> --- src/qemu/qemu_cgroup.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup.c index c76d5857f715..1e04a68e0488 100644 --- a/src/qemu/qemu_cgroup.c +++ b/src/qemu/qemu_cgroup.c @@ -1123,5 +1123,7 @@ qemuRemoveCgroup(virDomainObjPtr vm) VIR_DEBUG("Failed to terminate cgroup for %s", vm->def->name); } + VIR_FREE(priv->machineName); + return virCgroupRemove(priv->cgroup); } -- 2.8.1
On Mon, Apr 11, 2016 at 10:16:22AM +0200, Martin Kletzander wrote:
Commit c3bd0019c0e3 forgot to cleanup after itself.
Please include the public bug link in the commit message: https://bugzilla.redhat.com/show_bug.cgi?id=1325043
Signed-off-by: Martin Kletzander <mkletzan@redhat.com> --- src/qemu/qemu_cgroup.c | 2 ++ 1 file changed, 2 insertions(+)
ACK Jan
On Mon, Apr 11, 2016 at 10:35:01AM +0200, Ján Tomko wrote:
On Mon, Apr 11, 2016 at 10:16:22AM +0200, Martin Kletzander wrote:
Commit c3bd0019c0e3 forgot to cleanup after itself.
Please include the public bug link in the commit message: https://bugzilla.redhat.com/show_bug.cgi?id=1325043
Squashed in and pushed, thanks. I wrote the message before I discovered that bug, but sent it just now.
Signed-off-by: Martin Kletzander <mkletzan@redhat.com> --- src/qemu/qemu_cgroup.c | 2 ++ 1 file changed, 2 insertions(+)
ACK
Jan
participants (2)
-
Ján Tomko -
Martin Kletzander