diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2009-05-15 15:56:39 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2009-05-15 15:56:39 +0000 |
| commit | 4616d57dad1d082d488e7827ffe116755166a198 (patch) | |
| tree | 6f7567502eb5e56dd9e969e546ce82db7df4d68d /src/backend/storage | |
| parent | abc924519a0bc7886a4b8507b03c55b2b55b2c18 (diff) | |
| download | postgresql-4616d57dad1d082d488e7827ffe116755166a198.tar.gz postgresql-4616d57dad1d082d488e7827ffe116755166a198.zip | |
Fix all the server-side SIGQUIT handlers (grumble ... why so many identical
copies?) to ensure they really don't run proc_exit/shmem_exit callbacks,
as was intended. I broke this behavior recently by installing atexit
callbacks without thinking about the one case where we truly don't want
to run those callback functions. Noted in an example from Dave Page.
Diffstat (limited to 'src/backend/storage')
| -rw-r--r-- | src/backend/storage/ipc/ipc.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/backend/storage/ipc/ipc.c b/src/backend/storage/ipc/ipc.c index 5969e6e425b..5fa2d5d37ca 100644 --- a/src/backend/storage/ipc/ipc.c +++ b/src/backend/storage/ipc/ipc.c @@ -13,7 +13,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/storage/ipc/ipc.c,v 1.103 2009/05/05 20:06:07 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/storage/ipc/ipc.c,v 1.104 2009/05/15 15:56:39 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -166,7 +166,8 @@ proc_exit_prepare(int code) /* do our shared memory exits first */ shmem_exit(code); - elog(DEBUG3, "proc_exit(%d)", code); + elog(DEBUG3, "proc_exit(%d): %d callbacks to make", + code, on_proc_exit_index); /* * call all the registered callbacks. @@ -193,7 +194,8 @@ proc_exit_prepare(int code) void shmem_exit(int code) { - elog(DEBUG3, "shmem_exit(%d)", code); + elog(DEBUG3, "shmem_exit(%d): %d callbacks to make", + code, on_shmem_exit_index); /* * call all the registered callbacks. |