Re: Checking pgwin32_is_junction() errors - Mailing list pgsql-hackers

From r.zharkov@postgrespro.ru
Subject Re: Checking pgwin32_is_junction() errors
Date
Msg-id 1f4749dbfb9163bd2215021aa0ece514@postgrespro.ru
Whole thread Raw
In response to Re: Checking pgwin32_is_junction() errors  (Thomas Munro <thomas.munro@gmail.com>)
Responses Re: Checking pgwin32_is_junction() errors
List pgsql-hackers
On 2022-08-11 07:55, Thomas Munro wrote:
>> I checked a few variants:
>> 
>> 21.07.2022  15:11    <JUNCTION>     HOME [\??\Volume{GUID}\]
>> 09.08.2022  15:06    <JUNCTION>     Test1 [\\?\Volume{GUID}\]
>> 09.08.2022  15:06    <JUNCTION>     Test2 [\\.\Volume{GUID}\]
>> 09.08.2022  15:17    <JUNCTION>     Test3 [\??\Volume{GUID}\]
>> 09.08.2022  15:27    <JUNCTION>     Test4 [C:\temp\1]
>> 09.08.2022  15:28    <JUNCTION>     Test5 [C:\HOME\Temp\1]
> 
> One more thing I wondered about, now that we're following junctions
> outside PGDATA: can a junction point to another junction?  If so, I
> didn't allow for that: stat() gives up after one hop, because I
> figured that was enough for the stuff we expect inside PGDATA and I
> couldn't find any evidence in the man pages that referred to chains.
> But if you *are* allowed to create a junction "c:\huey" that points to
> junction "c:\dewey" that points to "c:\louie", and then you do initdb
> -D c:\huey\pgdata, then I guess it would fail.  Would you mind
> checking if that is a real possibility, and if so, testing this
> chain-following patch to see if it fixes it?

I made some junctions and rechecked both patches.

11.08.2022  16:11    <JUNCTION>     donald [C:\huey]
11.08.2022  13:23    <JUNCTION>     huey [C:\dewey]
11.08.2022  13:23    <JUNCTION>     dewey [C:\louie]
11.08.2022  16:57    <DIR>          louie

With the small attached patch initdb succeeded in any of these
"directories". If the junction chain is too long, initdb fails with
"could not create directory" as expected.

initdb -D huey/pgdata
...
Success.

initdb -N -D donald
...
Success.

11.08.2022  17:32    <DIR>          1
11.08.2022  17:32    <JUNCTION>     2 [C:\1]
11.08.2022  17:32    <JUNCTION>     3 [C:\2]
11.08.2022  17:32    <JUNCTION>     4 [C:\3]
11.08.2022  17:32    <JUNCTION>     5 [C:\4]
11.08.2022  17:32    <JUNCTION>     6 [C:\5]
11.08.2022  17:32    <JUNCTION>     7 [C:\6]
11.08.2022  17:32    <JUNCTION>     8 [C:\7]
11.08.2022  17:32    <JUNCTION>     9 [C:\8]
11.08.2022  17:32    <JUNCTION>     10 [C:\9]

initdb -D 10/pgdata
...
creating directory 10/pgdata ... initdb: error: could not create 
directory "10": File exists

initdb -D 9/pgdata
...
Success.

Attachment

pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: Avoid unecessary MemSet call (src/backend/utils/cache/relcache.c)
Next
From: Amit Kapila
Date:
Subject: Re: Allow logical replication to copy tables in binary format