diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2021-04-02 13:52:56 +0200 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2021-05-05 13:59:23 +0200 |
| commit | 5dd55303f4d7e4294e0d2f1098232bd0fa305832 (patch) | |
| tree | 7f320181d572b2cdecd85afbed05e766e875b765 /man/coredumpctl.xml | |
| parent | 4e3fbc133eb099acada182e3049d278023d91360 (diff) | |
| download | systemd-5dd55303f4d7e4294e0d2f1098232bd0fa305832.tar.gz | |
coredump: use "POSIX quotes" for cmdline
$ coredumpctl info |grep Command
Command Line: bash -c kill -SEGV $$ (before)
Command Line: bash -c "kill -SEGV \$\$" (road not taken, C quotes)
Command Line: bash -c $'kill -SEGV $$' (now, POSIX quotes)
Before we wouldn't use any quoting, making it impossible to figure how the
command line was split into arguments. We could use "normal" quotes, but this
has the disadvantage that the commandline *looks* like it could be pasted into
the terminal and executed, but this is not true: various non-printable
characters cannot be expressed in this quoting style. (This is not visible in
this example). Thus, "POSIX quotes" are used, which should allow any command
line to be expressed acurrately and pasted directly into a shell prompt to
reexecute.
I wonder if we should another field in the coredump entry that simply shows the
original cmdline with embedded NULs, in the original /proc/*/cmdline
format. This would allow clients to format the data as they see fit. But I
think we'd want to keep the serialized form anyway, for backwards compatibility.
Diffstat (limited to 'man/coredumpctl.xml')
| -rw-r--r-- | man/coredumpctl.xml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/man/coredumpctl.xml b/man/coredumpctl.xml index 68ba6dc9f2..6ceed41b05 100644 --- a/man/coredumpctl.xml +++ b/man/coredumpctl.xml @@ -362,7 +362,7 @@ Fri … 552351 1000 1000 SIGSEGV present /usr/lib64/firefox/firefox 28.7M GID: 1000 (user) Signal: 11 (SEGV) Timestamp: Mon 2021-01-01 00:00:01 CET (20s ago) - Command Line: bash -c kill -SEGV $$ + Command Line: bash -c $'kill -SEGV $$' Executable: /usr/bin/bash Control Group: /user.slice/user-1000.slice/… Unit: user@1000.service |