BUG #19718: pg_dump -Ft: restore.sql gets "\unrestrict (null)"/"_estrict (null)", so psql skips \i data files - Mailing list pgsql-bugs

From PG Bug reporting form
Subject BUG #19718: pg_dump -Ft: restore.sql gets "\unrestrict (null)"/"_estrict (null)", so psql skips \i data files
Date
Msg-id 19718-7e945d0ff9b5a589@postgresql.org
Whole thread
Responses Re: BUG #19718: pg_dump -Ft: restore.sql gets "\unrestrict (null)"/"_estrict (null)", so psql skips \i data files
List pgsql-bugs
The following bug has been logged on the website:

Bug reference:      19718
Logged by:          Pavel Kopylov
Email address:      pkopylov@cloudlinux.com
PostgreSQL version: 17.11
Operating system:   Debian 13
Description:

Since commit 71ea0d6795 ("Restrict psql meta-commands in plain-text dumps"),

```c
  _reconnectToDB() in src/bin/pg_dump/pg_backup_archiver.c writes

      ahprintf(AH, "\\unrestrict %s\n", ropt->restrict_key);
      ...
      ahprintf(AH, "\\restrict %s\n\n", ropt->restrict_key);
```
without checking ropt->restrict_key.

RestoreArchive() emits the same  markers only "if (ropt->restrict_key)".

pg_dump generates a restrict key only for --format=plain.  The tar format,
however, still writes a plain-text restore.sql through RestoreArchive() in
 _CloseArchive() (pg_backup_tar.c), using a copy of the dump's
 RestoreOptions, where restrict_key is NULL.  pg_dump always sets
 outputCreateDB for non-plain formats, so the DATABASE TOC entry always
 reaches _reconnectToDB(), and every tar-format restore.sql contains:

```
      \unrestrict (null)
      \connect srcdb
      \restrict (null)
```

  When the script is run with "psql -f restore.sql", the first line fails
  with "\unrestrict: not currently in restricted mode".  psql then enters
  restricted mode with the key "(null)" and rejects every later
  meta-command.  With pg_dump -Ft --inserts the table data is loaded through
  "\i $$PATH$$/NNNN.dat", so no table data is restored at all.

Reproduced on 18.6 and 17.11 (official Docker images).
The code on master is the same.

The logged output run on the official Docker image is:

```
$ cat repro-debian13.log
### Debian GNU/Linux 13 (trixie), postgresql-15-pllua postgresql-17
17.11-0+deb13u1postgresql-17-jit-llvm postgresql-17-pllua postgresql-9.1
+ createdb srcdb
+ psql -Xq srcdb -c 'CREATE TABLE t(i int); INSERT INTO t SELECT
generate_series(1,10)'
+ mkdir /tmp/x
+ cd /tmp/x
+ pg_dump -Ft --inserts srcdb -f d.tar
+ tar xf d.tar
+ sed -i 's|[$][$]PATH[$][$]|/tmp/x|g' restore.sql
+ dropdb srcdb
+ createdb srcdb
+ psql -X -d srcdb -f restore.sql
+ grep -i restrict
psql:restore.sql:37: error: \unrestrict: not currently in restricted mode
psql:restore.sql:72: error: backslash commands are restricted; only
\unrestrict is allowed
+ psql -XAt -d srcdb -c 'SELECT count(*) FROM t'
0
```

The expected output number MUST be 10 instead of 0.





pgsql-bugs by date:

Previous
From: Manu
Date:
Subject: Re: BUG #19695: JSON_VALUE ... RETURNING jsonb returns NULL for later evaluation once one evaluation returns NULL
Next
From: Manu
Date:
Subject: Re: autovacuum: automatically propagate updated parameters