call sftp.close in an ensure block in case seomthing throws an excpetion. we probably dont *have* to do this (as teh SFTP session will be torn down either way), but it seems like best practise *to* so this.

Author: sfewer-r7

modules/auxiliary/gather/progress_moveit_sftp_fileread_cve_2024_5806.rb

@@ -188,11 +188,11 @@ def read_file(sftp, file_path)
             print_error('SFTP read failed.')
           end
         end
-
-        sftp.close(open_response[:handle])
       else
         print_error('SFTP open failed. Is the TARGETFILE path correct?')
       end
+    ensure
+      sftp.close(open_response[:handle]) if open_response.ok?
     end
   end