Stop caring about order of keys in user_data

Author: egypt

lib/msf/core/module.rb

@@ -57,6 +57,10 @@ class Module
   # datastore, consumed by #replicant to allow clean override of MSF module methods.
   REPLICANT_EXTENSION_DS_KEY = 'ReplicantExtensions'
 
+  # The set of keys in {#user_data} that make {#user_data_is_match?} return
+  # true
+  MATCH_KEYS = Set.new([ :match, :match_set, :run ])
+
   # Make include public so we can runtime extend
   public_class_method :include
 
@@ -278,12 +282,12 @@ def fail_with(reason, msg=nil)
     raise RuntimeError, "#{reason.to_s}: #{msg}"
   end
 
-  # Whether {user_data} contains everything necessary to make a
+  # Whether {#user_data} contains everything necessary to make a
   # `MetasploitDataModels::AutomaticExploitation::MatchResult`
   #
   # @return [bool]
   def user_data_is_match?
-    user_data.kind_of?(Hash) && user_data.keys == [ :match, :match_set, :run ]
+    user_data.kind_of?(Hash) && Set.new(user_data.keys).superset?(MATCH_KEYS)
   end
 
   ##
@@ -329,6 +333,8 @@ def self.cached?
   # {Msf::Simple::Exploit#exploit_simple} or
   # {Msf::Simple::Auxiliary#run_simple} for correlating where modules came
   # from.
+  #
+  # @see #user_data_is_match?
   attr_accessor :user_data
 
   protected