From 5be41719cdfa5bb5cff1cce88a50ddd11c4675dc Mon Sep 17 00:00:00 2001 From: Bharath Rupireddy Date: Sat, 4 Nov 2023 18:11:08 +0000 Subject: [PATCH v24] Add function to log the backtrace of specified postgres process This commit adds pg_log_backtrace() function that requests to log the backtrace to console i.e. stderr of the specified backend or auxiliary process except logger and statistic collector. Only superusers are allowed to request to log the backtrace which is safe from a security standpoint because the backtrace might contain internal details. Bump catalog version. --- contrib/pg_prewarm/autoprewarm.c | 2 + doc/src/sgml/func.sgml | 76 ++++++++++++++++++++ src/backend/catalog/system_functions.sql | 2 + src/backend/postmaster/autovacuum.c | 4 ++ src/backend/postmaster/bgworker.c | 2 + src/backend/postmaster/bgwriter.c | 2 + src/backend/postmaster/checkpointer.c | 2 + src/backend/postmaster/pgarch.c | 2 + src/backend/postmaster/startup.c | 2 + src/backend/postmaster/walwriter.c | 2 + src/backend/replication/walreceiver.c | 2 + src/backend/replication/walsender.c | 2 + src/backend/storage/ipc/procsignal.c | 87 +++++++++++++++++++++++ src/backend/storage/ipc/signalfuncs.c | 26 +++++++ src/backend/tcop/postgres.c | 2 + src/include/catalog/pg_proc.dat | 5 ++ src/include/storage/procsignal.h | 2 + src/test/regress/expected/backtrace.out | 49 +++++++++++++ src/test/regress/expected/backtrace_1.out | 55 ++++++++++++++ src/test/regress/parallel_schedule | 2 +- src/test/regress/sql/backtrace.sql | 33 +++++++++ 21 files changed, 360 insertions(+), 1 deletion(-) create mode 100644 src/test/regress/expected/backtrace.out create mode 100644 src/test/regress/expected/backtrace_1.out create mode 100644 src/test/regress/sql/backtrace.sql diff --git a/contrib/pg_prewarm/autoprewarm.c b/contrib/pg_prewarm/autoprewarm.c index d0efc9e524..bf6556285e 100644 --- a/contrib/pg_prewarm/autoprewarm.c +++ b/contrib/pg_prewarm/autoprewarm.c @@ -177,6 +177,8 @@ autoprewarm_main(Datum main_arg) pqsignal(SIGUSR1, procsignal_sigusr1_handler); BackgroundWorkerUnblockSignals(); + LoadBacktraceFunctions(); + /* Create (if necessary) and attach to our shared memory area. */ if (apw_init_shmem()) first_time = false; diff --git a/doc/src/sgml/func.sgml b/doc/src/sgml/func.sgml index a6fcac0824..915f98ba49 100644 --- a/doc/src/sgml/func.sgml +++ b/doc/src/sgml/func.sgml @@ -25649,6 +25649,27 @@ SELECT collation for ('foo' COLLATE "de_DE"); + + + + pg_log_backtrace + + pg_log_backtrace ( pid integer ) + boolean + + + Requests to log the backtrace of the backend with the specified + process ID. This function can send the request to backends and + auxiliary processes except the logger and statistics collector. + The backtraces will be logged to stderr. + Typically, a backtrace identifies which function a process is + currently executing and it is useful for developers to diagnose + stuck processes and other problems. This function is supported + only if PostgreSQL was built with the ability to capture backtraces, + otherwise it will emit a warning. + + + @@ -26706,6 +26727,61 @@ LOG: Grand total: 1651920 bytes in 201 blocks; 622360 free (88 chunks); 1029560 because it may generate a large number of log messages. + + pg_log_backtrace can be used to log the backtrace of + a backend process. For example: + +postgres=# select pg_log_backtrace(pg_backend_pid()); + pg_log_backtrace +------------------ + t +(1 row) + +The backtrace will be logged as specified by the logging configuration. +For example: + +logging current backtrace of process with PID 3499242: +postgres: ubuntu postgres [local] INSERT(+0x61a355)[0x5559b94de355] +postgres: ubuntu postgres [local] INSERT(procsignal_sigusr1_handler+0x9e)[0x5559b94de4ef] +/lib/x86_64-linux-gnu/libc.so.6(+0x42520)[0x7f7e9c0e8520] +postgres: ubuntu postgres [local] INSERT(+0x868004)[0x5559b972c004] +postgres: ubuntu postgres [local] INSERT(pfree+0x1c)[0x5559b972e445] +postgres: ubuntu postgres [local] INSERT(heap_free_minimal_tuple+0x1c)[0x5559b8fa7b48] +postgres: ubuntu postgres [local] INSERT(+0x8826d6)[0x5559b97466d6] +postgres: ubuntu postgres [local] INSERT(+0x881126)[0x5559b9745126] +postgres: ubuntu postgres [local] INSERT(tuplestore_putvalues+0x83)[0x5559b9744ed5] +postgres: ubuntu postgres [local] INSERT(ExecMakeTableFunctionResult+0x68b)[0x5559b9284295] +postgres: ubuntu postgres [local] INSERT(+0x3dd14a)[0x5559b92a114a] +postgres: ubuntu postgres [local] INSERT(+0x3c1a5c)[0x5559b9285a5c] +postgres: ubuntu postgres [local] INSERT(ExecScan+0x77)[0x5559b9285ad5] +postgres: ubuntu postgres [local] INSERT(+0x3dd4f4)[0x5559b92a14f4] +postgres: ubuntu postgres [local] INSERT(+0x3bd485)[0x5559b9281485] +postgres: ubuntu postgres [local] INSERT(+0x3f96f0)[0x5559b92bd6f0] +postgres: ubuntu postgres [local] INSERT(+0x3ff25e)[0x5559b92c325e] +postgres: ubuntu postgres [local] INSERT(+0x3bd485)[0x5559b9281485] +postgres: ubuntu postgres [local] INSERT(+0x3b07a4)[0x5559b92747a4] +postgres: ubuntu postgres [local] INSERT(+0x3b3594)[0x5559b9277594] +postgres: ubuntu postgres [local] INSERT(standard_ExecutorRun+0x1f4)[0x5559b9274e8c] +postgres: ubuntu postgres [local] INSERT(ExecutorRun+0x5d)[0x5559b9274c95] +postgres: ubuntu postgres [local] INSERT(+0x64ee3a)[0x5559b9512e3a] +postgres: ubuntu postgres [local] INSERT(+0x6509c9)[0x5559b95149c9] +postgres: ubuntu postgres [local] INSERT(PortalRun+0x378)[0x5559b9513ec0] +postgres: ubuntu postgres [local] INSERT(+0x648c56)[0x5559b950cc56] +postgres: ubuntu postgres [local] INSERT(PostgresMain+0x80c)[0x5559b9511c46] +postgres: ubuntu postgres [local] INSERT(+0x576647)[0x5559b943a647] +postgres: ubuntu postgres [local] INSERT(+0x575ed3)[0x5559b9439ed3] +postgres: ubuntu postgres [local] INSERT(+0x572388)[0x5559b9436388] +postgres: ubuntu postgres [local] INSERT(PostmasterMain+0x14b3)[0x5559b9435ae5] +postgres: ubuntu postgres [local] INSERT(+0x43b563)[0x5559b92ff563] +/lib/x86_64-linux-gnu/libc.so.6(+0x29d90)[0x7f7e9c0cfd90] +/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0x80)[0x7f7e9c0cfe40] +postgres: ubuntu postgres [local] INSERT(_start+0x25)[0x5559b8f8f005] + + One can obtain the file name and line number from the logged details by using + gdb/addr2line in linux platforms (users must ensure gdb/addr2line is + already installed). + + diff --git a/src/backend/catalog/system_functions.sql b/src/backend/catalog/system_functions.sql index 35d738d576..dfa8f19fe1 100644 --- a/src/backend/catalog/system_functions.sql +++ b/src/backend/catalog/system_functions.sql @@ -742,6 +742,8 @@ REVOKE EXECUTE ON FUNCTION pg_ls_logicalmapdir() FROM PUBLIC; REVOKE EXECUTE ON FUNCTION pg_ls_replslotdir(text) FROM PUBLIC; +REVOKE EXECUTE ON FUNCTION pg_log_backtrace(integer) FROM PUBLIC; + -- -- We also set up some things as accessible to standard roles. -- diff --git a/src/backend/postmaster/autovacuum.c b/src/backend/postmaster/autovacuum.c index 3a6f24a023..64b01c9f65 100644 --- a/src/backend/postmaster/autovacuum.c +++ b/src/backend/postmaster/autovacuum.c @@ -475,6 +475,8 @@ AutoVacLauncherMain(int argc, char *argv[]) pqsignal(SIGFPE, FloatExceptionHandler); pqsignal(SIGCHLD, SIG_DFL); + LoadBacktraceFunctions(); + /* * Create a per-backend PGPROC struct in shared memory, except in the * EXEC_BACKEND case where this was done in SubPostmasterMain. We must do @@ -1547,6 +1549,8 @@ AutoVacWorkerMain(int argc, char *argv[]) pqsignal(SIGFPE, FloatExceptionHandler); pqsignal(SIGCHLD, SIG_DFL); + LoadBacktraceFunctions(); + /* * Create a per-backend PGPROC struct in shared memory, except in the * EXEC_BACKEND case where this was done in SubPostmasterMain. We must do diff --git a/src/backend/postmaster/bgworker.c b/src/backend/postmaster/bgworker.c index 48a9924527..906bb28c20 100644 --- a/src/backend/postmaster/bgworker.c +++ b/src/backend/postmaster/bgworker.c @@ -794,6 +794,8 @@ BackgroundWorkerMain(void) pqsignal(SIGUSR2, SIG_IGN); pqsignal(SIGCHLD, SIG_DFL); + LoadBacktraceFunctions(); + /* * If an exception is encountered, processing resumes here. * diff --git a/src/backend/postmaster/bgwriter.c b/src/backend/postmaster/bgwriter.c index f2e4f23d9f..29cb27e3f7 100644 --- a/src/backend/postmaster/bgwriter.c +++ b/src/backend/postmaster/bgwriter.c @@ -112,6 +112,8 @@ BackgroundWriterMain(void) */ pqsignal(SIGCHLD, SIG_DFL); + LoadBacktraceFunctions(); + /* * We just started, assume there has been either a shutdown or * end-of-recovery snapshot. diff --git a/src/backend/postmaster/checkpointer.c b/src/backend/postmaster/checkpointer.c index a3c1aba24e..b027bae7ce 100644 --- a/src/backend/postmaster/checkpointer.c +++ b/src/backend/postmaster/checkpointer.c @@ -198,6 +198,8 @@ CheckpointerMain(void) */ pqsignal(SIGCHLD, SIG_DFL); + LoadBacktraceFunctions(); + /* * Initialize so that first time-driven event happens at the correct time. */ diff --git a/src/backend/postmaster/pgarch.c b/src/backend/postmaster/pgarch.c index 46af349564..bc542e2f21 100644 --- a/src/backend/postmaster/pgarch.c +++ b/src/backend/postmaster/pgarch.c @@ -232,6 +232,8 @@ PgArchiverMain(void) /* Unblock signals (they were blocked when the postmaster forked us) */ sigprocmask(SIG_SETMASK, &UnBlockSig, NULL); + LoadBacktraceFunctions(); + /* We shouldn't be launched unnecessarily. */ Assert(XLogArchivingActive()); diff --git a/src/backend/postmaster/startup.c b/src/backend/postmaster/startup.c index 0e7de26bc2..3ea64ed449 100644 --- a/src/backend/postmaster/startup.c +++ b/src/backend/postmaster/startup.c @@ -259,6 +259,8 @@ StartupProcessMain(void) pqsignal(SIGUSR1, procsignal_sigusr1_handler); pqsignal(SIGUSR2, StartupProcTriggerHandler); + LoadBacktraceFunctions(); + /* * Reset some signals that are accepted by postmaster but not here */ diff --git a/src/backend/postmaster/walwriter.c b/src/backend/postmaster/walwriter.c index 266fbc2339..6982d05647 100644 --- a/src/backend/postmaster/walwriter.c +++ b/src/backend/postmaster/walwriter.c @@ -115,6 +115,8 @@ WalWriterMain(void) */ pqsignal(SIGCHLD, SIG_DFL); + LoadBacktraceFunctions(); + /* * Create a memory context that we will do all our work in. We do this so * that we can reset the context during error recovery and thereby avoid diff --git a/src/backend/replication/walreceiver.c b/src/backend/replication/walreceiver.c index a3128874b2..949c768c33 100644 --- a/src/backend/replication/walreceiver.c +++ b/src/backend/replication/walreceiver.c @@ -287,6 +287,8 @@ WalReceiverMain(void) /* Reset some signals that are accepted by postmaster but not here */ pqsignal(SIGCHLD, SIG_DFL); + LoadBacktraceFunctions(); + /* Load the libpq-specific functions */ load_file("libpqwalreceiver", false); if (WalReceiverFunctions == NULL) diff --git a/src/backend/replication/walsender.c b/src/backend/replication/walsender.c index e250b0567e..33e81cf02b 100644 --- a/src/backend/replication/walsender.c +++ b/src/backend/replication/walsender.c @@ -3270,6 +3270,8 @@ WalSndSignals(void) /* Reset some signals that are accepted by postmaster but not here */ pqsignal(SIGCHLD, SIG_DFL); + + LoadBacktraceFunctions(); } /* Report shared-memory space needed by WalSndShmemInit */ diff --git a/src/backend/storage/ipc/procsignal.c b/src/backend/storage/ipc/procsignal.c index b7427906de..e2b3b4009e 100644 --- a/src/backend/storage/ipc/procsignal.c +++ b/src/backend/storage/ipc/procsignal.c @@ -16,6 +16,9 @@ #include #include +#ifdef HAVE_EXECINFO_H +#include +#endif #include "access/parallel.h" #include "port/pg_bitutils.h" @@ -97,6 +100,10 @@ typedef struct #define BARRIER_CLEAR_BIT(flags, type) \ ((flags) &= ~(((uint32) 1) << (uint32) (type))) +#ifdef HAVE_BACKTRACE_SYMBOLS +static bool backtrace_functions_loaded = false; +#endif + static ProcSignalHeader *ProcSignal = NULL; static ProcSignalSlot *MyProcSignalSlot = NULL; @@ -609,6 +616,76 @@ ResetProcSignalBarrierBits(uint32 flags) InterruptPending = true; } +/* + * HandleLogBacktraceInterrupt - Handle receipt of an interrupt requesting to + * log a backtrace. + * + * We capture the backtrace within this signal handler and emit to stderr. Note + * that we ensured the backtrace-related functions are signal-safe, see + * LoadBacktraceFunctions() for more details. + * + * Emitting backtrace to stderr as opposed to writing to server log has an + * advantage - we don't need to allocate any dynamic memory while capturing + * backtrace which makes the signal handler safe. + */ +static void +HandleLogBacktraceInterrupt(void) +{ +#ifdef HAVE_BACKTRACE_SYMBOLS + void *buf[100]; + int nframes; + + /* Quickly exit if backtrace-related functions aren't loaded. */ + if (!backtrace_functions_loaded) + return; + + nframes = backtrace(buf, lengthof(buf)); + + write_stderr("logging current backtrace of process with PID %d:\n", + MyProcPid); + backtrace_symbols_fd(buf, nframes, fileno(stderr)); +#endif +} + +/* + * LoadBacktraceFunctions - call a backtrace-related function to ensure the + * shared library implementing them is loaded beforehand. + * + * Any backtrace-related functions when called for the first time dynamically + * loads the shared library, which usually triggers a call to malloc, making + * them unsafe to use in signal handlers. + * + * This functions is an attempt to make backtrace-related functions signal + * safe. + * + * NOTE: This function is supposed to be called in the early life of a process, + * preferably after SIGUSR1 handler is setup and before the backtrace-related + * functions are used in signal handlers. It is not supposed to be called from + * within a signal handler. + */ +void +LoadBacktraceFunctions(void) +{ +#ifdef HAVE_BACKTRACE_SYMBOLS + void *buf[100]; + + /* + * XXX: It is a bit of overkill to check if the shared library + * implementing backtrace-related functions is loaded already. Instead, we + * go ahead and call one function. + */ + + /* + * It is enough to call any one backtrace-related function to ensure that + * the corresponding shared library is dynamically loaded if not done + * already. + */ + backtrace(buf, lengthof(buf)); + + backtrace_functions_loaded = true; +#endif +} + /* * CheckProcSignal - check to see if a particular reason has been * signaled, and clear the signal flag. Should be called after receiving @@ -661,6 +738,16 @@ procsignal_sigusr1_handler(SIGNAL_ARGS) if (CheckProcSignal(PROCSIG_PARALLEL_APPLY_MESSAGE)) HandleParallelApplyMessageInterrupt(); + /* + * XXX: Since the log backtrace signal handler itself does the required + * job, returning without setting the latch may be a good idea here. + * However, it is better not to deviate from the tradition of a signal + * handler setting the latch to wake up the processes that are waiting on + * it. + */ + if (CheckProcSignal(PROCSIG_LOG_BACKTRACE)) + HandleLogBacktraceInterrupt(); + if (CheckProcSignal(PROCSIG_RECOVERY_CONFLICT_DATABASE)) HandleRecoveryConflictInterrupt(PROCSIG_RECOVERY_CONFLICT_DATABASE); diff --git a/src/backend/storage/ipc/signalfuncs.c b/src/backend/storage/ipc/signalfuncs.c index 94ae553b94..e24f04bbcb 100644 --- a/src/backend/storage/ipc/signalfuncs.c +++ b/src/backend/storage/ipc/signalfuncs.c @@ -311,3 +311,29 @@ pg_rotate_logfile_v2(PG_FUNCTION_ARGS) SendPostmasterSignal(PMSIGNAL_ROTATE_LOGFILE); PG_RETURN_BOOL(true); } + +/* + * pg_log_backtrace - signal a backend or an auxiliary process to log its + * current backtrace to stderr. + * + * By default, only superusers are allowed to request to log the backtrace + * which is safe from a security standpoint because the backtrace might contain + * internal details. However, a superuser can grant the execute permission to + * anyone, if it wishes. + */ +Datum +pg_log_backtrace(PG_FUNCTION_ARGS) +{ + int pid = PG_GETARG_INT32(0); + bool result; + +#ifndef HAVE_BACKTRACE_SYMBOLS + ereport(WARNING, + errmsg("backtrace generation is not supported by this installation"), + errhint("You need to rebuild PostgreSQL using a library containing backtrace_symbols.")); + PG_RETURN_BOOL(false); +#endif + + result = SendProcSignalBackendOrAuxproc(pid, PROCSIG_LOG_BACKTRACE); + PG_RETURN_BOOL(result); +} diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index 6a070b5d8c..c64fdb95d9 100644 --- a/src/backend/tcop/postgres.c +++ b/src/backend/tcop/postgres.c @@ -4189,6 +4189,8 @@ PostgresMain(const char *dbname, const char *username) */ pqsignal(SIGCHLD, SIG_DFL); /* system() requires this on some * platforms */ + + LoadBacktraceFunctions(); } /* Early initialization */ diff --git a/src/include/catalog/pg_proc.dat b/src/include/catalog/pg_proc.dat index 091f7e343c..e76c85a531 100644 --- a/src/include/catalog/pg_proc.dat +++ b/src/include/catalog/pg_proc.dat @@ -12083,4 +12083,9 @@ proname => 'any_value_transfn', prorettype => 'anyelement', proargtypes => 'anyelement anyelement', prosrc => 'any_value_transfn' }, +# function to get the backtrace of server process +{ oid => '9661', descr => 'log backtrace of server process', + proname => 'pg_log_backtrace', provolatile => 'v', prorettype => 'bool', + proargtypes => 'int4', prosrc => 'pg_log_backtrace' }, + ] diff --git a/src/include/storage/procsignal.h b/src/include/storage/procsignal.h index 548519117a..9e517395f6 100644 --- a/src/include/storage/procsignal.h +++ b/src/include/storage/procsignal.h @@ -36,6 +36,7 @@ typedef enum PROCSIG_BARRIER, /* global barrier interrupt */ PROCSIG_LOG_MEMORY_CONTEXT, /* ask backend to log the memory contexts */ PROCSIG_PARALLEL_APPLY_MESSAGE, /* Message from parallel apply workers */ + PROCSIG_LOG_BACKTRACE, /* ask backend to log the current backtrace */ /* Recovery conflict reasons */ PROCSIG_RECOVERY_CONFLICT_FIRST, @@ -71,5 +72,6 @@ extern void WaitForProcSignalBarrier(uint64 generation); extern void ProcessProcSignalBarrier(void); extern void procsignal_sigusr1_handler(SIGNAL_ARGS); +extern void LoadBacktraceFunctions(void); #endif /* PROCSIGNAL_H */ diff --git a/src/test/regress/expected/backtrace.out b/src/test/regress/expected/backtrace.out new file mode 100644 index 0000000000..5341a6adfe --- /dev/null +++ b/src/test/regress/expected/backtrace.out @@ -0,0 +1,49 @@ +-- +-- pg_log_backtrace() +-- +-- Backtraces are logged to stderr and not returned to the function. +-- Furthermore, their contents can vary depending on the timing. However, +-- we can at least verify that the code doesn't fail, and that the +-- permissions are set properly. +-- +SELECT pg_log_backtrace(pg_backend_pid()); + pg_log_backtrace +------------------ + t +(1 row) + +SELECT pg_log_backtrace(pid) FROM pg_stat_activity + WHERE backend_type = 'checkpointer'; + pg_log_backtrace +------------------ + t +(1 row) + +CREATE ROLE regress_log_backtrace; +SELECT has_function_privilege('regress_log_backtrace', + 'pg_log_backtrace(integer)', 'EXECUTE'); -- no + has_function_privilege +------------------------ + f +(1 row) + +GRANT EXECUTE ON FUNCTION pg_log_backtrace(integer) + TO regress_log_backtrace; +SELECT has_function_privilege('regress_log_backtrace', + 'pg_log_backtrace(integer)', 'EXECUTE'); -- yes + has_function_privilege +------------------------ + t +(1 row) + +SET ROLE regress_log_backtrace; +SELECT pg_log_backtrace(pg_backend_pid()); + pg_log_backtrace +------------------ + t +(1 row) + +RESET ROLE; +REVOKE EXECUTE ON FUNCTION pg_log_backtrace(integer) + FROM regress_log_backtrace; +DROP ROLE regress_log_backtrace; diff --git a/src/test/regress/expected/backtrace_1.out b/src/test/regress/expected/backtrace_1.out new file mode 100644 index 0000000000..899f330224 --- /dev/null +++ b/src/test/regress/expected/backtrace_1.out @@ -0,0 +1,55 @@ +-- +-- pg_log_backtrace() +-- +-- Backtraces are logged to stderr and not returned to the function. +-- Furthermore, their contents can vary depending on the timing. However, +-- we can at least verify that the code doesn't fail, and that the +-- permissions are set properly. +-- +SELECT pg_log_backtrace(pg_backend_pid()); +WARNING: backtrace generation is not supported by this installation +HINT: You need to rebuild PostgreSQL using a library containing backtrace_symbols. + pg_log_backtrace +------------------ + f +(1 row) + +SELECT pg_log_backtrace(pid) FROM pg_stat_activity + WHERE backend_type = 'checkpointer'; +WARNING: backtrace generation is not supported by this installation +HINT: You need to rebuild PostgreSQL using a library containing backtrace_symbols. + pg_log_backtrace +------------------ + f +(1 row) + +CREATE ROLE regress_log_backtrace; +SELECT has_function_privilege('regress_log_backtrace', + 'pg_log_backtrace(integer)', 'EXECUTE'); -- no + has_function_privilege +------------------------ + f +(1 row) + +GRANT EXECUTE ON FUNCTION pg_log_backtrace(integer) + TO regress_log_backtrace; +SELECT has_function_privilege('regress_log_backtrace', + 'pg_log_backtrace(integer)', 'EXECUTE'); -- yes + has_function_privilege +------------------------ + t +(1 row) + +SET ROLE regress_log_backtrace; +SELECT pg_log_backtrace(pg_backend_pid()); +WARNING: backtrace generation is not supported by this installation +HINT: You need to rebuild PostgreSQL using a library containing backtrace_symbols. + pg_log_backtrace +------------------ + f +(1 row) + +RESET ROLE; +REVOKE EXECUTE ON FUNCTION pg_log_backtrace(integer) + FROM regress_log_backtrace; +DROP ROLE regress_log_backtrace; diff --git a/src/test/regress/parallel_schedule b/src/test/regress/parallel_schedule index f0987ff537..32c286dd0c 100644 --- a/src/test/regress/parallel_schedule +++ b/src/test/regress/parallel_schedule @@ -78,7 +78,7 @@ test: brin_bloom brin_multi # psql depends on create_am # amutils depends on geometry, create_index_spgist, hash_index, brin # ---------- -test: create_table_like alter_generic alter_operator misc async dbsize merge misc_functions sysviews tsrf tid tidscan tidrangescan collate.icu.utf8 incremental_sort create_role +test: create_table_like alter_generic alter_operator misc async dbsize merge misc_functions sysviews tsrf tid tidscan tidrangescan collate.icu.utf8 incremental_sort create_role backtrace # collate.*.utf8 tests cannot be run in parallel with each other test: rules psql psql_crosstab amutils stats_ext collate.linux.utf8 collate.windows.win1252 diff --git a/src/test/regress/sql/backtrace.sql b/src/test/regress/sql/backtrace.sql new file mode 100644 index 0000000000..674e41c998 --- /dev/null +++ b/src/test/regress/sql/backtrace.sql @@ -0,0 +1,33 @@ +-- +-- pg_log_backtrace() +-- +-- Backtraces are logged to stderr and not returned to the function. +-- Furthermore, their contents can vary depending on the timing. However, +-- we can at least verify that the code doesn't fail, and that the +-- permissions are set properly. +-- + +SELECT pg_log_backtrace(pg_backend_pid()); + +SELECT pg_log_backtrace(pid) FROM pg_stat_activity + WHERE backend_type = 'checkpointer'; + +CREATE ROLE regress_log_backtrace; + +SELECT has_function_privilege('regress_log_backtrace', + 'pg_log_backtrace(integer)', 'EXECUTE'); -- no + +GRANT EXECUTE ON FUNCTION pg_log_backtrace(integer) + TO regress_log_backtrace; + +SELECT has_function_privilege('regress_log_backtrace', + 'pg_log_backtrace(integer)', 'EXECUTE'); -- yes + +SET ROLE regress_log_backtrace; +SELECT pg_log_backtrace(pg_backend_pid()); +RESET ROLE; + +REVOKE EXECUTE ON FUNCTION pg_log_backtrace(integer) + FROM regress_log_backtrace; + +DROP ROLE regress_log_backtrace; -- 2.34.1