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

Update enip-info.nse #1212

Closed
wants to merge 2 commits into from
Closed

Update enip-info.nse #1212

wants to merge 2 commits into from

Conversation

NothinRandom
Copy link

Added/updated more vendor codes from ODVA list

Added/updated more vendor codes from ODVA list
them commas...
@cldrn cldrn self-assigned this May 18, 2018
@cldrn cldrn added the NSE label May 18, 2018
@cldrn
Copy link
Member

cldrn commented May 18, 2018

Thanks!

@cldrn
Copy link
Member

cldrn commented May 28, 2018

Would you mind sharing the URL from ODVA where you got this list? It would be useful as future reference when updating the script again. I noticed the wireshark dissector url we have in the comments still has the old list.

@NothinRandom
Copy link
Author

NothinRandom commented May 29, 2018

At first I used the ODVA website to search one by one... that obviously took too long since I was already at 20 members past the old list and still going strong. I decided to contact ODVA directly and they were kind enough to generate a list vid.txt that contains all vendors for ControlNet, DeviceNet, and EtherNet/IP.

@cldrn
Copy link
Member

cldrn commented May 29, 2018 via email

@NothinRandom
Copy link
Author

Yes. All new members have been updated. Since ODVA doesn't remove the vendor after subscription, "Reserved" means that there used to be someone there, but no longer is subscribed to ODVA (pay a fee). If they decide to renew one day, then the list is refreshed.

@nmap-bot nmap-bot closed this in d077f1e Jun 10, 2018
@NothinRandom NothinRandom deleted the patch-2 branch September 5, 2018 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants