[libvirt] [PATCH] Prevent streams from becoming libvirtd controlling TTY

[Daniel P. Berrange]

From: "Daniel P. Berrange" <berrange at redhat.com>

When opening a stream to a device which is a TTY, that device
may become the controlling TTY of libvirtd, if libvirtd was
daemonized. This in turn means when the other end of the stream
closes, libvirtd gets SIGHUP, causing it to reload its config.
Prevent this by forcing O_NOCTTY on all streams that are opened

Signed-off-by: Daniel P. Berrange <berrange at redhat.com>
---
 src/fdstream.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/fdstream.c b/src/fdstream.c
index cc2dfe9..9a6f042 100644
--- a/src/fdstream.c
+++ b/src/fdstream.c
@@ -582,6 +582,8 @@ virFDStreamOpenFileInternal(virStreamPtr st,
     VIR_DEBUG("st=%p path=%s oflags=%x offset=%llu length=%llu mode=%o",
               st, path, oflags, offset, length, mode);
 
+    oflags |= O_NOCTTY;
+
     if (oflags & O_CREAT)
         fd = open(path, oflags, mode);
     else
-- 
1.8.1.4