Re: psql crash fix - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: psql crash fix
Date
Msg-id 20130404165637.GA26874@momjian.us
Whole thread Raw
In response to psql crash fix  (Bruce Momjian <bruce@momjian.us>)
List pgsql-hackers
On Tue, Apr  2, 2013 at 08:48:53PM -0400, Bruce Momjian wrote:
> I found that psql will crash if given a PSQLRC value containing a tilde:
> 
>     $ PSQLRC="~/x" psql test
>     *** glibc detected *** psql: free(): invalid pointer: 0x00007fffb7c933ec ***
> 
> This is on Debian Squeeze 6.0.7.  The fix is to pstrdup() the value
> returned by getenv(), so it can be free()'ed later --- you can't free
> getenv()-returned values:
> 
>        As typically implemented, getenv() returns a pointer to a string
>        within the environment list.  The caller must take care not to
>        modify this string, since that would change the environment of
>        the process.
> 
> This bug exists in 9.2 and git head.  I also removed the return value
> from expand_tilde() as no caller was using it.

Applied.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +



pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: corrupt pages detected by enabling checksums
Next
From: Heikki Linnakangas
Date:
Subject: Re: Regex with > 32k different chars causes a backend crash