Zsh Mailing List Archive
Messages sorted by:
Reverse Date,
Date,
Thread,
Author
Re: Get cursor position (Was: [bug report] prompt can erase messages written on the terminal by background processes)
2022-12-09 13:46:21 +0100, Philippe Altherr:
[...]
> assign() {
> > print -v "$1" "$2"
> > }
[...]
Note that that "assign" function has a command injection
vulnerability.
Even more so than for other commands, -- (- also works) should
always be used for "print" to separate options from non-options
at least when the first non-option argument is not guaranteed
not to start with - or +.
Try for instance:
assign var '-va[1$(reboot)]'
So:
assign() print -rv "$1" -- "$2"
Or the Bourne-compatible:
assign() eval "$1=\$2"
Or POSIX:
assign() { eval "$1=\$2"; }
A bit ironic that people often go to great lengths to avoid
using "eval" but end up coming up with unsafe solutions.
--
Stephane
Messages sorted by:
Reverse Date,
Date,
Thread,
Author