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.

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.