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

common: Don't rely on issetugid() when it is broken #167

Merged
merged 1 commit into from
May 30, 2018

Conversation

ueno
Copy link
Member

@ueno ueno commented May 30, 2018

On macOS and FreeBSD, issetugid() has a different semantics from the original OpenBSD implementation and cannot reliably detect if the process made setuid/setgid:
https://gist.github.com/nicowilliams/4daf74a3a0c86848d3cbd9d0cdb5e26e

This should fix:
https://bugs.freedesktop.org/show_bug.cgi?id=67451
https://bugs.freedesktop.org/show_bug.cgi?id=100287

@ueno ueno force-pushed the wip/dueno/issetugid branch 2 times, most recently from bc577a0 to 88795d9 Compare May 30, 2018 09:23
@ueno ueno added the bug label May 30, 2018
@ueno ueno added this to the 0.23.12 milestone May 30, 2018
@ueno ueno mentioned this pull request May 30, 2018
On macOS and FreeBSD, issetugid() has different semantics from the
original OpenBSD implementation and cannot reliably detect if the
process made setuid/setgid:
https://gist.github.com/nicowilliams/4daf74a3a0c86848d3cbd9d0cdb5e26e

This should fix:
https://bugs.freedesktop.org/show_bug.cgi?id=67451
https://bugs.freedesktop.org/show_bug.cgi?id=100287
@ueno ueno force-pushed the wip/dueno/issetugid branch from 88795d9 to 82df122 Compare May 30, 2018 13:01
@ueno ueno merged commit 71b62aa into p11-glue:master May 30, 2018
@ueno ueno deleted the wip/dueno/issetugid branch June 25, 2018 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant