From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Prevent potential overruns of fixed-size buffers. |
Date: | 2014-02-17 16:21:03 |
Message-ID: | E1WFQwJ-0001xJ-Q2@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Prevent potential overruns of fixed-size buffers.
Coverity identified a number of places in which it couldn't prove that a
string being copied into a fixed-size buffer would fit. We believe that
most, perhaps all of these are in fact safe, or are copying data that is
coming from a trusted source so that any overrun is not really a security
issue. Nonetheless it seems prudent to forestall any risk by using
strlcpy() and similar functions.
Fixes by Peter Eisentraut and Jozef Mlich based on Coverity reports.
In addition, fix a potential null-pointer-dereference crash in
contrib/chkpass. The crypt(3) function is defined to return NULL on
failure, but chkpass.c didn't check for that before using the result.
The main practical case in which this could be an issue is if libc is
configured to refuse to execute unapproved hashing algorithms (e.g.,
"FIPS mode"). This ideally should've been a separate commit, but
since it touches code adjacent to one of the buffer overrun changes,
I included it in this commit to avoid last-minute merge issues.
This issue was reported by Honza Horak.
Security: CVE-2014-0065 for buffer overruns, CVE-2014-0066 for crypt()
Branch
------
REL9_3_STABLE
Details
-------
http://git.postgresql.org/pg/commitdiff/e3208fec32586076cd1a24cae19ded158a60e15a
Modified Files
--------------
contrib/chkpass/chkpass.c | 29 ++++++++++++++++++++++++++---
contrib/pg_standby/pg_standby.c | 2 +-
src/backend/access/transam/xlog.c | 8 ++++----
src/backend/tsearch/spell.c | 2 +-
src/backend/utils/adt/datetime.c | 11 ++++++-----
src/bin/initdb/findtimezone.c | 4 ++--
src/bin/pg_basebackup/pg_basebackup.c | 8 ++++----
src/interfaces/ecpg/preproc/pgc.l | 2 +-
src/interfaces/libpq/fe-protocol2.c | 2 +-
src/interfaces/libpq/fe-protocol3.c | 2 +-
src/port/exec.c | 4 ++--
src/test/regress/pg_regress.c | 6 +++---
src/timezone/pgtz.c | 2 +-
13 files changed, 53 insertions(+), 29 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2014-02-17 16:21:04 | pgsql: Prevent potential overruns of fixed-size buffers. |
Previous Message | Noah Misch | 2014-02-17 15:05:48 | pgsql: Fix handling of wide datetime input/output. |