~ubuntu-branches/ubuntu/quantal/postgresql-9.1/quantal-updates

« back to all changes in this revision

Viewing changes to src/backend/tcop/postgres.c

  • Committer: Package Import Robot
  • Author(s): Martin Pitt
  • Date: 2012-06-04 06:47:45 UTC
  • mfrom: (1.1.8)
  • Revision ID: package-import@ubuntu.com-20120604064745-dvs00jnur0vgkwqs
Tags: 9.1.4-1
* Urgency medium due to security fixes.
* New upstream bug fix/security release:
  - Fix incorrect password transformation in "contrib/pgcrypto"'s DES
    crypt() function.
    If a password string contained the byte value 0x80, the remainder
    of the password was ignored, causing the password to be much weaker
    than it appeared. With this fix, the rest of the string is properly
    included in the DES hash. Any stored password values that are
    affected by this bug will thus no longer match, so the stored
    values may need to be updated. (CVE-2012-2143)
  - Ignore SECURITY DEFINER and SET attributes for a procedural
    language's call handler. Applying such attributes to a call handler
    could crash the server. (CVE-2012-2655)
  - Make "contrib/citext"'s upgrade script fix collations of citext
    arrays and domains over citext.
    Release 9.1.2 provided a fix for collations of citext columns and
    indexes in databases upgraded or reloaded from pre-9.1
    installations, but that fix was incomplete: it neglected to handle
    arrays and domains over citext. This release extends the module's
    upgrade script to handle these cases. As before, if you have
    already run the upgrade script, you'll need to run the collation
    update commands by hand instead. See the 9.1.2 release notes for
    more information about doing this.
  - Allow numeric timezone offsets in timestamp input to be up to 16
    hours away from UTC. Some historical time zones have offsets larger than
    15 hours, the previous limit. This could result in dumped data values
    being rejected during reload.
  - Fix timestamp conversion to cope when the given time is exactly the
    last DST transition time for the current timezone.
    This oversight has been there a long time, but was not noticed
    previously because most DST-using zones are presumed to have an
    indefinite sequence of future DST transitions.
  - Fix text to name and char to name casts to perform string
    truncation correctly in multibyte encodings.
  - Fix memory copying bug in to_tsquery().
  - Ensure txid_current() reports the correct epoch when executed in
    hot standby.
  - Fix planner's handling of outer PlaceHolderVars within subqueries.
    This bug concerns sub-SELECTs that reference variables coming from
    the nullable side of an outer join of the surrounding query. In
    9.1, queries affected by this bug would fail with "ERROR:
    Upper-level PlaceHolderVar found where not expected". But in 9.0
    and 8.4, you'd silently get possibly-wrong answers, since the value
    transmitted into the subquery wouldn't go to null when it should.
  - Fix planning of UNION ALL subqueries with output columns that are
    not simple variables.
    Planning of such cases got noticeably worse in 9.1 as a result of a
    misguided fix for "MergeAppend child's targetlist doesn't match
    MergeAppend" errors. Revert that fix and do it another way.
  - Fix slow session startup when pg_attribute is very large.
    If pg_attribute exceeds one-fourth of shared_buffers, cache
    rebuilding code that is sometimes needed during session start would
    trigger the synchronized-scan logic, causing it to take many times
    longer than normal. The problem was particularly acute if many new
    sessions were starting at once.
  - Ensure sequential scans check for query cancel reasonably often.
    A scan encountering many consecutive pages that contain no live
    tuples would not respond to interrupts meanwhile.
  - Ensure the Windows implementation of PGSemaphoreLock() clears
    ImmediateInterruptOK before returning.
    This oversight meant that a query-cancel interrupt received later
    in the same query could be accepted at an unsafe time, with
    unpredictable but not good consequences.
  - Show whole-row variables safely when printing views or rules.
    Corner cases involving ambiguous names (that is, the name could be
    either a table or column name of the query) were printed in an
    ambiguous way, risking that the view or rule would be interpreted
    differently after dump and reload. Avoid the ambiguous case by
    attaching a no-op cast.
  - Fix "COPY FROM" to properly handle null marker strings that
    correspond to invalid encoding.
    A null marker string such as E'\\0' should work, and did work in
    the past, but the case got broken in 8.4.
  - Fix "EXPLAIN VERBOSE" for writable CTEs containing RETURNING
    clauses.
  - Fix "PREPARE TRANSACTION" to work correctly in the presence of
    advisory locks.
    Historically, "PREPARE TRANSACTION" has simply ignored any
    session-level advisory locks the session holds, but this case was
    accidentally broken in 9.1.
  - Fix truncation of unlogged tables.
  - Ignore missing schemas during non-interactive assignments of
    search_path.
    This re-aligns 9.1's behavior with that of older branches.
    Previously 9.1 would throw an error for nonexistent schemas
    mentioned in search_path settings obtained from places such as
    "ALTER DATABASE SET".
  - Fix bugs with temporary or transient tables used in extension
    scripts.
    This includes cases such as a rewriting "ALTER TABLE" within an
    extension update script, since that uses a transient table behind
    the scenes.
  - Ensure autovacuum worker processes perform stack depth checking
    properly.
    Previously, infinite recursion in a function invoked by
    auto-"ANALYZE" could crash worker processes.
  - Fix logging collector to not lose log coherency under high load.
    The collector previously could fail to reassemble large messages if
    it got too busy.
  - Fix logging collector to ensure it will restart file rotation after
    receiving SIGHUP.
  - Fix "too many LWLocks taken" failure in GiST indexes.
  - Fix WAL replay logic for GIN indexes to not fail if the index was
    subsequently dropped.
  - Correctly detect SSI conflicts of prepared transactions after a
    crash.
  - Avoid synchronous replication delay when committing a transaction
    that only modified temporary tables.
    In such a case the transaction's commit record need not be flushed
    to standby servers, but some of the code didn't know that and
    waited for it to happen anyway.
  - Fix error handling in pg_basebackup.
  - Fix walsender to not go into a busy loop if connection is
    terminated.
  - Fix memory leak in PL/pgSQL's "RETURN NEXT" command.
  - Fix PL/pgSQL's "GET DIAGNOSTICS" command when the target is the
    function's first variable.
  - Ensure that PL/Perl package-qualifies the _TD variable.
    This bug caused trigger invocations to fail when they are nested
    within a function invocation that changes the current package.
  - Fix PL/Python functions returning composite types to accept a
    string for their result value.
    This case was accidentally broken by the 9.1 additions to allow a
    composite result value to be supplied in other formats, such as
    dictionaries.
  - Fix potential access off the end of memory in psql's expanded
    display ("\x") mode.
  - Fix several performance problems in pg_dump when the database
    contains many objects.
    pg_dump could get very slow if the database contained many schemas,
    or if many objects are in dependency loops, or if there are many
    owned sequences.
  - Fix memory and file descriptor leaks in pg_restore when reading a
    directory-format archive.
  - Fix pg_upgrade for the case that a database stored in a non-default
    tablespace contains a table in the cluster's default tablespace.
  - In ecpg, fix rare memory leaks and possible overwrite of one byte
    after the sqlca_t structure.
  - Fix "contrib/dblink"'s dblink_exec() to not leak temporary database
    connections upon error.
  - Fix "contrib/dblink" to report the correct connection name in error
    messages.
  - Fix "contrib/vacuumlo" to use multiple transactions when dropping
    many large objects.
    This change avoids exceeding max_locks_per_transaction when many
    objects need to be dropped. The behavior can be adjusted with the
    new -l (limit) option.
* debian/control: Bump debhelper build dependency to >= 8, as it does not
  build with earlier versions.
* debian/control: Move bzr branches to alioth, so that other members of
  pkg-postgresql can commit. Update Vcs-* tags.
* debian/control: Set Maintainer: to pkg-postgresql group, and move myself
  to Uploaders:.

Show diffs side-by-side

added added

removed removed

Lines of Context:
114
114
static long max_stack_depth_bytes = 100 * 1024L;
115
115
 
116
116
/*
117
 
 * Stack base pointer -- initialized by PostgresMain. This is not static
118
 
 * so that PL/Java can modify it.
 
117
 * Stack base pointer -- initialized by PostmasterMain and inherited by
 
118
 * subprocesses. This is not static because old versions of PL/Java modify
 
119
 * it directly. Newer versions use set_stack_base(), but we want to stay
 
120
 * binary-compatible for the time being.
119
121
 */
120
122
char       *stack_base_ptr = NULL;
121
123
 
3003
3005
 
3004
3006
 
3005
3007
/*
 
3008
 * set_stack_base: set up reference point for stack depth checking
 
3009
 *
 
3010
 * Returns the old reference point, if any.
 
3011
 */
 
3012
pg_stack_base_t
 
3013
set_stack_base(void)
 
3014
{
 
3015
        char            stack_base;
 
3016
        pg_stack_base_t old;
 
3017
 
 
3018
#if defined(__ia64__) || defined(__ia64)
 
3019
        old.stack_base_ptr = stack_base_ptr;
 
3020
        old.register_stack_base_ptr = register_stack_base_ptr;
 
3021
#else
 
3022
        old = stack_base_ptr;
 
3023
#endif
 
3024
 
 
3025
        /* Set up reference point for stack depth checking */
 
3026
        stack_base_ptr = &stack_base;
 
3027
#if defined(__ia64__) || defined(__ia64)
 
3028
        register_stack_base_ptr = ia64_get_bsp();
 
3029
#endif
 
3030
 
 
3031
        return old;
 
3032
}
 
3033
 
 
3034
/*
 
3035
 * restore_stack_base: restore reference point for stack depth checking
 
3036
 *
 
3037
 * This can be used after set_stack_base() to restore the old value. This
 
3038
 * is currently only used in PL/Java. When PL/Java calls a backend function
 
3039
 * from different thread, the thread's stack is at a different location than
 
3040
 * the main thread's stack, so it sets the base pointer before the call, and
 
3041
 * restores it afterwards.
 
3042
 */
 
3043
void
 
3044
restore_stack_base(pg_stack_base_t base)
 
3045
{
 
3046
#if defined(__ia64__) || defined(__ia64)
 
3047
        stack_base_ptr = base.stack_base_ptr;
 
3048
        register_stack_base_ptr = base.register_stack_base_ptr;
 
3049
#else
 
3050
        stack_base_ptr = base;
 
3051
#endif
 
3052
}
 
3053
 
 
3054
/*
3006
3055
 * check_stack_depth: check for excessively deep recursion
3007
3056
 *
3008
3057
 * This should be called someplace in any recursive routine that might possibly
3017
3066
        long            stack_depth;
3018
3067
 
3019
3068
        /*
3020
 
         * Compute distance from PostgresMain's local variables to my own
 
3069
         * Compute distance from reference point to to my local variables
3021
3070
         */
3022
3071
        stack_depth = (long) (stack_base_ptr - &stack_top_loc);
3023
3072
 
3459
3508
{
3460
3509
        const char *dbname;
3461
3510
        int                     firstchar;
3462
 
        char            stack_base;
3463
3511
        StringInfoData input_message;
3464
3512
        sigjmp_buf      local_sigjmp_buf;
3465
3513
        volatile bool send_ready_for_query = true;
3485
3533
 
3486
3534
        SetProcessingMode(InitProcessing);
3487
3535
 
3488
 
        /* Set up reference point for stack depth checking */
3489
 
        stack_base_ptr = &stack_base;
3490
 
#if defined(__ia64__) || defined(__ia64)
3491
 
        register_stack_base_ptr = ia64_get_bsp();
3492
 
#endif
3493
 
 
3494
3536
        /* Compute paths, if we didn't inherit them from postmaster */
3495
3537
        if (my_exec_path[0] == '\0')
3496
3538
        {