executor, pkg/csource: make fd numbers consistent

Currently when executor creates fd's it gets: 0, 3, 4.
When tun is enabled: 3, 4, 5.
For C programs: 3, 4, 5.
When run is enabled: 4, 5, 6.

Theoretically it should not matter,
but these fd numbers are probably sometimes are used as data.
So make them consistent in all these cases (3, 4, 5).

1 files changed, 20 insertions, 20 deletions

diff --git a/executor/executor.h b/executor/executor.h
index ee48981a9..063b61ff3 100644
--- a/executor/executor.h
+++ b/executor/executor.h
@@ -198,8 +198,10 @@ void setup_control_pipes()
 		fail("dup2(1, kOutPipeFd) failed");
 	if (dup2(2, 1) < 0)
 		fail("dup2(2, 1) failed");
-	if (close(0))
-		fail("close(0) failed");
+	// We used to close(0), but now we dup stderr to stdin to keep fd numbers
+	// stable across executor and C programs generated by pkg/csource.
+	if (dup2(2, 0) < 0)
+		fail("dup2(2, 0) failed");
 }
 
 void parse_env_flags(uint64 flags)
@@ -707,24 +709,22 @@ bool copyout(char* addr, uint64 size, uint64* res)
 {
 	bool ok = false;
 	NONFAILING(
-		switch (size) {
-		case 1:
-			*res = *(uint8*)addr;
-			break;
-		case 2:
-			*res = *(uint16*)addr;
-			break;
-		case 4:
-			*res = *(uint32*)addr;
-			break;
-		case 8:
-			*res = *(uint64*)addr;
-			break;
-		default:
-			fail("copyout: bad argument size %llu", size);
-		}
-		__atomic_store_n(&ok, true, __ATOMIC_RELEASE);
-	);
+	    switch (size) {
+		    case 1:
+			    *res = *(uint8*)addr;
+			    break;
+		    case 2:
+			    *res = *(uint16*)addr;
+			    break;
+		    case 4:
+			    *res = *(uint32*)addr;
+			    break;
+		    case 8:
+			    *res = *(uint64*)addr;
+			    break;
+		    default:
+			    fail("copyout: bad argument size %llu", size);
+	    } __atomic_store_n(&ok, true, __ATOMIC_RELEASE););
 	return ok;
 }