Re: drop tablespace failed when location contains .. on win32 - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: drop tablespace failed when location contains .. on win32
Date
Msg-id YTmDLXnd10Uf2n5H@paquier.xyz
Whole thread Raw
In response to RE: drop tablespace failed when location contains .. on win32  ("wangsh.fnst@fujitsu.com" <wangsh.fnst@fujitsu.com>)
Responses Re: drop tablespace failed when location contains .. on win32
Re: drop tablespace failed when location contains .. on win32
List pgsql-hackers
On Thu, Sep 09, 2021 at 02:35:52AM +0000, wangsh.fnst@fujitsu.com wrote:
> Do you mean changing the action of canonicalize_path(), like remove all the (..) ?
>
> I'm willing to fix this problem.

Looking at canonicalize_path(), we have already some logic around
pending_strips to remove paths when we find a "/.." in the path, so
that's a matter of adjusting this area to trim properly the previous
directory.

On *nix platforms, we don't apply this much caution either, say a
simple /tmp/path/../path/ results in this same path used in the link
from pg_tblspc.  But we are speaking about Windows here, and junction
points.

Based on the lack of complains over the years, that does not seem
really worth backpatching.  Just my 2c on this point.
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Greg Stark
Date:
Subject: Re: Add jsonlog log_destination for JSON server logs
Next
From: Michael Paquier
Date:
Subject: Re: Add jsonlog log_destination for JSON server logs