cli: ensure child processes are wait()ed on I/O error

cli/src/git_util.rs

@@ -94,30 +94,22 @@ fn pinentry_get_pw(url: &str) -> Option<String> {
         .stdout(Stdio::piped())
         .spawn()
         .ok()?;
+    let mut interact = || -> std::io::Result<_> {
         #[rustfmt::skip]
-    pinentry
+        let req = format!(
-        .stdin
-        .take()
-        .unwrap()
-        .write_all(
-            format!(
             "SETTITLE jj passphrase\n\
              SETDESC Enter passphrase for {url}\n\
              SETPROMPT Passphrase:\n\
              GETPIN\n"
-            )
+        );
-            .as_bytes(),
+        pinentry.stdin.take().unwrap().write_all(req.as_bytes())?;
-        )
-        .ok()?;
         let mut out = String::new();
-    pinentry
+        pinentry.stdout.take().unwrap().read_to_string(&mut out)?;
-        .stdout
+        Ok(out)
-        .take()
+    };
-        .unwrap()
+    let maybe_out = interact();
-        .read_to_string(&mut out)
-        .ok()?;
     _ = pinentry.wait();
-    for line in out.split('\n') {
+    for line in maybe_out.ok()?.split('\n') {
         if !line.starts_with("D ") {
             continue;
         }

cli/src/merge_tools/external.rs

@@ -552,7 +552,7 @@ pub fn generate_diff(
             tool_binary: tool.program.clone(),
             source,
         })?;
-    io::copy(&mut child.stdout.take().unwrap(), writer).map_err(ExternalToolError::Io)?;
+    let copy_result = io::copy(&mut child.stdout.take().unwrap(), writer);
     // Non-zero exit code isn't an error. For example, the traditional diff command
     // will exit with 1 if inputs are different.
     let exit_status = child.wait().map_err(ExternalToolError::Io)?;
@@ -560,6 +560,7 @@ pub fn generate_diff(
     if !exit_status.success() {
         writeln!(ui.warning(), "{}", format_tool_aborted(&exit_status)).ok();
     }
+    copy_result.map_err(ExternalToolError::Io)?;
     Ok(())
 }