Re: sendFileWithContent() does not advance the source pointer - Mailing list pgsql-hackers

From Robert Haas
Subject Re: sendFileWithContent() does not advance the source pointer
Date
Msg-id CA+TgmobxXJBp58e+UFqO8tCKPpmJzYOUrzrWJL3nzo=T_kh=Ow@mail.gmail.com
Whole thread Raw
In response to sendFileWithContent() does not advance the source pointer  (Antonin Houska <ah@cybertec.at>)
List pgsql-hackers
On Thu, Dec 8, 2022 at 2:43 PM Antonin Houska <ah@cybertec.at> wrote:
> When checking something else in the base backup code, I've noticed that
> sendFileWithContent() does not advance the 'content' pointer. The sink buffer
> is large enough (32kB) so that the first iteration usually processes the whole
> file (only special files are processed by this function), and thus that the
> problem is hidden.
>
> However it's possible to hit the issue: if there are too many tablespaces,
> pg_basebackup generates corrupted tablespace_map. Instead of writing all the
> tablespace paths it writes only some and then starts to write the contents
> from the beginning again.

Thanks for the report, analysis, and fix. I have committed your patch
and back-patched to v15.

-- 
Robert Haas
EDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: "Jonathan S. Katz"
Date:
Subject: Re: Raising the SCRAM iteration count
Next
From: Joseph Koshakow
Date:
Subject: Re: Date-Time dangling unit fix