ssh: Make 'send-files' more robust.
authorLudovic Courtès <ludo@gnu.org>
Sat, 24 Nov 2018 18:38:55 +0000 (19:38 +0100)
committerLudovic Courtès <ludo@gnu.org>
Sat, 24 Nov 2018 18:52:16 +0000 (19:52 +0100)
Possibly fixes <https://bugs.gnu.org/33239>.

* guix/ssh.scm (send-files): Call 'channel-get-exit-status' only when
RESULT is true.

guix/ssh.scm

index 25ec829..104f4f5 100644 (file)
@@ -297,9 +297,11 @@ Return the list of store items actually sent."
     (channel-send-eof port)
 
     ;; Wait for completion of the remote process and read the status sexp from
-    ;; PORT.
+    ;; PORT.  Wait for the exit status only when 'read' completed; otherwise,
+    ;; we might wait forever if the other end is stuck.
     (let* ((result (false-if-exception (read port)))
-           (status (zero? (channel-get-exit-status port))))
+           (status (and result
+                        (zero? (channel-get-exit-status port)))))
       (close-port port)
       (match result
         (('success . _)