Re: pg_dump: fix crash on error - Mailing list pgsql-patches

From Neil Conway
Subject Re: pg_dump: fix crash on error
Date
Msg-id 42E71518.3000904@samurai.com
Whole thread Raw
In response to Re: pg_dump: fix crash on error  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: pg_dump: fix crash on error  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-patches
Tom Lane wrote:
> [ scratches head... ]  Did this code change recently?  It's a tad hard
> to believe that such a thing could have gone unnoticed for long.

Seems the problem was introduced here:

http://developer.postgresql.org/cvsweb.cgi/pgsql/src/bin/pg_dump/pg_backup_db.c.diff?r1=1.58;r2=1.59

(I could just revert the code to using PQdb(), but I don't think there's
a need to mention the database in the error message ourselves --
PQerrorMessage() should include that information if appropriate.)

-Neil

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: pg_dump: fix crash on error
Next
From: Tom Lane
Date:
Subject: Re: pg_dump: fix crash on error