diff options
author | Robert Haas | 2015-11-12 14:00:33 +0000 |
---|---|---|
committer | Robert Haas | 2015-11-12 14:00:33 +0000 |
commit | ac1d7945f866b1928c2554c0f80fd52d7f977772 (patch) | |
tree | 7988b7964e59c7fe685d2da3de12ef87d25fc23c | |
parent | a05dc4d7fd57d4ae084c1f0801973e5c1a1aa26e (diff) |
Make idle backends exit if the postmaster dies.
Letting backends continue to run if the postmaster has exited prevents
PostgreSQL from being restarted, which in many environments is
catastrophic. Worse, if some other backend crashes, we no longer have
any protection against shared memory corruption. So, arrange for them
to exit instead. We don't want to expend many cycles on this, but
including postmaster death in the set of things that we wait for when
a backend is idle seems cheap enough.
Rajeev Rastogi and Robert Haas
-rw-r--r-- | src/backend/libpq/be-secure.c | 33 |
1 files changed, 31 insertions, 2 deletions
diff --git a/src/backend/libpq/be-secure.c b/src/backend/libpq/be-secure.c index 26d8faaf77..2ddcf428f8 100644 --- a/src/backend/libpq/be-secure.c +++ b/src/backend/libpq/be-secure.c @@ -35,6 +35,7 @@ #include "miscadmin.h" #include "tcop/tcopprot.h" #include "utils/memutils.h" +#include "storage/ipc.h" #include "storage/proc.h" @@ -144,9 +145,31 @@ retry: Assert(waitfor); w = WaitLatchOrSocket(MyLatch, - WL_LATCH_SET | waitfor, + WL_LATCH_SET | WL_POSTMASTER_DEATH | waitfor, port->sock, 0); + /* + * If the postmaster has died, it's not safe to continue running, + * because it is the postmaster's job to kill us if some other backend + * exists uncleanly. Moreover, we won't run very well in this state; + * helper processes like walwriter and the bgwriter will exit, so + * performance may be poor. Finally, if we don't exit, pg_ctl will + * be unable to restart the postmaster without manual intervention, + * so no new connections can be accepted. Exiting clears the deck + * for a postmaster restart. + * + * (Note that we only make this check when we would otherwise sleep + * on our latch. We might still continue running for a while if the + * postmaster is killed in mid-query, or even through multiple queries + * if we never have to wait for read. We don't want to burn too many + * cycles checking for this very rare condition, and this should cause + * us to exit quickly in most cases.) + */ + if (w & WL_POSTMASTER_DEATH) + ereport(FATAL, + (errcode(ERRCODE_ADMIN_SHUTDOWN), + errmsg("terminating connection due to unexpected postmaster exit"))); + /* Handle interrupt. */ if (w & WL_LATCH_SET) { @@ -223,9 +246,15 @@ retry: Assert(waitfor); w = WaitLatchOrSocket(MyLatch, - WL_LATCH_SET | waitfor, + WL_LATCH_SET | WL_POSTMASTER_DEATH | waitfor, port->sock, 0); + /* See comments in secure_read. */ + if (w & WL_POSTMASTER_DEATH) + ereport(FATAL, + (errcode(ERRCODE_ADMIN_SHUTDOWN), + errmsg("terminating connection due to unexpected postmaster exit"))); + /* Handle interrupt. */ if (w & WL_LATCH_SET) { |