pgsql: Fix race condition in pg_ctl reading postmaster.pid. - Mailing list pgsql-committers

From Heikki Linnakangas
Subject pgsql: Fix race condition in pg_ctl reading postmaster.pid.
Date
Msg-id E1TNfac-0008O0-SG@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix race condition in pg_ctl reading postmaster.pid.

If postmaster changed postmaster.pid while pg_ctl was reading it, pg_ctl
could overrun the buffer it allocated for the file. Fix by reading the
whole file to memory with one read() call.

initdb contains an identical copy of the readfile() function, but the files
that initdb reads are static, not modified concurrently. Nevertheless, add
a simple bounds-check there, if only to silence static analysis tools.

Per report from Dave Vitek. Backpatch to all supported branches.

Branch
------
REL9_0_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/1c95b5eea6bec742d5d6ee097c55690712dd3be6

Modified Files
--------------
src/bin/initdb/initdb.c |    9 +++--
src/bin/pg_ctl/pg_ctl.c |   88 +++++++++++++++++++++++++++++++----------------
2 files changed, 63 insertions(+), 34 deletions(-)


pgsql-committers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: pgsql: Fix race condition in pg_ctl reading postmaster.pid.
Next
From: Heikki Linnakangas
Date:
Subject: pgsql: Fix race condition in pg_ctl reading postmaster.pid.