pgsql: Reset OpenSSL randomness state in each postmaster child process.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Reset OpenSSL randomness state in each postmaster child process.
Date: 2013-03-27 22:51:27
Message-ID: E1UKzBn-0006cA-HE@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Reset OpenSSL randomness state in each postmaster child process.

Previously, if the postmaster initialized OpenSSL's PRNG (which it will do
when ssl=on in postgresql.conf), the same pseudo-random state would be
inherited by each forked child process. The problem is masked to a
considerable extent if the incoming connection uses SSL encryption, but
when it does not, identical pseudo-random state is made available to
functions like contrib/pgcrypto. The process's PID does get mixed into any
requested random output, but on most systems that still only results in 32K
or so distinct random sequences available across all Postgres sessions.
This might allow an attacker who has database access to guess the results
of "secure" operations happening in another session.

To fix, forcibly reset the PRNG after fork(). Each child process that has
need for random numbers from OpenSSL's generator will thereby be forced to
go through OpenSSL's normal initialization sequence, which should provide
much greater variability of the sequences. There are other ways we might
do this that would be slightly cheaper, but this approach seems the most
future-proof against SSL-related code changes.

This has been assigned CVE-2013-1900, but since the issue and the patch
have already been publicized on pgsql-hackers, there's no point in trying
to hide this commit.

Back-patch to all supported branches.

Marko Kreen

Branch
------
REL8_4_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/a6bd75a12786e3383e8e0b8e873d0d4ca639f7d0

Modified Files
--------------
src/backend/postmaster/fork_process.c | 9 +++++++++
1 files changed, 9 insertions(+), 0 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Michael Paquier 2013-03-28 01:00:43 Re: [COMMITTERS] pgsql: Allow external recovery_config_directory
Previous Message Simon Riggs 2013-03-27 21:37:51 Re: pgsql: Fix pasto which broke docs build.