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 | f1b81c42c9a6e53e3ce248e40ccc33a5b3ba51bc (patch) | |
| tree | 82e5b0ff0be8f61690e68a7db99f3e6a836278e6 /lib/custodian/protocol-tests/ftp.rb | |
| parent | f8fde5b7bba478a29feb2cb7fad562948ca80c46 (diff) | |
Updated to ensure we read a banner from the remote server - before we test against it.
Diffstat (limited to 'lib/custodian/protocol-tests/ftp.rb')
| -rwxr-xr-x | lib/custodian/protocol-tests/ftp.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/custodian/protocol-tests/ftp.rb b/lib/custodian/protocol-tests/ftp.rb index 2d0700a..1ecd68a 100755 --- a/lib/custodian/protocol-tests/ftp.rb +++ b/lib/custodian/protocol-tests/ftp.rb @@ -81,11 +81,11 @@ class FTPTest socket.puts( "QUIT") banner = socket.gets(nil) - banner = banner[0,20] + banner = banner[0,20] unless( banner.nil? ) socket.close() - if ( banner =~ /^220/ ) + if ( ( !banner.nil? ) && ( banner =~ /^220/ ) ) return true else @error = "Banner didn't report OK: #{banner}" |