[libvirt] [PATCH 2/2] uml:release config object when uml driver shutdown
--- src/uml/uml_driver.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/uml/uml_driver.c b/src/uml/uml_driver.c index b08212d..09a777c 100644 --- a/src/uml/uml_driver.c +++ b/src/uml/uml_driver.c @@ -640,6 +640,7 @@ umlShutdown(void) { virEventRemoveHandle(uml_driver->inotifyWatch); VIR_FORCE_CLOSE(uml_driver->inotifyFD); virObjectUnref(uml_driver->caps); + virObjectUnref(uml_driver->xmlconf); /* shutdown active VMs * XXX allow them to stay around & reconnect */ -- 1.7.11.2
On 03/27/2013 12:03 PM, Guannan Ren wrote:
--- src/uml/uml_driver.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/src/uml/uml_driver.c b/src/uml/uml_driver.c index b08212d..09a777c 100644 --- a/src/uml/uml_driver.c +++ b/src/uml/uml_driver.c @@ -640,6 +640,7 @@ umlShutdown(void) { virEventRemoveHandle(uml_driver->inotifyWatch); VIR_FORCE_CLOSE(uml_driver->inotifyFD); virObjectUnref(uml_driver->caps); + virObjectUnref(uml_driver->xmlconf);
/* shutdown active VMs * XXX allow them to stay around & reconnect */
Although I'm not very familiar with the uml codebase, from the looks of it, this sounds reasonable. ACK, Martin
On 03/28/2013 12:01 AM, Martin Kletzander wrote:
On 03/27/2013 12:03 PM, Guannan Ren wrote:
--- src/uml/uml_driver.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/src/uml/uml_driver.c b/src/uml/uml_driver.c index b08212d..09a777c 100644 --- a/src/uml/uml_driver.c +++ b/src/uml/uml_driver.c @@ -640,6 +640,7 @@ umlShutdown(void) { virEventRemoveHandle(uml_driver->inotifyWatch); VIR_FORCE_CLOSE(uml_driver->inotifyFD); virObjectUnref(uml_driver->caps); + virObjectUnref(uml_driver->xmlconf);
/* shutdown active VMs * XXX allow them to stay around & reconnect */
Although I'm not very familiar with the uml codebase, from the looks of it, this sounds reasonable.
ACK,
Martin
Pushed, thanks guy Guannan
participants (2)
-
Guannan Ren -
Martin Kletzander