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

polkit uses signed int for uid_t, while linux uses unsigned #191

Open
polkit-github-migration-bot opened this issue Mar 1, 2023 · 1 comment

Comments

@polkit-github-migration-bot
Copy link
Collaborator

In gitlab.freedesktop.org by jrybar on Mar 1, 2023, 12:34

Link to the original issue: https://gitlab.freedesktop.org/polkit/polkit/-/issues/190
Despite posix which doesn't state whether uid_t should be signed,
usual linux configuration uses __U32 (grep -rn "__UID_T_TYPE" /usr/include/bits/).

Proposed fix:
Use preprocessor branch detecting the platform and change definition in struct PolkitUnixProcess accordingly, along with rest of associated code.
This will preserve multiplatformness and also allow high UIDs on linux (where it seems to be more and more required and used).

@polkit-github-migration-bot
Copy link
Collaborator Author

In gitlab.freedesktop.org by jrybar on Mar 1, 2023, 12:36

Associated: !22

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

No branches or pull requests

1 participant