Re: proof concept: do statement parametrization - Mailing list pgsql-hackers

From Pavel Stehule
Subject Re: proof concept: do statement parametrization
Date
Msg-id AANLkTimJ_LumnGbn-eC9eRgS4lKRsWTBOGZG-uTAMUgk@mail.gmail.com
Whole thread Raw
In response to Re: proof concept: do statement parametrization  (Florian Pflug <fgp@phlo.org>)
List pgsql-hackers
2010/7/5 Florian Pflug <fgp@phlo.org>:
> On Jul4, 2010, at 13:57 , Pavel Stehule wrote:
>>> I don't really buy that argument. By using a psql variable, you simply move the quoting & escaping business from
SQLto the shell where psql is called. True, you avoid SQL injectiont, but in turn you make yourself vulnerable to shell
injection.
>>
>> can you show some example of shell injection? For me, this way via
>> psql variables is the best. There are clean interface between outer
>> and inner space. And I can call simply just psql scripts - without
>> external bash.
>
> Well, on the one hand you have (with your syntax)
> echo "DO (a int := $VALUE) $$ ... $$" | psql
> which allows sql injection if $VALUE isn't sanitized or quoted & escaped properly.

sure - but it is same for you syntax, isn't it? This is classical
dynamic SQL - and more used in from untyped language.

>
> On the other hand you have
> echo "DO (a int := :value) $$ ... $$$ | psql --variable value=$VALUE
> which allows at least injection of additional arguments to psql if $VALUE contains spaces. You might try to avoid
thatby encoding value=$VALUE in double quotes, but I doubt that it's 100% safe even then.
 

[pavel@nemesis ~]$ cat y.sh
a='some variable with " ajjaja" jjaja'
b='other variable with "jaja'
c="third 'variable"
psql postgres --variable a="$a" --variable b="$b" --variable c="$c" <<EOT
\echo 'a = ' :'a'
\echo 'b = ' :'b'
\echo 'c = ' :'c'
EOT
[pavel@nemesis ~]$ sh y.sh
a =  'some variable with " ajjaja" jjaja'
b =  'other variable with "jaja'
c =  'third ''variable'

it is safe - and it is only one really secure way. My design calculate with it

you can do

DO(a int := :'variable') ... and variable is well escaped and value is
casted to int. I am really very happy from :'xxx' feature.

regards

Pavel

>
> The point is that interpolating the value into the command is always risky, independent from whether it's a shell
commandor an sql command.
 
>
> best regards,
> Florian Pflug
>
>


pgsql-hackers by date:

Previous
From: Takahiro Itagaki
Date:
Subject: Always truncate segments before unlink
Next
From: Pavel Stehule
Date:
Subject: Implementation of median in PostgreSQL - questions