From 88295244be6a86aaa09a822ee857484cb46fd3ca Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Tue, 18 Jul 2006 22:36:46 +0000 Subject: [PATCH] copydir() and rmtree() need to use lstat, not stat, to behave as expected with symlinks. Noted while trying to use rmtree in new C-code pg_regress. --- src/port/copydir.c | 4 ++-- src/port/dirmod.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/port/copydir.c b/src/port/copydir.c index 718d99251b7..ef441e5ad3f 100644 --- a/src/port/copydir.c +++ b/src/port/copydir.c @@ -11,7 +11,7 @@ * as a service. * * IDENTIFICATION - * $PostgreSQL: pgsql/src/port/copydir.c,v 1.17 2006/03/05 15:59:10 momjian Exp $ + * $PostgreSQL: pgsql/src/port/copydir.c,v 1.18 2006/07/18 22:36:46 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -75,7 +75,7 @@ copydir(char *fromdir, char *todir, bool recurse) snprintf(fromfile, MAXPGPATH, "%s/%s", fromdir, xlde->d_name); snprintf(tofile, MAXPGPATH, "%s/%s", todir, xlde->d_name); - if (stat(fromfile, &fst) < 0) + if (lstat(fromfile, &fst) < 0) ereport(ERROR, (errcode_for_file_access(), errmsg("could not stat file \"%s\": %m", fromfile))); diff --git a/src/port/dirmod.c b/src/port/dirmod.c index d806a9ba8d7..e58384a749c 100644 --- a/src/port/dirmod.c +++ b/src/port/dirmod.c @@ -10,7 +10,7 @@ * Win32 (NT, Win2k, XP). replace() doesn't work on Win95/98/Me. * * IDENTIFICATION - * $PostgreSQL: pgsql/src/port/dirmod.c,v 1.42 2006/03/05 15:59:10 momjian Exp $ + * $PostgreSQL: pgsql/src/port/dirmod.c,v 1.43 2006/07/18 22:36:46 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -434,7 +434,7 @@ rmtree(char *path, bool rmtopdir) { snprintf(filepath, MAXPGPATH, "%s/%s", path, *filename); - if (stat(filepath, &statbuf) != 0) + if (lstat(filepath, &statbuf) != 0) goto report_and_fail; if (S_ISDIR(statbuf.st_mode)) -- 2.39.5