Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FreeBSD Support - Fix syscall(SYS__pthread_chdir) failed to set current directory #141

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fronald
Copy link

@fronald fronald commented Jun 10, 2022

Hello guys!

I am a developer and co-founder of Wiselabs Software in Brazil. We have some mission-critical systems that we chose to migrate to FreeBSD. In some of these systems we use NuProcess, and in the quality assurance process we found the following problem:

"syscall(SYS__pthread_chdir) failed to set current directory"

We also found a bug with the order in the process registration call (Kevent).

We chose to contribute to the project and resolve this bug. I hope it will be useful to you and other users.

Thank you so much for this wonderful library.

@fronald
Copy link
Author

fronald commented Jun 10, 2022

We apologize for not having noticed that there was already another PR by jasonkafer. Feel free to decline ours.
Or if you prefer, we can make the changes you need.

Thanks.

@bturner
Copy link
Collaborator

bturner commented May 5, 2024

Sorry for the very long delay in responding to this pull request. As was explained (by a helpful developer) on pull request #109, the change here sets the syscall to chdir, which changes the working directory for the entire process, not pthread_chdir, which changes it for the current thread. That aspect of this change won't be accepted, because using chdir has a number of issues (see this comment for details).

Adding ext to kevent may be compatible with the FreeBSD variant you're using, but is not compatible with macOS; ext is part of a separate kevent64_s struct instead (per man kqueue). We'd need some other way to handle that field to avoid memory offset issues on macOS.

I'm interested in the "bug with the order in the process registration call". Can you link me to some documentation or other code that explains why the existing order is wrong, and why the new order proposed here is correct?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants