Merge pull request mozart#281 from layus/fix-dup

Use dup2 to remove warnings and explicit the intent
layus · Sep 19, 2016 · 7f83066 · 7f83066
2 parents 0b50284 + 516dc42
commit 7f83066
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/vm/boostenv/main/modos.hh b/vm/boostenv/main/modos.hh
@@ -885,8 +885,8 @@ public:
                 raiseLastOSError(vm, "open");
             }
 
-            dup(dn); // stdout
-            dup(dn); // stderr
+            dup2(dn, 1); // stdout
+            dup2(dn, 2); // stderr
           }
 #endif
 
@@ -1090,9 +1090,9 @@ public:
                   close(i);
                 }
               }
-              dup(socketHandle);
-              dup(socketHandle);
-              dup(socketHandle);
+              dup2(socketHandle, 0); // stdin
+              dup2(socketHandle, 1); // stdout
+              dup2(socketHandle, 2); // stderr
 
               auto c_executable = executable.string;
               auto c_argv = new char*[argc+1];