aboutsummaryrefslogtreecommitdiffstats
path: root/executor
diff options
context:
space:
mode:
authorAleksandr Nogikh <nogikh@google.com>2021-12-06 11:30:24 +0000
committerAleksandr Nogikh <wp32pw@gmail.com>2021-12-06 14:28:24 +0100
commita5338d9cd2efcf914ba78edfe2615c7c0657b157 (patch)
treeb05d71e5ceac9986fb2414215fb714091745c79d /executor
parentb18e90e04ac0c6a06518fc9469dc7d12d83db901 (diff)
executor: spread overlapping fds
There's a chance that the methods from common_bsd.h and common_linux.h could dup2 (and thus close) an fd belonging to a kcov instance. Prevent this by adjusting fd consts.
Diffstat (limited to 'executor')
-rw-r--r--executor/common_bsd.h2
-rw-r--r--executor/common_linux.h6
2 files changed, 4 insertions, 4 deletions
diff --git a/executor/common_bsd.h b/executor/common_bsd.h
index 332e68e5f..7293df993 100644
--- a/executor/common_bsd.h
+++ b/executor/common_bsd.h
@@ -230,7 +230,7 @@ static void initialize_tun(int tun_id)
}
// Remap tun onto higher fd number to hide it from fuzzer and to keep
// fd numbers stable regardless of whether tun is opened or not (also see kMaxFd).
- const int kTunFd = 240;
+ const int kTunFd = 200;
if (dup2(tunfd, kTunFd) < 0)
fail("dup2(tunfd, kTunFd) failed");
close(tunfd);
diff --git a/executor/common_linux.h b/executor/common_linux.h
index 7df9476eb..cbf311b22 100644
--- a/executor/common_linux.h
+++ b/executor/common_linux.h
@@ -567,7 +567,7 @@ static void initialize_tun(void)
}
// Remap tun onto higher fd number to hide it from fuzzer and to keep
// fd numbers stable regardless of whether tun is opened or not (also see kMaxFd).
- const int kTunFd = 240;
+ const int kTunFd = 200;
if (dup2(tunfd, kTunFd) < 0)
fail("dup2(tunfd, kTunFd) failed");
close(tunfd);
@@ -632,7 +632,7 @@ static void initialize_tun(void)
#endif
#if SYZ_EXECUTOR || __NR_syz_init_net_socket || SYZ_DEVLINK_PCI
-const int kInitNetNsFd = 239; // see kMaxFd
+const int kInitNetNsFd = 201; // see kMaxFd
#endif
#if SYZ_EXECUTOR || SYZ_DEVLINK_PCI || SYZ_NET_DEVICES
@@ -2582,7 +2582,7 @@ static void initialize_vhci()
// Remap vhci onto higher fd number to hide it from fuzzer and to keep
// fd numbers stable regardless of whether vhci is opened or not (also see kMaxFd).
- const int kVhciFd = 241;
+ const int kVhciFd = 202;
if (dup2(vhci_fd, kVhciFd) < 0)
fail("dup2(vhci_fd, kVhciFd) failed");
close(vhci_fd);