Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable full backtrace for exception in process spawn #14796

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

straight-shoota
Copy link
Member

If an exception raises in the code that prepares a forked process for exec, the error message of this exception is written through a pipe to the original process, which then raises an exception for the calling code (Process.run).
The error only includes a message, no stack trace. So the only stack trace you get is that of the handler which reads the message from the pipe and raises in the original process.
But that's not very relevant. We want to know the location of the original exception.
This patch changes from sending just the exception message, to printing the entire backtrace (inspect_with_backtrace).

# process-run-exception.cr
Crystal::System::ORIGINAL_STDIN.close

Process.run("true")

Without this change:

$  crystal run process-run-exception.cr
Unhandled exception: Error executing process: 'true': Could not reopen file descriptor: Bad file descriptor (RuntimeError)
  from /home/linuxbrew/.linuxbrew/Cellar/crystal/1.12.2/share/crystal/src/crystal/system/unix/process.cr:221:9 in 'spawn'
  from /home/linuxbrew/.linuxbrew/Cellar/crystal/1.12.2/share/crystal/src/process.cr:280:5 in 'initialize'
  from /home/linuxbrew/.linuxbrew/Cellar/crystal/1.12.2/share/crystal/src/process.cr:272:3 in 'new'
  from /home/linuxbrew/.linuxbrew/Cellar/crystal/1.12.2/share/crystal/src/process.cr:178:14 in 'run'
  from .test/process-run-exception.cr:3:1 in '__crystal_main'
  from /home/linuxbrew/.linuxbrew/Cellar/crystal/1.12.2/share/crystal/src/crystal/main.cr:129:5 in 'main_user_code'
  from /home/linuxbrew/.linuxbrew/Cellar/crystal/1.12.2/share/crystal/src/crystal/main.cr:115:7 in 'main'
  from /home/linuxbrew/.linuxbrew/Cellar/crystal/1.12.2/share/crystal/src/crystal/main.cr:141:3 in 'main'
  from /lib/x86_64-linux-gnu/libc.so.6 in '??'
  from /lib/x86_64-linux-gnu/libc.so.6 in '__libc_start_main'
  from /home/johannes/.cache/crystal/crystal-run-process-run-exception.tmp in '_start'
  from ??

With this change:

$ bin/crystal run process-run-exception.cr
Unhandled exception: Error executing process: 'true': Could not reopen file descriptor: Bad file descriptor (IO::Error)
  from src/crystal/system/unix/file_descriptor.cr:94:5 in 'system_reopen'
  from src/io/file_descriptor.cr:252:5 in 'reopen'
  from src/crystal/system/unix/process.cr:348:5 in 'reopen_io'
  from src/crystal/system/unix/process.cr:304:5 in 'try_replace'
  from src/crystal/system/unix/process.cr:241:9 in 'spawn'
  from src/process.cr:280:5 in 'initialize'
  from src/process.cr:272:3 in 'new'
  from src/process.cr:178:14 in 'run'
  from .test/process-run-exception.cr:3:1 in '__crystal_main'
  from src/crystal/main.cr:118:5 in 'main_user_code'
  from src/crystal/main.cr:104:7 in 'main'
  from src/crystal/main.cr:130:3 in 'main'
  from /lib/x86_64-linux-gnu/libc.so.6 in '??'
  from /lib/x86_64-linux-gnu/libc.so.6 in '__libc_start_main'
  from /home/johannes/.cache/crystal/crystal-run-process-run-exception.tmp in '_start'
  from ???
 (RuntimeError)
  from src/crystal/system/unix/process.cr:267:9 in 'spawn'
  from src/process.cr:280:5 in 'initialize'
  from src/process.cr:272:3 in 'new'
  from src/process.cr:178:14 in 'run'
  from .test/process-run-exception.cr:3:1 in '__crystal_main'
  from src/crystal/main.cr:118:5 in 'main_user_code'
  from src/crystal/main.cr:104:7 in 'main'
  from src/crystal/main.cr:130:3 in 'main'
  from /lib/x86_64-linux-gnu/libc.so.6 in '??'
  from /lib/x86_64-linux-gnu/libc.so.6 in '__libc_start_main'
  from /home/johannes/.cache/crystal/crystal-run-process-run-exception.tmp in '_start'
  from ???

@straight-shoota straight-shoota added this to the 1.14.0 milestone Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants