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

updpatch: ncrack #3194

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

Conversation

XiePengHan
Copy link
Contributor

This is the link: nmap/ncrack#130
Resolve the issue detected during the compilation of opensshlib, where the configure script identifies that the zlib version on your system is outdated and contains acknowledged security vulnerabilities.

@r-value
Copy link
Collaborator

r-value commented Oct 31, 2023

We strongly suggest you to move on and try to fix some other types of errors, instead of just sticking to simple & naive config.guess issues, which may help us to get a wider sense about your tech skills and abilities.

@XiePengHan
Copy link
Contributor Author

We strongly suggest you to move on and try to fix some other types of errors, instead of just sticking to simple & naive config.guess issues, which may help us to get a wider sense about your tech skills and abilities.

Thank you for your suggestion. I'm currently in the process of learning and exploring some other questions.

@felixonmars
Copy link
Owner

the configure script identifies that the zlib version on your system is outdated and contains acknowledged security vulnerabilities

Any idea why this is the case? We provide zlib 1.3 which is sufficiently up-to-date according to upstream configure.ac (which asks for 1.2.3+).

@XiePengHan
Copy link
Contributor Author

the configure script identifies that the zlib version on your system is outdated and contains acknowledged security vulnerabilities

Any idea why this is the case? We provide zlib 1.3 which is sufficiently up-to-date according to upstream configure.ac (which asks for 1.2.3+).

Thank you for your patient explanation. I am deeply grateful for it. I didn't delve into the reasons after resolving the issue, and I will be more mindful of this in my future contributions.

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

3 participants