[libvirt] [PATCH] Close logfile fd after spawning qemu

[Ryota Ozaki]

Hi,

This patch closes logfile fd after spawing qemu in qemudStartVMDaemon.
The fd seems to be closed in the error path, but not in the normal path.
The fd is passed to virExecDaemonize though, but looks not being closed
inside it. Eventually, the fd is never closed during libvirtd lifetime.

Thanks,
  ozaki-r

>From b3d3e0f24c5df5c7677e539bbc2598f2d7fbc3b8 Mon Sep 17 00:00:00 2001
From: Ryota Ozaki <ozaki.ryota at gmail.com>
Date: Thu, 10 Sep 2009 12:53:56 +0900
Subject: [PATCH] Close logfile fd after spawning qemu

---
 src/qemu_driver.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/src/qemu_driver.c b/src/qemu_driver.c
index 8f16e72..f2182c4 100644
--- a/src/qemu_driver.c
+++ b/src/qemu_driver.c
@@ -2187,6 +2187,7 @@ static int qemudStartVMDaemon(virConnectPtr conn,
                            VIR_EXEC_NONBLOCK | VIR_EXEC_CLEAR_CAPS,
                            qemudSecurityHook, &hookData,
                            pidfile);
+    close(logfile);
     VIR_FREE(pidfile);

     /* wait for qemu process to to show up */
-- 
1.6.0.6
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Close-logfile-fd-after-spawning-qemu.patch
Type: application/octet-stream
Size: 762 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20090910/5dda1582/attachment-0001.obj>