[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add shell-quasiquote.
From: |
Taylan Ulrich Bayırlı/Kammer |
Subject: |
Re: [PATCH] Add shell-quasiquote. |
Date: |
Mon, 19 Oct 2015 15:53:42 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Random832 <address@hidden> writes:
> address@hidden (Taylan Ulrich "Bayırlı/Kammer") writes:
>> It was not criticism of shell-quote-argument (those are separate).
>> Indeed it quotes arguments. My variant also quotes things that may be
>> the name of the command and not an argument.
>
> But why does it *need* to?
>
> Do you realize that you are now suggesting an injection scenario whereby
> the attacker is _legitimately_ permitted to supply an arbitrary string
> for an ordinary command to be executed, but somehow letting them execute
> "if" [which will be a syntax error anyway since they can't supply the
> then/fi as separate statements] becomes a security hole?
It's mostly just a side-effect of the simpler implementation. If
there's a /bin/if on the system, (shqq (if blah blah)) will call it.
Not very useful, but consistent.
It isn't necessary for shell-quote-argument to do something like that
for me to decide to use it, only the safety guarantees are necessary.
Taylan
- Re: [PATCH] Add shell-quasiquote., (continued)
Re: [PATCH] Add shell-quasiquote., Artur Malabarba, 2015/10/17
Re: [PATCH] Add shell-quasiquote., Taylan Ulrich Bayırlı/Kammer, 2015/10/19
- Re: [PATCH] Add shell-quasiquote., David Kastrup, 2015/10/19
- Re: [PATCH] Add shell-quasiquote., Taylan Ulrich Bayırlı/Kammer, 2015/10/19
- Re: [PATCH] Add shell-quasiquote., Random832, 2015/10/19
- Re: [PATCH] Add shell-quasiquote.,
Taylan Ulrich Bayırlı/Kammer <=
- Re: [PATCH] Add shell-quasiquote., Paul Eggert, 2015/10/19
- Re: [PATCH] Add shell-quasiquote., Taylan Ulrich Bayırlı/Kammer, 2015/10/19
- Re: [PATCH] Add shell-quasiquote., Paul Eggert, 2015/10/19
- Re: [PATCH] Add shell-quasiquote., Taylan Ulrich Bayırlı/Kammer, 2015/10/20
- Re: [PATCH] Add shell-quasiquote., Nicolas Richard, 2015/10/20
- Re: [PATCH] Add shell-quasiquote., Dmitry Gutov, 2015/10/20
- Re: [PATCH] Add shell-quasiquote., Taylan Ulrich Bayırlı/Kammer, 2015/10/20
- Re: [PATCH] Add shell-quasiquote., Dmitry Gutov, 2015/10/20
- Re: [PATCH] Add shell-quasiquote., Taylan Ulrich Bayırlı/Kammer, 2015/10/20
- Re: [PATCH] Add shell-quasiquote., Dmitry Gutov, 2015/10/20