From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Gavan Schneider <list(dot)pg(dot)gavan(at)pendari(dot)org> |
Cc: | Durumdara <durumdara(at)gmail(dot)com>, Postgres General <pgsql-general(at)postgresql(dot)org> |
Subject: | Re: How to convert escaped text column - force E prefix |
Date: | 2021-01-06 11:03:04 |
Message-ID: | CAFj8pRBJsiwxQN3sNk1hKECbB7u=yz8+DFrM+rr+=aGhqiryUQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general |
st 6. 1. 2021 v 10:54 odesílatel Gavan Schneider <list(dot)pg(dot)gavan(at)pendari(dot)org>
napsal:
> On 6 Jan 2021, at 19:43, Pavel Stehule wrote:
>
> Currently there are not any functions that you need. You need to write
> your
> own.
>
> CREATE OR REPLACE FUNCTION public.unistr(text)
> RETURNS text
> LANGUAGE plpgsql
> IMMUTABLE STRICT
> AS $function$
> declare r text;
> begin
> execute 'select e''' || quote_literal($1) || '''' into r;
> return r;
> end;
> $function$;
>
> Attention: This is ugly and possible sql injection vulnerable!!! But there
> is not another way. The fix is in queue
>
> https://www.postgresql.org/docs/current/functions-string.html
> quote_literal ( text ) → text
> Returns the given string suitably quoted to be used as a string literal in an SQL statement string.
> Embedded single-quotes and backslashes are properly doubled.
> Note that quote_literal returns null on null input; if the argument might be null, quote_nullable is often more suitable.
> See also Example 42.1. quote_literal(E'O\'Reilly') → 'O''Reilly'
>
> It is even more ugly but would it at least help with the SQL injection
> risk?
>
it cannot work, because \ will be replaced by \\
postgres=# CREATE OR REPLACE FUNCTION public.unistr(text)
RETURNS text
LANGUAGE plpgsql
IMMUTABLE STRICT
AS $function$
declare r text;
begin
execute 'select ' || quote_literal($1) into r;
return r;
end;
$function$
;
CREATE FUNCTION
postgres=# select unistr('Az ad\u00f3kulcsonk\u00e9nti');
┌──────────────────────────────┐
│ unistr │
╞══════════════════════════════╡
│ Az ad\u00f3kulcsonk\u00e9nti │
└──────────────────────────────┘
(1 row)
Gavan Schneider
> ——
> Gavan Schneider, Sodwalls, NSW, Australia
> Explanations exist; they have existed for all time; there is always a
> well-known solution to every human problem — neat, plausible, and wrong.
> — H. L. Mencken, 1920
>
From | Date | Subject | |
---|---|---|---|
Next Message | Li EF Zhang | 2021-01-06 11:14:24 | RE: SQL to query running transactions with subtransactions that exceeds 64 |
Previous Message | Gavan Schneider | 2021-01-06 09:53:53 | Re: How to convert escaped text column - force E prefix |