From dcddc3f813ff39d0e33c2b7c77092a6c0b0e9d0b Mon Sep 17 00:00:00 2001 From: Michael Paquier Date: Fri, 7 Feb 2020 10:51:17 +0900 Subject: [PATCH] Revert "Prevent running pg_basebackup as root" This reverts commit 7bae0ad, as this is not ideal with the tar format, and we may want to explore more options like what is done by tar with some equivalents of --owner and --group, but for pg_basebackup. Per complaints from Magnus Hagander and Stephen Frost. Discussion: https://postgr.es/m/20200205172259.GW3195@tamriel.snowman.net --- src/bin/pg_basebackup/pg_basebackup.c | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/src/bin/pg_basebackup/pg_basebackup.c b/src/bin/pg_basebackup/pg_basebackup.c index 4e12cdb4467..2551cf38c91 100644 --- a/src/bin/pg_basebackup/pg_basebackup.c +++ b/src/bin/pg_basebackup/pg_basebackup.c @@ -2090,22 +2090,6 @@ main(int argc, char **argv) } } - /* - * Don't allow pg_basebackup to be run as root, to avoid creating files in - * the data directory with ownership rights incompatible with the - * postmaster. - */ -#ifndef WIN32 - if (geteuid() == 0) /* 0 is root's uid */ - { - pg_log_error("cannot be run as root"); - fprintf(stderr, - _("Please log in (using, e.g., \"su\") as the (unprivileged) user that will\n" - "own the server process.\n")); - exit(1); - } -#endif - atexit(cleanup_directories_atexit); while ((c = getopt_long(argc, argv, "CD:F:r:RS:T:X:l:nNzZ:d:c:h:p:U:s:wWkvP", -- 2.39.5