-
Notifications
You must be signed in to change notification settings - Fork 241
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
lib/shadow.c is 0xDEADBEEF #1228
Comments
alejandro-colomar
added a commit
to alejandro-colomar/shadow
that referenced
this issue
Feb 24, 2025
We didn't even have prototypes for these APIs since long ago, when the prototypes were removed, but misteriously the implementations remained. Both glibc and musl provide getspnam(3), so this file was effectively being ignored by the compiler. Just remove it. Also remove the check for getspnam, which isn't used elsewhere. Fixes: 0ee095a (2007-10-07; "[svn-upgrade] Integrating new upstream version, shadow (4.0.7)") Closes: <shadow-maint#1228> Signed-off-by: Alejandro Colomar <[email protected]>
getusershell also seems documented for both glibc and musl. |
alejandro-colomar
added a commit
to alejandro-colomar/shadow
that referenced
this issue
Feb 24, 2025
We didn't even have prototypes for these APIs since long ago, when the prototypes were removed, but misteriously the implementations remained. Both glibc and musl provide getspnam(3), so this file was effectively being ignored by the compiler. Just remove it. Also remove the check for getspnam, which isn't used elsewhere. Fixes: 0ee095a (2007-10-07; "[svn-upgrade] Integrating new upstream version, shadow (4.0.7)") Closes: <shadow-maint#1228> Signed-off-by: Alejandro Colomar <[email protected]>
alejandro-colomar
added a commit
to alejandro-colomar/shadow
that referenced
this issue
Mar 3, 2025
We didn't even have prototypes for these APIs since long ago, when the prototypes were removed, but misteriously the implementations remained. Both glibc and musl provide getspnam(3), so this file was effectively being ignored by the compiler. Just remove it. Also remove the check for getspnam, which isn't used elsewhere. Fixes: 0ee095a (2007-10-07; "[svn-upgrade] Integrating new upstream version, shadow (4.0.7)") Closes: <shadow-maint#1228> Signed-off-by: Alejandro Colomar <[email protected]>
alejandro-colomar
added a commit
to alejandro-colomar/shadow
that referenced
this issue
Mar 3, 2025
We didn't even have prototypes for these APIs since long ago, when the prototypes were removed, but misteriously the implementations remained. Both glibc and musl provide getspnam(3), so this file was effectively being ignored by the compiler. Just remove it. Also remove the check for getspnam, which isn't used elsewhere. Fixes: 0ee095a (2007-10-07; "[svn-upgrade] Integrating new upstream version, shadow (4.0.7)") Closes: <shadow-maint#1228> Signed-off-by: Alejandro Colomar <[email protected]>
alejandro-colomar
added a commit
to alejandro-colomar/shadow
that referenced
this issue
Mar 3, 2025
We didn't even have prototypes for these APIs since long ago, when the prototypes were removed, but misteriously the implementations remained. Both glibc and musl provide getspnam(3), so this file was effectively being ignored by the compiler. Just remove it. Also remove the check for getspnam, which isn't used elsewhere. Fixes: 0ee095a (2007-10-07; "[svn-upgrade] Integrating new upstream version, shadow (4.0.7)") Closes: <shadow-maint#1228> Signed-off-by: Alejandro Colomar <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
shadow/lib/shadow.c
Line 13 in d3fa0ba
musl libc has getspnam(3), so this very much looks like a dead file.
I'm grinding the axe. :)
The text was updated successfully, but these errors were encountered: