diff options
author | Steve Kemp <steve@steve.org.uk> | 2012-11-19 16:24:30 +0000 |
---|---|---|
committer | Steve Kemp <steve@steve.org.uk> | 2012-11-19 16:24:30 +0000 |
commit | 56c7a4e9b58bd7c944db1880a27ee435bbce5866 (patch) | |
tree | 82e5b0ff0be8f61690e68a7db99f3e6a836278e6 /lib/custodian/protocol-tests/smtp.rb | |
parent | 6066abd440c42333106e9394cb392b3dfafeac2d (diff) |
Updated to ensure we read a banner from the remote server - before we test against it.
Diffstat (limited to 'lib/custodian/protocol-tests/smtp.rb')
-rwxr-xr-x | lib/custodian/protocol-tests/smtp.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/custodian/protocol-tests/smtp.rb b/lib/custodian/protocol-tests/smtp.rb index 6a3895d..5da672f 100755 --- a/lib/custodian/protocol-tests/smtp.rb +++ b/lib/custodian/protocol-tests/smtp.rb @@ -81,11 +81,11 @@ class SMTPTest socket.puts( "QUIT") banner = socket.gets(nil) - banner = banner[0,40] + banner = banner[0,40] unless( banner.nil? ) socket.close() - if ( banner =~ /SMTP/i ) + if ( ( !banner.nil? ) && ( banner =~ /SMTP/i ) ) puts "SMTP alive: #{banner}" if ( @test_data['verbose'] ) return true else |