From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Jeff Davis <pgsql(at)j-davis(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-general(at)postgresql(dot)org |
Subject: | Re: Possible problem with PQescapeStringConn and |
Date: | 2006-11-23 04:02:13 |
Message-ID: | 200611230402.kAN42DB12033@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general |
I tried to add this to the documentation, config.sgml, but everything I
come up with sounds so long-winded that it would do more harm than good.
---------------------------------------------------------------------------
Jeff Davis wrote:
> On Thu, 2006-10-26 at 19:46 -0400, Tom Lane wrote:
> > Jeff Davis <pgsql(at)j-davis(dot)com> writes:
> > > You can set standard_conforming_strings in postgresql.conf at any time
> > > and reload the config, changing the value for all active connections.
> > > That means that if a client opens a connection, and you SIGHUP postgres,
> > > and then the client issues a PQescapeStringConn, the client will get an
> > > incorrectly-escaped string.
> >
> > The window for this is pretty narrow, because PQescapeStringConn will
> > use the latest-delivered parameter status, but it's certainly true that
> > randomly changing standard_conforming_strings wouldn't be a bright idea.
> > Probably a documentation note recommending against changing it via
> > SIGHUP would be sufficient.
> >
>
> It's not a narrow time window (which was my original test), but you're
> right that it is narrow in the sense that any command executed on that
> connection will update the status. So, a potential attacker has one
> chance :)
>
> It seems like a documentation note would be sufficient to prevent people
> from changing it too haphazardly. You wouldn't want to change it at
> runtime if the bulk of your queries involved escape sequences.
>
> Regards,
> Jeff Davis
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 9: In versions below 8.0, the planner will ignore your desire to
> choose an index scan if your joining column's datatypes do not
> match
--
Bruce Momjian bruce(at)momjian(dot)us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2006-11-23 05:09:40 | Re: ISO week dates |
Previous Message | Richard Broersma Jr | 2006-11-23 02:36:58 | Re: IS it a good practice to use SERIAL as Primary Key? |