Convert variable name to canonical spelling before checking for matches
in GUCArrayAdd/GUCArrayDelete.  This prevents the multiple-entry bug
exhibited by Frank Lupo 28-Jan-2003.

Revert off setting mention for client_min_messages.

Set log_min_error_messages to the proper and agreed-upon default, PANIC
(off).  (BACKPATCH)

Prevent core dump when die_horribly() is called with null AH pointer.
Problem reported and fixed by Oliver Elphick.

Back-patch fix for not shell-quoting editor name, as it also fixes
silly buffer overrun (someone needs to learn to count characters...)

Fix obsolete error message (isImmutable doesn't exist).

Change CREATE TABLE AS / SELECT INTO to create the new table with OIDs,
for backwards compatibility with pre-7.3 behavior.  Per discussion on
pgsql-general and pgsql-hackers.

Update 'Mathematical Functions' table to reflect 7.3 addition of float8
forms of ceil(), floor(), sign().  Back-patch this and other recent
doc fixes in this file to the 7.3 branch.

Make estimation of mergejoin scan selectivities more robust, per recent
example from RaÇl GutiÅrrez.

Fix coredump problem in plpgsql's RETURN NEXT.  When a SELECT INTO
that's selecting into a RECORD variable returns zero rows, make it
assign an all-nulls row to the RECORD; this is consistent with what
happens when the SELECT INTO target is not a RECORD.  In support of
this, tweak the SPI code so that a valid tuple descriptor is returned
even when a SPI select returns no rows.

Back-patch typprtlen fix, two other minor post-7.3 fixes.

Fix bogus formatting of DROP DATABASE command, per Christopher Kings-Lynne.

Repair an embarrassingly large number of alphabetization mistakes in the
datetime token tables.  Even more embarrassing, the regression tests
revealed some of the problems --- but evidently the bogus output wasn't
questioned.  Add code to postmaster startup to directly check the tables
for correct ordering, in hopes of not being embarrassed like this again.

More comment cleanups for shared_buffers in postgresql.conf.

Update shared_buffer comment in postgresql.conf.

This patch fixes an incorrect statement and makes a few cleanups to
contrib/fulltextindex/README.fti

Backpatched to 7.3.X too.

Neil Conway

Bumped up build number to 107

Backport to 7.3 of patch to correctly handle fractional seconds

 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSet.java
  jdbc/org/postgresql/test/jdbc2/TimestampTest.java

Patch from Kris Jurka to improve the performance of getImportedKeys().  The
fix uses explicit joins to avoid using the genetic query optimizer for
consistent results.

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

Fix capitalization.

Cause symlinks for shared-library versioning to run in the standard
direction on HP-UX; our former approach has been obsolete since HPUX 9.

Giles Lean

Fix typo in an error messages

> > This patch improves the documentation for the shared_buffers GUC param.
>
> I'd suggest that the runtime.sgml description explicitly say "values of
> at least a few thousand are recommended for production installations".

Neil Conway

A tiny patch to fix a typo in configure.in and another one in
RELEASE_CHANGES.

Manfred Koizar

Add missing <limits.h> for INT64_IS_BUSTED case.

Fix indenting of log_min_error_statement.

Repair inconsistent rounding behavior for timestamp, time, interval,
per gripe from Csaba Nagy.  There is still potential for platform-specific
behavior for values that are exactly halfway between integers, but at
least we now get the expected answer for all other cases.

Remove 'const' from secure_write, to prevent compiler warning.

The second was that renegotiation was just plain broken.  I can't
believe I didn't notice this before -- once 64k was sent to/from the
server the client would crash.  Basicly, in 7.3 the server SSL code set
the initial state to "about to renegotiate" without actually starting
the renegotiation.  In addition, the server and client didn't properly
handle the SSL_ERROR_WANT_(READ|WRITE) error.  This is fixed in the
second patch.

Nathan Mueller

I was playing around with 7.3.1 and found some more SSL problems.  The
first, that I missed when checking over 7.3.1, was that the client
method was switched to SSLv23 along with the server.  The SSLv23 client
method does SSLv2 by default, but can also understand SSLv3.  In our
situation the SSLv2 backwords compatibility is really only needed on the
server.  This is the first patch.

The last thing is that I found a way for the server to understand SSLv2
HELLO messages (sent by pre-7.3 clients) but then get them to talk
SSLv3.  This is the last one.

Nathan Mueller

Fix dumping of DEFERRABLE/INITIALLY DEFERRED:

> The big problem is that while pg_dump's dump_trigger() looks at
> tginitdeferred and dumps accordingly, pg_get_constraintdef doesn't look
> at tginitdeferred, and therefore doesn't record the requirement as part
> of ALTER TABLE ADD CONSTRAINT.

pg_get_constraintdef should probably be looking at condeferrable and
condeferred in the pg_constraint row it's looking at.  Maybe something
like the attached.

(Added, output only non-default values.)

Stephan Szabo

Repair bug noticed by Deepak Bhole: a shell type should have a dependency
on its namespace, so that it will go away if the schema is dropped.

Fix pgpass to work with PQconnectDB().

Backpatch to 7.3.X.

Report from ljb.

Fix for bug #866.  7.3 contains new logic for avoiding redundant calls to
the index AM when we know we are fetching a unique row.  However, this
logic did not consider the possibility that it would be asked to fetch
backwards.  Also fix mark/restore to work correctly in this scenario.

Backpatch extra slash removal into 7.3.X. HEAD already has it.

Fix doc chmod .pgpass line in 7.3.X.  Head already fixed.

Upgrade to my2pg 1.9

Upgrade to even newer ora2pg 2.0.

Upgrade to ora2pg 1.10.  Backpatch to 7.3.X.

Fix inherited UPDATE for cases where child column numbering doesn't
match parent table.  This used to work, but was broken in 7.3 by
rearrangement of code that handles targetlist sorting.  Add a regression
test to catch future breakage.

Add missing include --- seems include file references are different in
CVS tip than in this branch.

Translation updates

Add casts between oid and lo in contrib.

Fix BSD/OS thread file locking bug in new fseeko code.

Backpatch to 7.3.X.

fastpath code neglected to check whether user has privileges to call the
target function.  Also, move SetQuerySnapshot() call to avoid assert
failure when a fastpath call is attempted in an aborted transaction.

Awhile back I wrote that freebsd.h was probably broken in the places where
it diverged from netbsd.h and openbsd.h.  This has now been confirmed.
Accordingly, make all three exactly alike.

Point does not return a center of lseg.

Kenji Sugita

Adjust hash table sizing algorithm to avoid integer overflow in
ExecHashJoinGetBatch().  Fixes core dump on large hash joins, as in
example from Rae Stiening.

Deliver better error message when a relation name is used in an expression.
Per report from Ian Barwick.

Remove overenthusiastic free'ing of comment dependencies; could lead to
core dump in pg_dump when dumping views having comments.  See bug #855.

Small fix in documentation and some examples of usage. Please, apply to
7.3 and current CVS

Oleg Bartunov

Trivial patch to increase max_fsm_pages as per earlier discussion.

Philip Warner

Update SSL error message for improper permissions.  Backpatch to 7.3.X.

Fix possible buffer overrun in \d command: substr(..., 128) produces
a result of at most 128 chars, but that could be more than 128 bytes.
Also ensure we don't try to pfree uninitialized pointers during error
cleanup.

Update 7.3.1 to give proper attribution to pg_dump item.

Fix release notes and installation instructions for 7.3.1 release.

Update Russian FAQ.  Update all FAQ's for 7.3.X branch.

Update RELEASE checklist.

Update INSTALL file for 7.3.1.

From the SSL_CTX_new man page:

"SSLv23_method(void), SSLv23_server_method(void), SSLv23_client_method(void)

 A TLS/SSL connection established with these methods will understand the SSLv2,
 SSLv3, and TLSv1 protocol. A client will send out SSLv2 client hello messages
 and will indicate that it also understands SSLv3 and TLSv1. A server will
 understand SSLv2, SSLv3, and TLSv1 client hello messages. This is the best
 choice when compatibility is a concern."

This will maintain backwards compatibility for those us that don't use
TLS connections ...

Update FAQ for 7.3.1.

Stamp 7.3.1.

Fix ALTER TABLE ADD COLUMN to disallow the same column types that are
disallowed by CREATE TABLE (eg, pseudo-types); also disallow these types
from being introduced by the range-function syntax.  While at it, allow
CREATE TABLE to create zero-column tables, per recent pghackers discussion.
I am back-patching this into 7.3 since failure to disallow pseudo-types
is arguably a security hole.

The actual segfault was caused by a double pfree(), but ISTM that
failing to find pg_hba.conf should be a fatal error anyway, so I
increased the priority of the elog() from LOG to FATAL and refactored
the code a little bit.

Neil Conway

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.