new NSE script broadcast-sonicwall-discover#165
Closed
pfuender wants to merge 1 commit intonmap:masterfrom
Closed
Conversation
- scans the local network for available Sonicwall firewalls
|
👍 |
Author
|
Any chance on getting this merged, or are there modifications needed from my side in order to do so? Thanks! |
|
@pfuender I'm sorry it has taken so long to get back on this. I've made a few changes to the way data is unpacked and reported, but since I don't have a sonicwall available for testing, I'd appreciate your feedback. A little back-and-forth and I'm confident we can get this included in Nmap. I've uploaded my changed version here: https://gist.github.com/dmiller-nmap/cf4f21c99300e615ce90 |
Author
|
@dmiller-nmap No worries! Thanks for the proposal, I quickly fixed the lines for Rom and Firmware version, as well as for the modified output. So far everything looks fine. |
qha
pushed a commit
to qha/nmap
that referenced
this pull request
Dec 16, 2015
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi Guys,
I've got a new script for you! It discovers a locally attached Sonicwall-firewall and prints some details about it. Find an example output right here:
Let me know if you should have any questions - and else, thanks for merging! ;-)
Cheers
Pfuender