pgsql: Fix encode(...bytea..., 'escape') so that it converts all

From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix encode(...bytea..., 'escape') so that it converts all
Date: 2008-02-26 02:54:14
Message-ID: 20080226025414.792D5754108@cvs.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Fix encode(...bytea..., 'escape') so that it converts all high-bit-set byte
values into \nnn octal escape sequences. When the database encoding is
multibyte this is *necessary* to avoid generating invalidly encoded text.
Even in a single-byte encoding, the old behavior seems very hazardous ---
consider for example what happens if the text is transferred to another
database with a different encoding. Decoding would then yield some other
bytea value than what was encoded, which is surely undesirable. Per gripe
from Hernan Gonzalez.

Backpatch to 8.3, but not further. This is a bit of a judgment call, but I
make it on these grounds: pre-8.3 we don't really have much encoding safety
anyway because of the convert() function family, and we would also have much
higher risk of breaking existing apps that may not be expecting this behavior.
8.3 is still new enough that we can probably get away with making this change
in the function's behavior.

Tags:
----
REL8_3_STABLE

Modified Files:
--------------
pgsql/src/backend/utils/adt:
encode.c (r1.20 -> r1.20.2.1)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/encode.c?r1=1.20&r2=1.20.2.1)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2008-02-26 02:59:48 Re: [COMMITTERS] pgsql: Link postgres from all object files at once, to avoid the
Previous Message Tom Lane 2008-02-26 02:54:08 pgsql: Fix encode(...bytea..., 'escape') so that it converts all