Handle the edge case where IPRange#=== is given another kind of object.

Author: postmodern

lib/ronin/support/network/ip_range/cidr.rb

@@ -264,8 +264,10 @@ def ===(other)
             case other
             when CIDR
               include?(other.first) && include?(other.last)
-            else
+            when Enumerable
               other.all? { |ip| include?(ip) }
+            else
+              false
             end
           end
 

lib/ronin/support/network/ip_range/glob.rb

@@ -264,7 +264,12 @@ def ==(other)
           # @since 1.1.0
           #
           def ===(other)
-            other.all? { |ip| include?(ip) }
+            case other
+            when Enumerable
+              other.all? { |ip| include?(ip) }
+            else
+              false
+            end
           end
 
           #

lib/ronin/support/network/ip_range/range.rb

@@ -221,8 +221,10 @@ def ===(other)
             case other
             when IPRange::Range
               self.begin <= other.begin && self.end >= other.end
-            else
+            when Enumerable
               other.all? { |ip| include?(ip) }
+            else
+              false
             end
           end
 

spec/network/ip_range/cidr_spec.rb

@@ -460,6 +460,14 @@
         end
       end
     end
+
+    context "when given another kind of Object" do
+      let(:other) { Object.new }
+
+      it "must return false" do
+        expect(subject === other).to be(false)
+      end
+    end
   end
 
   describe "#each" do

spec/network/ip_range/glob_spec.rb

@@ -682,6 +682,14 @@
         end
       end
     end
+
+    context "when given another kind of Object" do
+      let(:other) { Object.new }
+
+      it "must return false" do
+        expect(subject === other).to be(false)
+      end
+    end
   end
 
   describe "#each" do

spec/network/ip_range/range_spec.rb

@@ -353,6 +353,14 @@
         end
       end
     end
+
+    context "when given another kind of Object" do
+      let(:other) { Object.new }
+
+      it "must return false" do
+        expect(subject === other).to be(false)
+      end
+    end
   end
 
   describe "#size" do

spec/network/ip_range_spec.rb

@@ -502,6 +502,14 @@
           end
         end
       end
+
+      context "when given another kind of Object" do
+        let(:other) { Object.new }
+
+        it "must return false" do
+          expect(subject === other).to be(false)
+        end
+      end
     end
 
     context "when initialized with an IP glob range" do
@@ -568,6 +576,14 @@
           end
         end
       end
+
+      context "when given another kind of Object" do
+        let(:other) { Object.new }
+
+        it "must return false" do
+          expect(subject === other).to be(false)
+        end
+      end
     end
   end