Ok, I think I've gotten this figured out now.  I saw this comment in
pqcomm.c, switched the ERROR logs to COMMERROR logs and it all works.
I've attached a patch to be-secure.c that fixes all my problems.

Nathan Mueller

Backpatch SSL fixes into 7.3.1.

Increment libpq major number for 7.3.1 and minor for 7.4.

This patch fixes minor bugs in dictionary generator in contrib/tsearch
(contrib/tsearch/makedict/makedict.pl)

[ Backpatched to 7.3.]

Teodor Sigaev

Guard against overrunning CTZName buffer when TZ is bogus.

Back-patch fix to prevent core dump in EXPLAIN if optimizer has
simplified function call to a constant.  (7.3 won't actually execute
such a plan anyway, but core dump is bad regardless.)

Mark SCO Openserver 5.0.4. supported by 7.3.1 as per chat report from user.
template/sco updated.

Fix line count error reporting in config files, like pg_hba.conf, per
report from Oliver Elphick.

Backpatch to 7.3.

Bump version for 7.3 and 7.4.

Translation updates

This patch updates doc/src/sgml/runtime.sgml to refer to "MacOS X", not
"OS/X".

Neil Conway

Fix Latin1/Unicode conversion by selecting the right table.

As far as I figured from the source code this function only deals with
cleaning up locale names and nothing else. Since all the locale names
are in plain  ASCII I think it will be safe to use ASCII-only lower-case
conversion.

Nicolai Tufar

Explain's code for showing quals of SubqueryScan nodes has been broken
all along; not noticed till now.  It's a scan not an upper qual ...

Thank you very much, you catch it :). This bug had a long life, because it
exists if and only if locale of postmaster
was a different from C (or ru_RU.KOI8-R).

Please, apply patch for current CVS & 7.3.1

Magnus Naeslund(f) wrote:
> Ok, I nailed the bug, but i'm not sure what the correct fix is.
> Attached tsearch_morph.diff that remedies this problem by avoiding it.
> Also there's a debug aid patch if someone would like to know how i
> finally found it out :)
>
> There problem in the lemmatize() function is that GETDICT(...) returned
> a value not handled (BYLOCALE).
> The value (-1) and later used as an index into the dicts[] array.
> After that everything went berserk stack went crazy somehow so trapping
> the fault sent me to the wrong place, and every time i read the value it
> was positive ;)
>
> So now i just return the initial word passed to the lemmatize function,
> because i don't know what to do with it.

Magnus Naeslund

localbuf.c must be able to do blind writes.

Avoid pulling up sublinks from a subselect's targetlist.  Works around
problems that occur if sublink is referenced via a join alias variable.
Perhaps this can be improved later, but a simple and safe fix is needed
for 7.3.1.

It looks like the problem was introduced when the "SET autocommit" and
"SET search_path"  commands were added to the beginning of the script.

The attatched patch should fix the problem. It probably should be
applied  against the 7.3 and 7.4 branches.

Steven Singer

Allow 'password' encryption even when pg_shadow has MD5 passwords, per
report from Terry Yapt and Hiroshi.

Backpatch to 7.3.

changed sort of column names to sort by attnum, not attname

Repair oversight in recent change of dependency extraction code: when
recursing to handle a join alias var, the context had better be set to
be appropriate to the join var's query level.  Per report from Hristo Neshev.

Fix off-by-one bug in usage check.

Repair core dump when trying to delete an entry from an already-NULL
datconfig or useconfig field.  Per report from Dustin Sallings.

Fix script name in README.

Fix dependency generation for multicolumn foreign keys.  From Adam Buraczewski.

Fix pg_dump failure against 7.1 or older servers, per
report from Philip Warner.

Repair coredump in COPY OUT when client encoding is different from
server encoding, per report from Nicolai Tufar.  This is a minimum-risk
patch for 7.3.1 --- I intend to apply a more general repair in HEAD.

Repair prehistoric logic error in lseg_eq and lseg_ne.

rm should be rm -f, per comment from Peter Harris.
(Checked with Peter Eisentraut.)

Update resultmap pattern for geometry-solaris-i386-pc: should match
solaris 2.8 as well as 2.7.  Per report from Martin Renters.

Update TODO in 7.3 branch

Recreate HISTORY file to match release.sgml changes.  Stamp 7.3 final in
configure/configure.in.

Update build rules for /HISTORY.

Fix typo in release notes.

Allocate proper length for sprintf string, from Tatsuo.

Guard against 0 length string encoding conversion case.

Remove check for pg_group file as part of initdb -W password assignment
--- not needed.

Fix inappropriate quoting in dblink.  From Joe Conway.

Update ports list from recent regression-test-database entries.

Release note improvements (Neil, Tom)

Remove inappropriate double-quoting in connectby() code; adjust
regression test to avoid using VALUE as a name.  From Joe Conway.

Fix a dozen or so places that were passing unpredictable data strings
as elog format strings.  Hai-Chen Tu pointed out the problem in
contrib/dbmirror, but it wasn't the only such error.

Fix elog in dbmirror to use %s.

Hai-Chen Tu

Quote database name properly when invoking pg_dump.  Per report from
Christopher Kings-Lynne.

TAS code originally written for s390 (32-bit) does not work for s390x
(64-bit).  Fix it.  Per report from Permaine Cheung.

Fixups for man pages

Fix breakage in new-in-7.3 timetz_zone() function: was giving random
results due to doing arithmetic on uninitialized values.  Add some
documentation about the AT TIME ZONE construct.  Update some other
date/time documentation that seemed out of date for 7.3.

Regenerate

Code review for superuser_reserved_connections patch.  Don't try to do
database access outside a transaction; revert bogus performance improvement
in SIBackendInit(); improve comments; add documentation (this part courtesy
Neil Conway).

NetBSD versions 1.6 and (we hope) later can use the standard geometry
comparison file, instead of geometry-positive-zeros.  Per report from
Patrick Welche.

This locale-specific variant file seems not to have been updated
when a test was added to select_having.sql back in June.

Update ports list.

Fix a stupid cut and paste error from the commit last night.

 Modified Files:
  Tag: REL7_3_STABLE
  AbstractJdbc2Statement.java

Backport bug with executeBatch.  This bug was a regression from 7.2 and has
been reported by Marko Strukelj and Keith Wannamaker

 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/test/jdbc2/BatchExecuteTest.java

Back out NetBSD/arm32 port.

Update ports list.

Fix markup breakage.

Fix typo and markup.

Update ports list.

Stamp RC2.

Put back error test for DECLARE CURSOR outside a transaction block ...
but do it correctly now.

Revise SQL features list.

mis-tag'd as the release instead of rc1 ...

Update TODO/FAQ for 7.3 release.

Update for RC1.

Push qual clauses containing subplans to the back of the qual list
at each plan node.  Per gripe from Ross Reedstrom.

Update ports list.

Fix Magnus's name.

Fix version number for FreeBSD/alpha on ports list.

Update ports list.

Backpatch from head: fixes for a threading bug, a problem with setNull involving
server prepared statements and an improved error message

 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/errors.properties
  jdbc/org/postgresql/core/Encoding.java
  jdbc/org/postgresql/core/QueryExecutor.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1DatabaseMetaData.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java

Update ports list.

Update ports list.

Update German translations.

Allow +1300 as a numeric timezone specifier; we already accept FJST as meaning +1300.

Use cat instead of echo or awk, to avoid portability problems with backslashes.

Add AIX as supported platform.

Update AIX resultmap for horology tests, per Andreas.

Fix broken GB18030 <--> UTF-8 conversion map

Remove NO_MKTIME_BEFORE_1970.  I had speculated that it was not needed
anymore given the mktime() workaround now done in DetermineLocalTimeZone.
This has now been confirmed by Robert Bruccoleri for Irix, and I'm going
to extrapolate to AIX as well.

Improve formatting of release notes.  Update makefile rule to make
HISTORY file.

Revise platform FAQs.

Improve wording.

Merge changes from head. (update classname for PGConnection)

Merge fix from head for proper join to pg_description

 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/jdbc1/AbstractJdbc1DatabaseMetaData.java
  jdbc/org/postgresql/test/jdbc2/DatabaseMetaDataTest.java

Add cast to suppress compile warning on Alphas.

Update russian FAQ, from Viktor Vislobokov.

Cleanup pass over User's Guide.  Key word table updated.

Fix markup.

Update FAQ_SCO from Larry Rosenman, correction from Tom Lane.

Back-patch changes to address Perl _GNU_SOURCE problem.

Update libpq PQcmdTuples() and PQcmdStatus() docs: remove const param
mention to match code and include files.

I'm using libpgtcl with PostgreSQL-7.2.3. There is a discrepancy between
the documentation and behavior of "pg_result $res -oid" when $res is not
from an INSERT.  The documentation says it should return an empty
string, but in fact it returns 0. I think it used to return an empty
string around PostgreSQL-7.1.3 when PQoidStatus() was used, but now it
uses PQoidValue() which returns InvalidOid, 0 in this case.

Assuming the current behavior is desired, here is a patch to the
documentation doc/src/sgml/libpgtcl.sgml to match what really happens:

ljb

Update release docs for 7.3.

Remove inappropriate inclusions of OpenSSL internal header e_os.h,
as well as unnecessary (and incorrect on Windows) assignments to
errno/SOCK_ERRNO.

Remove no-longer-needed inclusions to improve backward compatibility
with older bison versions.

Cleanup pass over Tutorial and Admin Guide

Remove GPL file pursuant to relicensing.

Stamp beta5.

Change dbmirror lincense to BSD.