~ubuntu-branches/ubuntu/oneiric/postgresql-9.1/oneiric-security

« back to all changes in this revision

Viewing changes to src/bin/pg_dump/common.c

  • Committer: Package Import Robot
  • Author(s): Martin Pitt
  • Date: 2012-02-27 14:48:01 UTC
  • mfrom: (1.1.7) (10.1.3 oneiric-proposed)
  • Revision ID: package-import@ubuntu.com-20120227144801-7a94i4eksyukp22k
Tags: 9.1.3-0ubuntu0.11.10
* New upstream security/bug fix release: (LP: #941912)
  - Require execute permission on the trigger function for "CREATE
    TRIGGER".
    This missing check could allow another user to execute a trigger
    function with forged input data, by installing it on a table he
    owns. This is only of significance for trigger functions marked
    SECURITY DEFINER, since otherwise trigger functions run as the
    table owner anyway. (CVE-2012-0866)
  - Remove arbitrary limitation on length of common name in SSL
    certificates.
    Both libpq and the server truncated the common name extracted from
    an SSL certificate at 32 bytes. Normally this would cause nothing
    worse than an unexpected verification failure, but there are some
    rather-implausible scenarios in which it might allow one
    certificate holder to impersonate another. The victim would have to
    have a common name exactly 32 bytes long, and the attacker would
    have to persuade a trusted CA to issue a certificate in which the
    common name has that string as a prefix. Impersonating a server
    would also require some additional exploit to redirect client
    connections. (CVE-2012-0867)
  - Convert newlines to spaces in names written in pg_dump comments.
    pg_dump was incautious about sanitizing object names that are
    emitted within SQL comments in its output script. A name containing
    a newline would at least render the script syntactically incorrect.
    Maliciously crafted object names could present a SQL injection risk
    when the script is reloaded. (CVE-2012-0868)
  - Fix btree index corruption from insertions concurrent with
    vacuuming.
    An index page split caused by an insertion could sometimes cause a
    concurrently-running "VACUUM" to miss removing index entries that
    it should remove. After the corresponding table rows are removed,
    the dangling index entries would cause errors (such as "could not
    read block N in file ...") or worse, silently wrong query results
    after unrelated rows are re-inserted at the now-free table
    locations. This bug has been present since release 8.2, but occurs
    so infrequently that it was not diagnosed until now. If you have
    reason to suspect that it has happened in your database, reindexing
    the affected index will fix things.
  - Fix transient zeroing of shared buffers during WAL replay.
    The replay logic would sometimes zero and refill a shared buffer,
    so that the contents were transiently invalid. In hot standby mode
    this can result in a query that's executing in parallel seeing
    garbage data. Various symptoms could result from that, but the most
    common one seems to be "invalid memory alloc request size".
  - Fix handling of data-modifying WITH subplans in READ COMMITTED
    rechecking.
    A WITH clause containing "INSERT"/"UPDATE"/"DELETE" would crash if
    the parent "UPDATE" or "DELETE" command needed to be re-evaluated
    at one or more rows due to concurrent updates in READ COMMITTED
    mode.
  - Fix corner case in SSI transaction cleanup.
    When finishing up a read-write serializable transaction, a crash
    could occur if all remaining active serializable transactions are
    read-only.
  - Fix postmaster to attempt restart after a hot-standby crash.
    A logic error caused the postmaster to terminate, rather than
    attempt to restart the cluster, if any backend process crashed
    while operating in hot standby mode.
  - Fix "CLUSTER"/"VACUUM FULL" handling of toast values owned by
    recently-updated rows.
    This oversight could lead to "duplicate key value violates unique
    constraint" errors being reported against the toast table's index
    during one of these commands.
  - Update per-column permissions, not only per-table permissions, when
    changing table owner.
    Failure to do this meant that any previously granted column
    permissions were still shown as having been granted by the old
    owner. This meant that neither the new owner nor a superuser could
    revoke the now-untraceable-to-table-owner permissions.
  - Support foreign data wrappers and foreign servers in "REASSIGN
    OWNED".
    This command failed with "unexpected classid" errors if it needed
    to change the ownership of any such objects.
  - Allow non-existent values for some settings in "ALTER USER/DATABASE
    SET".
    Allow default_text_search_config, default_tablespace, and
    temp_tablespaces to be set to names that are not known. This is
    because they might be known in another database where the setting
    is intended to be used, or for the tablespace cases because the
    tablespace might not be created yet. The same issue was previously
    recognized for search_path, and these settings now act like that
    one.
  - Fix "unsupported node type" error caused by COLLATE in an "INSERT"
    expression.
  - Avoid crashing when we have problems deleting table files
    post-commit.
    Dropping a table should lead to deleting the underlying disk files
    only after the transaction commits. In event of failure then (for
    instance, because of wrong file permissions) the code is supposed
    to just emit a warning message and go on, since it's too late to
    abort the transaction. This logic got broken as of release 8.4,
    causing such situations to result in a PANIC and an unrestartable
    database.
  - Recover from errors occurring during WAL replay of "DROP
    TABLESPACE".
    Replay will attempt to remove the tablespace's directories, but
    there are various reasons why this might fail (for example,
    incorrect ownership or permissions on those directories). Formerly
    the replay code would panic, rendering the database unrestartable
    without manual intervention. It seems better to log the problem and
    continue, since the only consequence of failure to remove the
    directories is some wasted disk space.
  - Fix race condition in logging AccessExclusiveLocks for hot standby.
    Sometimes a lock would be logged as being held by "transaction
    zero". This is at least known to produce assertion failures on
    slave servers, and might be the cause of more serious problems.
  - Track the OID counter correctly during WAL replay, even when it
    wraps around.
  - Prevent emitting misleading "consistent recovery state reached" log
    message at the beginning of crash recovery.
  - Fix initial value of pg_stat_replication.replay_location.
  - Fix regular expression back-references with - attached.
    Rather than enforcing an exact string match, the code would
    effectively accept any string that satisfies the pattern
    sub-expression referenced by the back-reference symbol.
    A similar problem still afflicts back-references that are embedded
    in a larger quantified expression, rather than being the immediate
    subject of the quantifier. This will be addressed in a future
    PostgreSQL release.
  - Fix recently-introduced memory leak in processing of inet/cidr
    values.
  - Fix planner's ability to push down index-expression restrictions
    through UNION ALL.
  - Fix planning of WITH clauses referenced in "UPDATE"/"DELETE" on an
    inherited table.
    This bug led to "could not find plan for CTE" failures.
  - Fix GIN cost estimation to handle column IN (...) index conditions.
    This oversight would usually lead to crashes if such a condition
    could be used with a GIN index.
  - Fix dangling pointer after "CREATE TABLE AS"/"SELECT INTO" in a
    SQL-language function.
    In most cases this only led to an assertion failure in
    assert-enabled builds, but worse consequences seem possible.
  - Fix I/O-conversion-related memory leaks in plpgsql.
  - Work around bug in perl's SvPVutf8() function.
    This function crashes when handed a typeglob or certain read-only
    objects such as $^V. Make plperl avoid passing those to it.
  - In pg_dump, don't dump contents of an extension's configuration
    tables if the extension itself is not being dumped.
  - Improve pg_dump's handling of inherited table columns.
    pg_dump mishandled situations where a child column has a different
    default expression than its parent column. If the default is
    textually identical to the parent's default, but not actually the
    same (for instance, because of schema search path differences) it
    would not be recognized as different, so that after dump and
    restore the child would be allowed to inherit the parent's default.
    Child columns that are NOT NULL where their parent is not could
    also be restored subtly incorrectly.
  - Fix pg_restore's direct-to-database mode for INSERT-style table data.
    Direct-to-database restores from archive files made with
    "--inserts" or "--column-inserts" options fail when using
    pg_restore from a release dated September or December 2011, as a
    result of an oversight in a fix for another problem. The archive
    file itself is not at fault, and text-mode output is okay.
  - Teach pg_upgrade to handle renaming of plpython's shared library.
    Upgrading a pre-9.1 database that included plpython would fail
    because of this oversight.
  - Allow pg_upgrade to process tables containing regclass columns.
    Since pg_upgrade now takes care to preserve pg_class OIDs, there
    was no longer any reason for this restriction.
  - Make libpq ignore ENOTDIR errors when looking for an SSL client
    certificate file.
    This allows SSL connections to be established, though without a
    certificate, even when the user's home directory is set to
    something like /dev/null.
  - Fix some more field alignment issues in ecpg's SQLDA area.
  - Allow AT option in ecpg DEALLOCATE statements.
    The infrastructure to support this has been there for awhile, but
    through an oversight there was still an error check rejecting the
    case.
  - Do not use the variable name when defining a varchar structure in
    ecpg.
  - Fix "contrib/auto_explain"'s JSON output mode to produce valid JSON.
  - Fix error in "contrib/intarray"'s int[] & int[] operator.
    If the smallest integer the two input arrays have in common is 1,
    and there are smaller values in either array, then 1 would be
    incorrectly omitted from the result.
  - Fix error detection in "contrib/pgcrypto"'s encrypt_iv() and
    decrypt_iv().
    These functions failed to report certain types of invalid-input
    errors, and would instead return random garbage values for
    incorrect input.
  - Fix one-byte buffer overrun in "contrib/test_parser".
    The code would try to read one more byte than it should, which
    would crash in corner cases. Since "contrib/test_parser" is only
    example code, this is not a security issue in itself, but bad
    example code is still bad.
  - Use __sync_lock_test_and_set() for spinlocks on ARM, if available.
    This function replaces our previous use of the SWPB instruction,
    which is deprecated and not available on ARMv6 and later. Reports
    suggest that the old code doesn't fail in an obvious way on recent
    ARM boards, but simply doesn't interlock concurrent accesses,
    leading to bizarre failures in multiprocess operation.
  - Use "-fexcess-precision=standard" option when building with gcc
    versions that accept it.
    This prevents assorted scenarios wherein recent versions of gcc
    will produce creative results.
  - Allow use of threaded Python on FreeBSD (Chris Rees)
    Our configure script previously believed that this combination
    wouldn't work; but FreeBSD fixed the problem, so remove that error
    check.
* Drop 00git_inet_cidr_unpack.patch, 01-armel-tas.patch: Applied upstream.

Show diffs side-by-side

added added

removed removed

Lines of Context:
125
125
        funinfo = getFuncs(&numFuncs);
126
126
        funinfoindex = buildIndexArray(funinfo, numFuncs, sizeof(FuncInfo));
127
127
 
128
 
        /* this must be after getFuncs */
 
128
        /* this must be after getTables and getFuncs */
129
129
        if (g_verbose)
130
130
                write_msg(NULL, "reading user-defined types\n");
131
131
        typinfo = getTypes(&numTypes);
283
283
 
284
284
/* flagInhAttrs -
285
285
 *       for each dumpable table in tblinfo, flag its inherited attributes
286
 
 * so when we dump the table out, we don't dump out the inherited attributes
 
286
 *
 
287
 * What we need to do here is detect child columns that inherit NOT NULL
 
288
 * bits from their parents (so that we needn't specify that again for the
 
289
 * child) and child columns that have DEFAULT NULL when their parents had
 
290
 * some non-null default.  In the latter case, we make up a dummy AttrDefInfo
 
291
 * object so that we'll correctly emit the necessary DEFAULT NULL clause;
 
292
 * otherwise the backend will apply an inherited default to the column.
287
293
 *
288
294
 * modifies tblinfo
289
295
 */
299
305
                TableInfo  *tbinfo = &(tblinfo[i]);
300
306
                int                     numParents;
301
307
                TableInfo **parents;
302
 
                TableInfo  *parent;
303
308
 
304
309
                /* Sequences and views never have parents */
305
310
                if (tbinfo->relkind == RELKIND_SEQUENCE ||
316
321
                if (numParents == 0)
317
322
                        continue;                       /* nothing to see here, move along */
318
323
 
319
 
                /*----------------------------------------------------------------
320
 
                 * For each attr, check the parent info: if no parent has an attr
321
 
                 * with the same name, then it's not inherited. If there *is* an
322
 
                 * attr with the same name, then only dump it if:
323
 
                 *
324
 
                 * - it is NOT NULL and zero parents are NOT NULL
325
 
                 *       OR
326
 
                 * - it has a default value AND the default value does not match
327
 
                 *       all parent default values, or no parents specify a default.
328
 
                 *
329
 
                 * See discussion on -hackers around 2-Apr-2001.
330
 
                 *----------------------------------------------------------------
331
 
                 */
 
324
                /* For each column, search for matching column names in parent(s) */
332
325
                for (j = 0; j < tbinfo->numatts; j++)
333
326
                {
334
 
                        bool            foundAttr;              /* Attr was found in a parent */
335
327
                        bool            foundNotNull;   /* Attr was NOT NULL in a parent */
336
 
                        bool            defaultsMatch;  /* All non-empty defaults match */
337
 
                        bool            defaultsFound;  /* Found a default in a parent */
338
 
                        AttrDefInfo *attrDef;
339
 
 
340
 
                        foundAttr = false;
 
328
                        bool            foundDefault;   /* Found a default in a parent */
 
329
 
 
330
                        /* no point in examining dropped columns */
 
331
                        if (tbinfo->attisdropped[j])
 
332
                                continue;
 
333
 
341
334
                        foundNotNull = false;
342
 
                        defaultsMatch = true;
343
 
                        defaultsFound = false;
344
 
 
345
 
                        attrDef = tbinfo->attrdefs[j];
346
 
 
 
335
                        foundDefault = false;
347
336
                        for (k = 0; k < numParents; k++)
348
337
                        {
 
338
                                TableInfo  *parent = parents[k];
349
339
                                int                     inhAttrInd;
350
340
 
351
 
                                parent = parents[k];
352
341
                                inhAttrInd = strInArray(tbinfo->attnames[j],
353
342
                                                                                parent->attnames,
354
343
                                                                                parent->numatts);
355
 
 
356
 
                                if (inhAttrInd != -1)
 
344
                                if (inhAttrInd >= 0)
357
345
                                {
358
 
                                        AttrDefInfo *inhDef = parent->attrdefs[inhAttrInd];
359
 
 
360
 
                                        foundAttr = true;
361
346
                                        foundNotNull |= parent->notnull[inhAttrInd];
362
 
                                        if (inhDef != NULL)
363
 
                                        {
364
 
                                                defaultsFound = true;
365
 
 
366
 
                                                /*
367
 
                                                 * If any parent has a default and the child doesn't,
368
 
                                                 * we have to emit an explicit DEFAULT NULL clause for
369
 
                                                 * the child, else the parent's default will win.
370
 
                                                 */
371
 
                                                if (attrDef == NULL)
372
 
                                                {
373
 
                                                        attrDef = (AttrDefInfo *) malloc(sizeof(AttrDefInfo));
374
 
                                                        attrDef->dobj.objType = DO_ATTRDEF;
375
 
                                                        attrDef->dobj.catId.tableoid = 0;
376
 
                                                        attrDef->dobj.catId.oid = 0;
377
 
                                                        AssignDumpId(&attrDef->dobj);
378
 
                                                        attrDef->adtable = tbinfo;
379
 
                                                        attrDef->adnum = j + 1;
380
 
                                                        attrDef->adef_expr = strdup("NULL");
381
 
 
382
 
                                                        attrDef->dobj.name = strdup(tbinfo->dobj.name);
383
 
                                                        attrDef->dobj.namespace = tbinfo->dobj.namespace;
384
 
 
385
 
                                                        attrDef->dobj.dump = tbinfo->dobj.dump;
386
 
 
387
 
                                                        attrDef->separate = false;
388
 
                                                        addObjectDependency(&tbinfo->dobj,
389
 
                                                                                                attrDef->dobj.dumpId);
390
 
 
391
 
                                                        tbinfo->attrdefs[j] = attrDef;
392
 
                                                }
393
 
                                                if (strcmp(attrDef->adef_expr, inhDef->adef_expr) != 0)
394
 
                                                {
395
 
                                                        defaultsMatch = false;
396
 
 
397
 
                                                        /*
398
 
                                                         * Whenever there is a non-matching parent
399
 
                                                         * default, add a dependency to force the parent
400
 
                                                         * default to be dumped first, in case the
401
 
                                                         * defaults end up being dumped as separate
402
 
                                                         * commands.  Otherwise the parent default will
403
 
                                                         * override the child's when it is applied.
404
 
                                                         */
405
 
                                                        addObjectDependency(&attrDef->dobj,
406
 
                                                                                                inhDef->dobj.dumpId);
407
 
                                                }
408
 
                                        }
 
347
                                        foundDefault |= (parent->attrdefs[inhAttrInd] != NULL);
409
348
                                }
410
349
                        }
411
350
 
412
 
                        /*
413
 
                         * Based on the scan of the parents, decide if we can rely on the
414
 
                         * inherited attr
415
 
                         */
416
 
                        if (foundAttr)          /* Attr was inherited */
 
351
                        /* Remember if we found inherited NOT NULL */
 
352
                        tbinfo->inhNotNull[j] = foundNotNull;
 
353
 
 
354
                        /* Manufacture a DEFAULT NULL clause if necessary */
 
355
                        if (foundDefault && tbinfo->attrdefs[j] == NULL)
417
356
                        {
418
 
                                /* Set inherited flag by default */
419
 
                                tbinfo->inhAttrs[j] = true;
420
 
                                tbinfo->inhAttrDef[j] = true;
421
 
                                tbinfo->inhNotNull[j] = true;
422
 
 
423
 
                                /*
424
 
                                 * Clear it if attr had a default, but parents did not, or
425
 
                                 * mismatch
426
 
                                 */
427
 
                                if ((attrDef != NULL) && (!defaultsFound || !defaultsMatch))
428
 
                                {
429
 
                                        tbinfo->inhAttrs[j] = false;
430
 
                                        tbinfo->inhAttrDef[j] = false;
431
 
                                }
432
 
 
433
 
                                /*
434
 
                                 * Clear it if NOT NULL and none of the parents were NOT NULL
435
 
                                 */
436
 
                                if (tbinfo->notnull[j] && !foundNotNull)
437
 
                                {
438
 
                                        tbinfo->inhAttrs[j] = false;
439
 
                                        tbinfo->inhNotNull[j] = false;
440
 
                                }
441
 
 
442
 
                                /* Clear it if attr has local definition */
443
 
                                if (tbinfo->attislocal[j])
444
 
                                        tbinfo->inhAttrs[j] = false;
 
357
                                AttrDefInfo *attrDef;
 
358
 
 
359
                                attrDef = (AttrDefInfo *) malloc(sizeof(AttrDefInfo));
 
360
                                attrDef->dobj.objType = DO_ATTRDEF;
 
361
                                attrDef->dobj.catId.tableoid = 0;
 
362
                                attrDef->dobj.catId.oid = 0;
 
363
                                AssignDumpId(&attrDef->dobj);
 
364
                                attrDef->dobj.name = strdup(tbinfo->dobj.name);
 
365
                                attrDef->dobj.namespace = tbinfo->dobj.namespace;
 
366
                                attrDef->dobj.dump = tbinfo->dobj.dump;
 
367
 
 
368
                                attrDef->adtable = tbinfo;
 
369
                                attrDef->adnum = j + 1;
 
370
                                attrDef->adef_expr = strdup("NULL");
 
371
 
 
372
                                /* Will column be dumped explicitly? */
 
373
                                if (shouldPrintColumn(tbinfo, j))
 
374
                                {
 
375
                                        attrDef->separate = false;
 
376
                                        /* No dependency needed: NULL cannot have dependencies */
 
377
                                }
 
378
                                else
 
379
                                {
 
380
                                        /* column will be suppressed, print default separately */
 
381
                                        attrDef->separate = true;
 
382
                                        /* ensure it comes out after the table */
 
383
                                        addObjectDependency(&attrDef->dobj,
 
384
                                                                                tbinfo->dobj.dumpId);
 
385
                                }
 
386
 
 
387
                                tbinfo->attrdefs[j] = attrDef;
445
388
                        }
446
389
                }
447
390
        }