Git
/
ssh-audit
mirror of https://github.com/jtesta/ssh-audit.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Suppressed more unused variables warnings.

This commit is contained in:
Joe Testa 2017-09-27 22:22:42 -04:00
parent a3c6d16500
commit b7bf8ab38a
1 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
ssh-audit.py
View File

@ -566,7 +566,8 @@ class SSH2(object): # pylint: disable=too-few-public-methods
# If the connection is closed, re-open it and get the kex again. # If the connection is closed, re-open it and get the kex again.
if not s.is_connected(): if not s.is_connected():
s.connect() s.connect()
unused1, unused2, err = s.get_banner() unused = None # pylint: disable=unused-variable
unused, unused, err = s.get_banner()
if err is not None: if err is not None:
s.close() s.close()
return return
@ -648,12 +649,14 @@ class SSH2(object): # pylint: disable=too-few-public-methods
return return
s.connect() s.connect()
unused = None # pylint: disable=unused-variable
unused, unused, err = s.get_banner() unused, unused, err = s.get_banner()
if err is not None: if err is not None:
s.close() s.close()
return False return False
# Parse the server's initial KEX. # Parse the server's initial KEX.
packet_type = 0 # pylint: disable=unused-variable
packet_type, payload = s.read_packet(2) packet_type, payload = s.read_packet(2)
kex = SSH2.Kex.parse(payload) kex = SSH2.Kex.parse(payload)
@ -700,7 +703,7 @@ class SSH2(object): # pylint: disable=too-few-public-methods
# larger than the requested max. So just because we # larger than the requested max. So just because we
# got here, doesn't mean the server is vulnerable... # got here, doesn't mean the server is vulnerable...
smallest_modulus = kex_group.get_dh_modulus_size() smallest_modulus = kex_group.get_dh_modulus_size()
except Exception as e: # pylint: disable=bare-except except Exception: # pylint: disable=bare-except
pass pass
finally: finally:
s.close() s.close()
@ -722,7 +725,7 @@ class SSH2(object): # pylint: disable=too-few-public-methods
kex_group.send_init_gex(s, bits, bits, bits) kex_group.send_init_gex(s, bits, bits, bits)
kex_group.recv_reply(s) kex_group.recv_reply(s)
smallest_modulus = kex_group.get_dh_modulus_size() smallest_modulus = kex_group.get_dh_modulus_size()
except Exception as e: # pylint: disable=bare-except except Exception: # pylint: disable=bare-except
pass pass
finally: finally:
# The server is in a state that is not re-testable, # The server is in a state that is not re-testable,
@ -2147,7 +2150,7 @@ class KexDH(object): # pragma: nocover
key_id, key_id_len, ptr = KexDH.__get_bytes(hostkey, ptr) key_id, key_id_len, ptr = KexDH.__get_bytes(hostkey, ptr)
# The principles, which are... I don't know what. # The principles, which are... I don't know what.
principles, prinicples_len, ptr = KexDH.__get_bytes(hostkey, ptr) principles, princicples_len, ptr = KexDH.__get_bytes(hostkey, ptr)
# The timestamp that this certificate is valid after. # The timestamp that this certificate is valid after.
valid_after = hostkey[ptr:ptr + 8] valid_after = hostkey[ptr:ptr + 8]