]> xenbits.xensource.com Git - libvirt.git/commitdiff
win32: Pretend that close-on-exec works
authorMatthias Bolte <matthias.bolte@googlemail.com>
Tue, 9 Oct 2012 20:49:46 +0000 (22:49 +0200)
committerMatthias Bolte <matthias.bolte@googlemail.com>
Tue, 9 Oct 2012 21:55:54 +0000 (23:55 +0200)
Currently virNetSocketNew fails because virSetCloseExec fails as there
is no proper implementation for it on Windows at the moment. Workaround
this by pretending that setting close-on-exec on the fd works. This can
be done because libvirt currently lacks the ability to create child
processes on Windows anyway. So there is no point in failing to set a
flag that isn't useful at the moment anyway.

src/util/util.c

index 348c3888dcd2432751edc5d4fc2ce8a8de4afc86..bf608375b3e82157f38790f006c14eb3436198e2 100644 (file)
@@ -275,7 +275,11 @@ int virSetInherit(int fd, bool inherit) {
 
 int virSetInherit(int fd ATTRIBUTE_UNUSED, bool inherit ATTRIBUTE_UNUSED)
 {
-    return -1;
+    /* FIXME: Currently creating child processes is not supported on
+     * Win32, so there is no point in failing calls that are only relevant
+     * when creating child processes. So just pretend that we changed the
+     * inheritance property of the given fd as requested. */
+    return 0;
 }
 
 #endif /* WIN32 */