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

Ms17 010 error handling #121

Merged
merged 4 commits into from
Nov 26, 2023
Merged

Ms17 010 error handling #121

merged 4 commits into from
Nov 26, 2023

Conversation

Marshall-Hallenbeck
Copy link
Collaborator

Catch ValueError when SMB header response is not the proper size (indicates not vulnerable) and catch ConnectionResetError in same location so the user isn't spammed with stacktraces

@Marshall-Hallenbeck Marshall-Hallenbeck added the bug-fix This Pull Request fixes a bug label Nov 13, 2023
@NeffIsBack NeffIsBack added the reviewed code Label for when a static code review was done label Nov 13, 2023
@mpgn mpgn added this to the v1.2.0 milestone Nov 25, 2023
@Marshall-Hallenbeck Marshall-Hallenbeck merged commit a6be6c1 into main Nov 26, 2023
2 checks passed
@Marshall-Hallenbeck Marshall-Hallenbeck deleted the ms17-010-error-handling branch November 26, 2023 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix This Pull Request fixes a bug reviewed code Label for when a static code review was done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants