(polaris) branch main updated: fix(verify-release): fix bash pipe FD leak causing deadlock on maven checksum verification (#4024)

[ad...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

adnan pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/main by this push:
     new 52e4c56af2 fix(verify-release): fix bash pipe FD leak causing deadlock on maven checksum verification (#4024)
52e4c56af2 is described below

commit 52e4c56af2e10e75491ec6c8c5e2aec28a6e6ff8
Author: Adnan Hemani <ad...@berkeley.edu>
AuthorDate: Wed Apr 1 21:41:34 2026 -0700

    fix(verify-release): fix bash pipe FD leak causing deadlock on maven checksum verification (#4024)
    
    * fix(verify-release): fix bash pipe FD leak causing deadlock on maven checksum verification
    
    * Review from @dimas-b
---
 tools/verify-release/verify-release.sh | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/tools/verify-release/verify-release.sh b/tools/verify-release/verify-release.sh
index 19420bf54f..3ea7846b34 100755
--- a/tools/verify-release/verify-release.sh
+++ b/tools/verify-release/verify-release.sh
@@ -219,14 +219,21 @@ function log_info {
 #   Following arguments: process arguments
 function proc_exec {
   local err_msg
-  local output
+  local output_file
+  local exit_code
   err_msg=$1
   shift
-  output=()
-  IFS=$'\n' read -r -d '' -a output < <( "${@}" 2>&1 && printf '\0' ) || (
-    log_fatal "${err_msg}" "${output[@]}"
+  output_file=$(mktemp)
+  "${@}" > "${output_file}" 2>&1
+  exit_code=$?
+  if [[ $exit_code -ne 0 ]]; then
+    local output
+    output=$(cat "${output_file}")
+    rm -f "${output_file}"
+    log_fatal "${err_msg}" "${output}"
     return 1
-  )
+  fi
+  rm -f "${output_file}"
 }
 
 function mirror {