]> granicus.if.org Git - postgresql/commit
Have pg_terminate/cancel_backend not ERROR on non-existent processes
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Thu, 27 Sep 2012 15:15:03 +0000 (12:15 -0300)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Thu, 27 Sep 2012 15:29:51 +0000 (12:29 -0300)
commitae90ffada4cb27c9194797c43d864a86ce88d72c
tree69a2ddee386449ba6b64bb71bb31fa71b2efd17b
parent55c1687a97c3c2b6cbf7c1b45830b49f03641908
Have pg_terminate/cancel_backend not ERROR on non-existent processes

This worked fine for superusers, but not for ordinary users trying to
cancel their own processes.  Tweak the order the checks are done in so
that we correctly return SIGNAL_BACKEND_ERROR (which current callers
know to ignore without erroring out) so that an ordinary user can loop
through a resultset without fearing that a process might exit in the
middle of said looping -- causing the remaining processes to go
unsignalled.

Incidentally, the last in-core caller of IsBackendPid() is now gone.
However, the function is exported and must remain in place, because
there are plenty of callers in external modules.

Author: Josh Kupershmidt

Reviewed by Noah Misch
src/backend/utils/adt/misc.c