Commit eef66dee authored by Max Kellermann's avatar Max Kellermann

system/FileDescriptor: pass FileDescriptor to CheckDuplicate()

parent 7ad440ca
...@@ -205,9 +205,9 @@ FileDescriptor::DisableCloseOnExec() noexcept ...@@ -205,9 +205,9 @@ FileDescriptor::DisableCloseOnExec() noexcept
} }
bool bool
FileDescriptor::CheckDuplicate(int new_fd) noexcept FileDescriptor::CheckDuplicate(FileDescriptor new_fd) noexcept
{ {
if (fd == new_fd) { if (*this == new_fd) {
DisableCloseOnExec(); DisableCloseOnExec();
return true; return true;
} else } else
......
...@@ -168,8 +168,8 @@ public: ...@@ -168,8 +168,8 @@ public:
/** /**
* Duplicate the file descriptor onto the given file descriptor. * Duplicate the file descriptor onto the given file descriptor.
*/ */
bool Duplicate(int new_fd) const noexcept { bool Duplicate(FileDescriptor new_fd) const noexcept {
return ::dup2(Get(), new_fd) == 0; return ::dup2(Get(), new_fd.Get()) == 0;
} }
/** /**
...@@ -178,7 +178,7 @@ public: ...@@ -178,7 +178,7 @@ public:
* this method to inject file descriptors into a new child * this method to inject file descriptors into a new child
* process, to be used by a newly executed program. * process, to be used by a newly executed program.
*/ */
bool CheckDuplicate(int new_fd) noexcept; bool CheckDuplicate(FileDescriptor new_fd) noexcept;
#endif #endif
#ifdef __linux__ #ifdef __linux__
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment