fix: prevent SU2_PY runner stderr deadlock and improve error handling by shbhmexe · Pull Request #2653

fix: prevent SU2_PY runner stderr deadlock and improve error handling by shbhmexe · Pull Request #2653 · su2code/SU2

added 2 commits

December 22, 2025 14:38

… copies

- SU2_PY: use subprocess.communicate() to reliably drain stderr while keeping stdout streamed,
  avoiding potential hangs when stderr output is large.
- SU2_PY: fix incorrect path reporting in error messages (abspath(".") instead of abspath(",")).
- C++ entrypoints: guard/limit copies into fixed-size config filename buffers to prevent
  potential overflow on long paths.

Signed-off-by: shbhmexe <shubhushukla586@gmail.com>

Signed-off-by: shbhmexe <shubhushukla586@gmail.com>

Keeping only Python fixes:
- stderr deadlock prevention via subprocess.communicate()
- path reporting bug fix (abspath('.') not abspath(','))

Reverted strcpy  strncpy changes from SU2_CFD, SU2_DEF, SU2_DOT,
SU2_GEO, and SU2_SOL per reviewer's earlier feedback to keep it simple.

Signed-off-by: shbhmexe <shubhushukla586@gmail.com>