[COMMITTERS] pgsql: Added fixed from the coverity report send in by Joachim Wieland

2006-06-21 Thread Michael Meskes
Log Message:
---
Added fixed from the coverity report send in by Joachim Wieland <[EMAIL 
PROTECTED]>
Added missing error handling in a few functions in ecpglib

Modified Files:
--
pgsql/src/interfaces/ecpg:
ChangeLog (r1.304 -> r1.305)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ChangeLog.diff?r1=1.304&r2=1.305)
pgsql/src/interfaces/ecpg/compatlib:
informix.c (r1.42 -> r1.43)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/compatlib/informix.c.diff?r1=1.42&r2=1.43)
pgsql/src/interfaces/ecpg/ecpglib:
connect.c (r1.28 -> r1.29)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/connect.c.diff?r1=1.28&r2=1.29)
data.c (r1.31 -> r1.32)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/data.c.diff?r1=1.31&r2=1.32)
descriptor.c (r1.14 -> r1.15)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/descriptor.c.diff?r1=1.14&r2=1.15)
execute.c (r1.46 -> r1.47)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/execute.c.diff?r1=1.46&r2=1.47)
extern.h (r1.16 -> r1.17)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/extern.h.diff?r1=1.16&r2=1.17)
pgsql/src/interfaces/ecpg/pgtypeslib:
datetime.c (r1.28 -> r1.29)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/datetime.c.diff?r1=1.28&r2=1.29)
dt_common.c (r1.34 -> r1.35)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/dt_common.c.diff?r1=1.34&r2=1.35)
numeric.c (r1.26 -> r1.27)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/numeric.c.diff?r1=1.26&r2=1.27)
timestamp.c (r1.34 -> r1.35)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/timestamp.c.diff?r1=1.34&r2=1.35)
pgsql/src/interfaces/ecpg/test:
dt_test.pgc (r1.9 -> r1.10)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/test/dt_test.pgc.diff?r1=1.9&r2=1.10)

---(end of broadcast)---
TIP 6: explain analyze is your friend


[COMMITTERS] pgsql: Added fixes from the coverity report send in by Joachim Wieland

2006-06-21 Thread Michael Meskes
Log Message:
---
Added fixes from the coverity report send in by Joachim Wieland <[EMAIL 
PROTECTED]>
Added missing error handling in a few functions in ecpglib.

Tags:

REL8_1_STABLE

Modified Files:
--
pgsql/src/interfaces/ecpg/compatlib:
informix.c (r1.39.2.2 -> r1.39.2.3)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/compatlib/informix.c.diff?r1=1.39.2.2&r2=1.39.2.3)
pgsql/src/interfaces/ecpg/ecpglib:
connect.c (r1.26.2.2 -> r1.26.2.3)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/connect.c.diff?r1=1.26.2.2&r2=1.26.2.3)
data.c (r1.29.2.1 -> r1.29.2.2)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/data.c.diff?r1=1.29.2.1&r2=1.29.2.2)
descriptor.c (r1.12.6.2 -> r1.12.6.3)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/descriptor.c.diff?r1=1.12.6.2&r2=1.12.6.3)
execute.c (r1.43.2.2 -> r1.43.2.3)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/execute.c.diff?r1=1.43.2.2&r2=1.43.2.3)
extern.h (r1.15 -> r1.15.2.1)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/extern.h.diff?r1=1.15&r2=1.15.2.1)
pgsql/src/interfaces/ecpg/pgtypeslib:
datetime.c (r1.25.2.1 -> r1.25.2.2)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/datetime.c.diff?r1=1.25.2.1&r2=1.25.2.2)
dt_common.c (r1.32.2.1 -> r1.32.2.2)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/dt_common.c.diff?r1=1.32.2.1&r2=1.32.2.2)
numeric.c (r1.23.2.2 -> r1.23.2.3)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/numeric.c.diff?r1=1.23.2.2&r2=1.23.2.3)
timestamp.c (r1.34 -> r1.34.2.1)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/timestamp.c.diff?r1=1.34&r2=1.34.2.1)
pgsql/src/interfaces/ecpg/test:
dt_test.pgc (r1.8 -> r1.8.2.1)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/test/dt_test.pgc.diff?r1=1.8&r2=1.8.2.1)

---(end of broadcast)---
TIP 4: Have you searched our list archives?

   http://archives.postgresql.org


[COMMITTERS] pgsql: Added fixes from the coverity report send in by Joachim Wieland

2006-06-21 Thread Michael Meskes
Log Message:
---
Added fixes from the coverity report send in by Joachim Wieland <[EMAIL 
PROTECTED]>
Added missing error handling in a few functions in ecpglib.

Tags:

REL8_0_STABLE

Modified Files:
--
pgsql/src/interfaces/ecpg/compatlib:
informix.c (r1.38.4.1 -> r1.38.4.2)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/compatlib/informix.c.diff?r1=1.38.4.1&r2=1.38.4.2)
pgsql/src/interfaces/ecpg/ecpglib:
connect.c (r1.24.4.3 -> r1.24.4.4)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/connect.c.diff?r1=1.24.4.3&r2=1.24.4.4)
data.c (r1.27.4.2 -> r1.27.4.3)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/data.c.diff?r1=1.27.4.2&r2=1.27.4.3)
descriptor.c (r1.12.4.2 -> r1.12.4.3)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/descriptor.c.diff?r1=1.12.4.2&r2=1.12.4.3)
execute.c (r1.38.4.4 -> r1.38.4.5)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/execute.c.diff?r1=1.38.4.4&r2=1.38.4.5)
extern.h (r1.11.4.2 -> r1.11.4.3)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/extern.h.diff?r1=1.11.4.2&r2=1.11.4.3)
pgsql/src/interfaces/ecpg/pgtypeslib:
datetime.c (r1.20 -> r1.20.4.1)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/datetime.c.diff?r1=1.20&r2=1.20.4.1)
dt_common.c (r1.20 -> r1.20.4.1)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/dt_common.c.diff?r1=1.20&r2=1.20.4.1)
numeric.c (r1.20.4.1 -> r1.20.4.2)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/numeric.c.diff?r1=1.20.4.1&r2=1.20.4.2)
timestamp.c (r1.23 -> r1.23.4.1)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/pgtypeslib/timestamp.c.diff?r1=1.23&r2=1.23.4.1)
pgsql/src/interfaces/ecpg/test:
dt_test.pgc (r1.7 -> r1.7.6.1)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/test/dt_test.pgc.diff?r1=1.7&r2=1.7.6.1)

---(end of broadcast)---
TIP 9: In versions below 8.0, the planner will ignore your desire to
   choose an index scan if your joining column's datatypes do not
   match


[COMMITTERS] pgsql: Fixed small typo in changelog

2006-06-21 Thread Michael Meskes
Log Message:
---
Fixed small typo in changelog

Modified Files:
--
pgsql/src/interfaces/ecpg:
ChangeLog (r1.305 -> r1.306)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ChangeLog.diff?r1=1.305&r2=1.306)

---(end of broadcast)---
TIP 6: explain analyze is your friend


[COMMITTERS] pgsql: Added some more coverity report patches send in by Martijn van

2006-06-21 Thread Michael Meskes
Log Message:
---
Added some more coverity report patches send in by Martijn van Oosterhout 
.

Modified Files:
--
pgsql/src/interfaces/ecpg:
ChangeLog (r1.306 -> r1.307)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ChangeLog.diff?r1=1.306&r2=1.307)
pgsql/src/interfaces/ecpg/ecpglib:
connect.c (r1.29 -> r1.30)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/connect.c.diff?r1=1.29&r2=1.30)
execute.c (r1.47 -> r1.48)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/execute.c.diff?r1=1.47&r2=1.48)
misc.c (r1.26 -> r1.27)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/misc.c.diff?r1=1.26&r2=1.27)

---(end of broadcast)---
TIP 5: don't forget to increase your free space map settings


[COMMITTERS] pgsql: Added some more coverity report patches send in by Martijn van

2006-06-21 Thread Michael Meskes
Log Message:
---
Added some more coverity report patches send in by Martijn van Oosterhout 
.

Tags:

REL8_1_STABLE

Modified Files:
--
pgsql/src/interfaces/ecpg/ecpglib:
connect.c (r1.26.2.3 -> r1.26.2.4)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/connect.c.diff?r1=1.26.2.3&r2=1.26.2.4)
execute.c (r1.43.2.3 -> r1.43.2.4)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/execute.c.diff?r1=1.43.2.3&r2=1.43.2.4)
misc.c (r1.26 -> r1.26.2.1)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/misc.c.diff?r1=1.26&r2=1.26.2.1)

---(end of broadcast)---
TIP 9: In versions below 8.0, the planner will ignore your desire to
   choose an index scan if your joining column's datatypes do not
   match


[COMMITTERS] pgsql: Added some more coverity report patches send in by Martijn van

2006-06-21 Thread Michael Meskes
Log Message:
---
Added some more coverity report patches send in by Martijn van Oosterhout 
.

Tags:

REL8_0_STABLE

Modified Files:
--
pgsql/src/interfaces/ecpg/ecpglib:
connect.c (r1.24.4.4 -> r1.24.4.5)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/connect.c.diff?r1=1.24.4.4&r2=1.24.4.5)
execute.c (r1.38.4.5 -> r1.38.4.6)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/execute.c.diff?r1=1.38.4.5&r2=1.38.4.6)
misc.c (r1.24.4.1 -> r1.24.4.2)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ecpglib/misc.c.diff?r1=1.24.4.1&r2=1.24.4.2)

---(end of broadcast)---
TIP 1: if posting/reading through Usenet, please send an appropriate
   subscribe-nomail command to [EMAIL PROTECTED] so that your
   message can get through to the mailing list cleanly


[COMMITTERS] fulldisjunction - fd: Some more deformed tuple structures consolidation

2006-06-21 Thread User Pgstudy
Log Message:
---
Some more deformed tuple structures consolidation works.

Modified Files:
--
fd:
algutils.c (r1.8 -> r1.9)

(http://cvs.pgfoundry.org/cgi-bin/cvsweb.cgi/fulldisjunction/fd/algutils.c.diff?r1=1.8&r2=1.9)
odmbfd.c (r1.11 -> r1.12)

(http://cvs.pgfoundry.org/cgi-bin/cvsweb.cgi/fulldisjunction/fd/odmbfd.c.diff?r1=1.11&r2=1.12)
queues.c (r1.10 -> r1.11)

(http://cvs.pgfoundry.org/cgi-bin/cvsweb.cgi/fulldisjunction/fd/queues.c.diff?r1=1.10&r2=1.11)
queuesfuncs.h (r1.7 -> r1.8)

(http://cvs.pgfoundry.org/cgi-bin/cvsweb.cgi/fulldisjunction/fd/queuesfuncs.h.diff?r1=1.7&r2=1.8)
tset.c (r1.7 -> r1.8)

(http://cvs.pgfoundry.org/cgi-bin/cvsweb.cgi/fulldisjunction/fd/tset.c.diff?r1=1.7&r2=1.8)
tsetfuncs.h (r1.5 -> r1.6)

(http://cvs.pgfoundry.org/cgi-bin/cvsweb.cgi/fulldisjunction/fd/tsetfuncs.h.diff?r1=1.5&r2=1.6)

---(end of broadcast)---
TIP 1: if posting/reading through Usenet, please send an appropriate
   subscribe-nomail command to [EMAIL PROTECTED] so that your
   message can get through to the mailing list cleanly


[COMMITTERS] pgsql: Clean up psql variable code a little: eliminate unnecessary tests

2006-06-21 Thread Tom Lane
Log Message:
---
Clean up psql variable code a little: eliminate unnecessary tests in
GetVariable() and be consistent about treatment of the list header.
Motivated by noticing strspn() taking an unreasonable percentage of
runtime --- the call removed from GetVariable() was the only one that
could be in a high-usage path ...

Modified Files:
--
pgsql/src/bin/psql:
variables.c (r1.24 -> r1.25)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/variables.c.diff?r1=1.24&r2=1.25)

---(end of broadcast)---
TIP 5: don't forget to increase your free space map settings


[COMMITTERS] pgsql: - During dblink_open, if transaction state was IDLE, force cursor

2006-06-21 Thread Joe Conway
Log Message:
---
- During dblink_open, if transaction state was IDLE, force cursor count to
  initially be 0. This is needed as a previous ABORT might have wiped out
  an automatically opened transaction without maintaining the cursor count.
- Fix regression test expected file for the correct ERROR message, which
  we now get given the above bug fix.

Modified Files:
--
pgsql/contrib/dblink:
dblink.c (r1.55 -> r1.56)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/dblink/dblink.c.diff?r1=1.55&r2=1.56)
pgsql/contrib/dblink/expected:
dblink.out (r1.16 -> r1.17)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/dblink/expected/dblink.out.diff?r1=1.16&r2=1.17)

---(end of broadcast)---
TIP 6: explain analyze is your friend


[COMMITTERS] pgsql: - During dblink_open, if transaction state was IDLE, force cursor

2006-06-21 Thread Joe Conway
Log Message:
---
- During dblink_open, if transaction state was IDLE, force cursor count to
  initially be 0. This is needed as a previous ABORT might have wiped out
  an automatically opened transaction without maintaining the cursor count.
- Fix regression test expected file for the correct ERROR message, which
  we now get given the above bug fix.

Tags:

REL8_1_STABLE

Modified Files:
--
pgsql/contrib/dblink:
dblink.c (r1.48.2.2 -> r1.48.2.3)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/dblink/dblink.c.diff?r1=1.48.2.2&r2=1.48.2.3)
pgsql/contrib/dblink/expected:
dblink.out (r1.16 -> r1.16.2.1)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/dblink/expected/dblink.out.diff?r1=1.16&r2=1.16.2.1)

---(end of broadcast)---
TIP 9: In versions below 8.0, the planner will ignore your desire to
   choose an index scan if your joining column's datatypes do not
   match


[COMMITTERS] pgsql: Eliminate a gratuitously different wording of the 'cannot use

2006-06-21 Thread Tom Lane
Log Message:
---
Eliminate a gratuitously different wording of the 'cannot use aggregate 
function in check constraint' error message.

Modified Files:
--
pgsql/src/backend/commands:
typecmds.c (r1.90 -> r1.91)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/commands/typecmds.c.diff?r1=1.90&r2=1.91)

---(end of broadcast)---
TIP 6: explain analyze is your friend


[COMMITTERS] pgsql: Disallow aggregate functions in UPDATE commands (unless within a

2006-06-21 Thread Tom Lane
Log Message:
---
Disallow aggregate functions in UPDATE commands (unless within a sub-SELECT).
This is disallowed by the SQL spec because it doesn't have any very sensible
interpretation.  Historically Postgres has allowed it but behaved strangely.
As of PG 8.1 a server crash is possible if the MIN/MAX index optimization gets
applied; rather than try to "fix" that, it seems best to just enforce the
spec restriction.  Per report from Josh Drake and Alvaro Herrera.

Modified Files:
--
pgsql/src/backend/parser:
analyze.c (r1.334 -> r1.335)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/parser/analyze.c.diff?r1=1.334&r2=1.335)

---(end of broadcast)---
TIP 2: Don't 'kill -9' the postmaster


[COMMITTERS] pgsql: Disallow aggregate functions in UPDATE commands (unless within a

2006-06-21 Thread Tom Lane
Log Message:
---
Disallow aggregate functions in UPDATE commands (unless within a sub-SELECT).
This is disallowed by the SQL spec because it doesn't have any very sensible
interpretation.  Historically Postgres has allowed it but behaved strangely.
As of PG 8.1 a server crash is possible if the MIN/MAX index optimization gets
applied; rather than try to "fix" that, it seems best to just enforce the
spec restriction.  Per report from Josh Drake and Alvaro Herrera.

Tags:

REL8_1_STABLE

Modified Files:
--
pgsql/src/backend/parser:
analyze.c (r1.326.2.1 -> r1.326.2.2)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/parser/analyze.c.diff?r1=1.326.2.1&r2=1.326.2.2)

---(end of broadcast)---
TIP 9: In versions below 8.0, the planner will ignore your desire to
   choose an index scan if your joining column's datatypes do not
   match


[COMMITTERS] pgsql: Remove ancient kluge that kept nodeAgg.c from crashing on UPDATEs

2006-06-21 Thread Tom Lane
Log Message:
---
Remove ancient kluge that kept nodeAgg.c from crashing on UPDATEs involving
aggregates.  We just disallowed that, and AFAICS there should be no other
cases where direct (non-aggregated) references to input columns are allowed
in a query with aggregation and no GROUP BY.

Modified Files:
--
pgsql/src/backend/executor:
nodeAgg.c (r1.139 -> r1.140)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/nodeAgg.c.diff?r1=1.139&r2=1.140)

---(end of broadcast)---
TIP 5: don't forget to increase your free space map settings


[COMMITTERS] pgsql: Move setup_cancel_handler() up near start of psql main(), where

2006-06-21 Thread Tom Lane
Log Message:
---
Move setup_cancel_handler() up near start of psql main(), where the
setup_win32_locks() call formerly was, to ensure that cancelConnLock is
valid when it needs to be.  Per Yoshiyuki Asaba.

Modified Files:
--
pgsql/src/bin/psql:
startup.c (r1.133 -> r1.134)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/startup.c.diff?r1=1.133&r2=1.134)

---(end of broadcast)---
TIP 6: explain analyze is your friend