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

Remove unused <netinet/in_systm.h> and <resolv.h> inclusions. #531

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

Conversation

sortie
Copy link
Contributor

@sortie sortie commented Nov 2, 2024

netinet/in_systm.h defines n_short, n_long, and n_time which are not used anywhere in openssh.

resolv.h is not used in the affected files.

netinet/in_systm.h defines n_short, n_long, and n_time which are not
used anywhere in openssh.

resolv.h is not used in the affected files.
@djmdjm
Copy link
Contributor

djmdjm commented Dec 3, 2024

resolv.h is needed for base64 functions on some platforms

hostfile.c:97:8: error: call to undeclared function '__b64_pton'; ISO C99 and later do not support implicit function declarations [-Werror,-Wimplicit-function-declaration]
        ret = __b64_pton(b64salt, salt, salt_len);

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.

2 participants