Implement deregistering an option by alias

Rather than having to 'unregister' both 'RHOST' and 'RHOSTS' for modules using a network-connecting mixin, this allows just unregistering one option by exposing the alias key lookup to the module options library and allowing it to remove the datastore options for the module from validation.

Noted while testing #11108 that the module's unregister options were incomplete, and there were a few more like it. This allows modules operating in this style to work without modifications.

lib/msf/core/data_store.rb

@@ -306,8 +306,6 @@ def each(&block)
     list.each(&block)
   end
 
-protected
-
   #
   # Case-insensitive key lookup
   #

lib/msf/core/module/options.rb

@@ -29,8 +29,12 @@ def validate
   #
   def deregister_options(*names)
     names.each { |name|
-      self.options.remove_option(name)
+      real_name = self.datastore.find_key_case(name)
       self.datastore.delete(name)
+      self.options.remove_option(name)
+      if real_name != name
+        self.options.remove_option(real_name)
+      end
     }
   end
 
@@ -62,4 +66,4 @@ def register_options(options, owner = self.class)
     self.datastore.import_options(self.options, 'self', true)
     import_defaults(false)
   end
-end
+end