use synchronous calls to open, read and close (as the async calls were not being waited on, so moving to the sync implmentations of these avoids that problem), thanks @cdelafuente-r7 :)

Author: sfewer-r7

modules/auxiliary/gather/progress_moveit_sftp_fileread_cve_2024_5806.rb

@@ -160,11 +160,11 @@ def run
   end
 
   def read_file(sftp, file_path)
-    sftp.open(file_path) do |open_response|
+    sftp.open!(file_path) do |open_response|
       if open_response.ok?
         file_size = sftp.fstat!(open_response[:handle]).size
 
-        sftp.read(open_response[:handle], 0, file_size) do |read_response|
+        sftp.read!(open_response[:handle], 0, file_size) do |read_response|
           if read_response.ok?
 
             file_data = read_response[:data].to_s
@@ -192,7 +192,7 @@ def read_file(sftp, file_path)
         print_error('SFTP open failed. Is the TARGETFILE path correct?')
       end
     ensure
-      sftp.close(open_response[:handle]) if open_response.ok?
+      sftp.close!(open_response[:handle]) if open_response.ok?
     end
   end