Skip to content

Add missing require statements#7

Open
zeroSteiner wants to merge 1 commit into
rapid7:masterfrom
zeroSteiner:fix/add-missing-requries
Open

Add missing require statements#7
zeroSteiner wants to merge 1 commit into
rapid7:masterfrom
zeroSteiner:fix/add-missing-requries

Conversation

@zeroSteiner

Copy link
Copy Markdown
Contributor

This adds missing require statements so once the WindowsError module is loaded, the Win32, NTStatus and HResult modules are all accessible without needing to be required individually.

Trying to fix instances like this in Metasploit Framework where WindowsError is loaded but the others are not.

[-] 192.168.159.10:445 - Auxiliary failed: NameError uninitialized constant WindowsError::HResult
[-] 192.168.159.10:445 - Call stack:
[-] 192.168.159.10:445 -   /home/smcintyre/Repositories/metasploit-framework/lib/msf/core/exploit/remote/ms_icpr.rb:187:in `do_request_cert'
[-] 192.168.159.10:445 -   /home/smcintyre/Repositories/metasploit-framework/lib/msf/core/exploit/remote/ms_icpr.rb:100:in `request_certificate'
[-] 192.168.159.10:445 -   /home/smcintyre/Repositories/metasploit-framework/modules/auxiliary/admin/dcerpc/icpr_cert.rb:68:in `block in action_request_cert'
[-] 192.168.159.10:445 -   /home/smcintyre/Repositories/metasploit-framework/modules/auxiliary/admin/dcerpc/icpr_cert.rb:83:in `with_ipc_tree'
[-] 192.168.159.10:445 -   /home/smcintyre/Repositories/metasploit-framework/modules/auxiliary/admin/dcerpc/icpr_cert.rb:67:in `action_request_cert'
[-] 192.168.159.10:445 -   /home/smcintyre/Repositories/metasploit-framework/modules/auxiliary/admin/dcerpc/icpr_cert.rb:53:in `run'

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.

1 participant