windows/gather/enum_domains: Return early if no domains are found

[bcoles]

Fixes #15541.

Before:

msf6 post(windows/gather/enum_domains) > rexploit 
[*] Reloading module...

[-] ERROR_NO_BROWSER_SERVERS_FOUND
[-] Post failed: NoMethodError undefined method `each' for nil:NilClass
[-] Call stack:
[-]   /root/Desktop/metasploit-framework/modules/post/windows/gather/enum_domains.rb:27:in `run'
[*] Post module execution completed

After:

msf6 post(windows/gather/enum_domains) > rexploit 
[*] Reloading module...

[-] ERROR_NO_BROWSER_SERVERS_FOUND
[*] Post module execution completed

[3V3RYONE]

May I start by reviewing this PR? :)

[bcoles]

May I start by reviewing this PR? :)

You can if you like, but it fixes an issue reported by @adfoster-r7 and would require his input.

It also changes only a single line. The reasoning behind the change is described in #15541 (comment).

[3V3RYONE]

May I start by reviewing this PR? :)

You can if you like, but it fixes an issue reported by @adfoster-r7 and would require his input.

It also changes only a single line. The reasoning behind the change is described in #15541 (comment).

Yes! I checked that.. In that case, let adfoster-r7 review! I'll pick up another one! :)

[adfoster-r7]

Release Notes

Fixes a crash in post/windows/gather/enum_domains when no domains are found

[adfoster-r7]

Thanks! 🎉

[gwillcox-r7]

Adding in missing release note tags.