Ensure that hostnames used for ping-tests are valid - to avoid the security hole.

1 files changed, 20 insertions, 0 deletions

diff --git a/t/test-parser.rb b/t/test-parser.rb
index a037448..cb9a9cb 100755
--- a/t/test-parser.rb
+++ b/t/test-parser.rb
@@ -608,4 +608,24 @@ class TestParser < Test::Unit::TestCase
     end
   end
 
+
+  #
+  # Test the potential security-hole for ping-tests
+  #
+  def test_ping_security_hole
+
+
+    parser = MonitorConfig.new("/dev/null" )
+
+    assert_raise ArgumentError do
+      parser.parse_line( "$(/tmp/exploit) must ping ." )
+    end
+
+    assert_nothing_raised do
+      parser.parse_line( "test.example.vm.bytemark.co.uk must ping ." )
+    end
+
+  end
+
+
 end