Skip to content

Fixed compiler warnings#2

Merged
rbsec merged 1 commit intorbsec:masterfrom
digineo:warnings_fix
Dec 1, 2013
Merged

Fixed compiler warnings#2
rbsec merged 1 commit intorbsec:masterfrom
digineo:warnings_fix

Conversation

@corny
Copy link
Copy Markdown
Contributor

@corny corny commented Dec 1, 2013

I removed the unused variable status and added the const keyword before SSL_METHOD.

rbsec added a commit that referenced this pull request Dec 1, 2013
@rbsec rbsec merged commit 9b02d97 into rbsec:master Dec 1, 2013
@rbsec
Copy link
Copy Markdown
Owner

rbsec commented Dec 1, 2013

Merged, thanks. This fork started from just wanting some saner default options and better output, so I've not really spent any time on cleaning up the code that was working (hence the warnings).

Nice to see someone else actually using the fork.

@corny corny deleted the warnings_fix branch December 2, 2013 00:02
@corny
Copy link
Copy Markdown
Contributor Author

corny commented Dec 2, 2013

Hi rbsec, thanks for your work. I am trying to integrate sslscan into my service at http://ssl-tools.net/ .
Support for IPv6 would be nice - maybe we find someone who could add it.

@rbsec
Copy link
Copy Markdown
Owner

rbsec commented Dec 2, 2013

I created an issue for IPv6 support (#3).

@corny
Copy link
Copy Markdown
Contributor Author

corny commented Dec 2, 2013

Great, I will take a look at this.

janiko71 referenced this pull request in janiko71/sslscan Jul 8, 2016
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