Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 51aec226 authored by Nick Kralevich's avatar Nick Kralevich Committed by android-build-merger
Browse files

Merge "more O_CLOEXEC"

am: 350f4832

Change-Id: I14c8a19b241e29ad4a9bd74d8ed904aa2ef7e5b3
parents 6130adad 350f4832
Loading
Loading
Loading
Loading
+3 −3
Original line number Diff line number Diff line
@@ -418,13 +418,13 @@ bool FileDescriptorInfo::GetSocketName(const int fd, std::string* result) {
}

void FileDescriptorInfo::DetachSocket(fail_fn_t fail_fn) const {
  const int dev_null_fd = open("/dev/null", O_RDWR);
  const int dev_null_fd = open("/dev/null", O_RDWR | O_CLOEXEC);
  if (dev_null_fd < 0) {
    fail_fn(std::string("Failed to open /dev/null: ").append(strerror(errno)));
  }

  if (dup2(dev_null_fd, fd) == -1) {
    fail_fn(android::base::StringPrintf("Failed dup2 on socket descriptor %d: %s",
  if (dup3(dev_null_fd, fd, O_CLOEXEC) == -1) {
    fail_fn(android::base::StringPrintf("Failed dup3 on socket descriptor %d: %s",
                                        fd,
                                        strerror(errno)));
  }