Re: Patch for psql to avoid altering the PGresult strings - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: Patch for psql to avoid altering the PGresult strings
Date
Msg-id 200306240333.h5O3XkU15754@candle.pha.pa.us
Whole thread Raw
In response to Patch for psql to avoid altering the PGresult strings  (Dennis Björklund <db@zigo.dhs.org>)
Responses Re: Patch for psql to avoid altering the PGresult strings
List pgsql-patches
What should be done with this patch?

---------------------------------------------------------------------------

Dennis Bj�rklund wrote:
> The mbvalidate() function was called on the PGresult strings, and it
> changes the strings (removes unknown characters).
>
> I've change the validation function to one that either returns a new
> string when needed or NULL when the string already was validating. The
> normal case is to have validating strings so in most cases no new strings
> are created.
>
> I have not included changes to avoid sending non validating strings to the
> server. It's not clear what is the best way to solve it. Maybe one should
> just do a stupid fix for 7.4 and validate the strings like above and just
> delete non validating strings. It is possible to solve it in a better way,
> but it's more complicated. We have 4 different charsets to think about in
> psql. The server, the client, the terminal and the message catalog (the
> last two usually matches, but not always). I would prefer to find a good
> solution instead of just patching one small problem after another.
>
> --
> /Dennis

Content-Description:

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 7: don't forget to increase your free space map settings

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: [HACKERS] Subtraction carry bug in xlog.c in 7.3 and 7.4
Next
From: Tom Lane
Date:
Subject: Re: Patch to be verbose about being unable to read ~/.pgpasss...