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

Band-aid for MySQL (Probably a part of #9) #57

Merged
merged 2 commits into from
Jun 2, 2019

Conversation

chris-pcguy
Copy link
Contributor

The printf returns "mysql_native_password" with three 0xff-bytes at the end.

You might want to put that band-aid at the "packet_number == 2"-condition too, but having it at the "packet_number == 0"-condition seems to be enough to avoid a crash.

Tested on Ubuntu 19.04 with a local MySQL server:
ncrack --user root --pass root mysql://127.0.0.0/31

Signed-off-by: Christian Inci chris.gh@broke-the-inter.net

The printf returns "mysql_native_password" with three 0xff-bytes at the end.

You might want to put that band-aid at the "packet_number == 2"-condition too, but having it at the "packet_number == 0"-condition seems to be enough to avoid a crash.

Tested on Ubuntu 19.04 with a local MySQL server:
ncrack --user root --pass root mysql://127.0.0.0/31

Signed-off-by: Christian Inci <chris.gh@broke-the-inter.net>
@chris-pcguy
Copy link
Contributor Author

It seems to crash somewhere else too (with some servers), I'll look into it.

The "packet_number == 2"-condition is also affected.

The (added) printf returns "caching_sha2_password" with three 0xff-bytes at the end.

Tested with MySQL 8. It crashed while having a single target selected too. (mysql://x.x.x.x/32 or just mysql://x.x.x.x/)

Signed-off-by: Christian Inci <chris.gh@broke-the-inter.net>
@chris-pcguy
Copy link
Contributor Author

It were actually needed at the "packet_number == 2"-condition too. (MySQL 8, caching_sha2_password)

@ithilgore ithilgore merged commit 2047c2c into nmap:master Jun 2, 2019
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