Extra condition that's not actually needed
Don't actually need to check nil res, because no code will actually try to access res when it's nil anyway. And the 'return' at the of the function will catch it when the response times out.unstable
parent
cb95a7b520
commit
64b8696e3c
|
@ -80,8 +80,6 @@ class Metasploit3 < Msf::Exploit::Remote
|
|||
'uri' => '/'
|
||||
})
|
||||
|
||||
return Exploit::CheckCode::Unknown if res.nil?
|
||||
|
||||
if res and res.code == 200 and res.body =~ /<strong>Distro Release: <\/strong>Openfiler [NE]SA 2\./
|
||||
return Exploit::CheckCode::Appears
|
||||
elsif res and res.code == 200 and res.body =~ /<title>Openfiler Storage Control Center<\/title>/
|
||||
|
|
Loading…
Reference in New Issue