Skip to content

Commit

Permalink
Continue with EBADF in AutoRemoteSyscalls::retrieve_fd_arch too.
Browse files Browse the repository at this point in the history
bin/rr --fatal-errors --check-cached-mmaps record bin/pid_ns_reap

[FATAL src/AutoRemoteSyscalls.cc:659:retrieve_fd_arch() errno: EBADF]
 (task 693273 (rec:693273) at time 308)
 -> Assertion `errno == ENOSYS' failed to hold. Failed in pidfd_getfd errno=EBADF

Is there a reason not to continue here?
Seems to just show up with "--fatal-errors --check-cached-mmaps".
  • Loading branch information
bernhardu committed Apr 1, 2023
1 parent 7936141 commit b2e9c5e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/AutoRemoteSyscalls.cc
Original file line number Diff line number Diff line change
Expand Up @@ -655,7 +655,7 @@ template <typename Arch> ScopedFd AutoRemoteSyscalls::retrieve_fd_arch(int fd) {
if (ret.is_open()) {
return ret;
}
ASSERT(t, errno == ENOSYS) << "Failed in pidfd_getfd errno=" << errno_name(errno);
ASSERT(t, errno == ENOSYS || errno == EBADF) << "Failed in pidfd_getfd errno=" << errno_name(errno);
}

// Clear out any pending message in the socket.
Expand Down

0 comments on commit b2e9c5e

Please sign in to comment.