Msf::OptionContainer: Replace .sorted Array attribute with self.sort

Author: bcoles

lib/msf/core/option_container.rb

@@ -50,12 +50,18 @@ class OptionContainer < Hash
     # as necessary.
     #
     def initialize(opts = {})
-      self.sorted = []
       self.groups = {}
 
       add_options(opts)
     end
 
+    #
+    # Return the sorted array of options.
+    #
+    def sorted
+      self.sort
+    end
+
     #
     # Return the value associated with the supplied name.
     #
@@ -116,10 +122,6 @@ def has_evasion_options?
     # @param [String] name the option name
     def remove_option(name)
       delete(name)
-      sorted.each_with_index { |e, idx|
-        sorted[idx] = nil if (e[0] == name)
-      }
-      sorted.delete(nil)
     end
 
     #
@@ -170,9 +172,6 @@ def add_option(option, name = nil, owner = nil, advanced = false, evasion = fals
       option.owner    = owner
 
       self.store(option.name, option)
-
-      # Re-calculate the sorted list
-      self.sorted = self.sort
     end
 
     #
@@ -330,18 +329,11 @@ def remove_group(group_name)
       groups.delete(group_name)
     end
 
-    #
-    # The sorted array of options.
-    #
-    attr_reader :sorted
-
     # @return [Hash<String, Msf::OptionGroup>]
     attr_reader :groups
 
     protected
 
-    attr_writer :sorted # :nodoc:
-
     attr_writer :groups
   end