pgsql: Avoid dangling-pointer usage in pg_basebackup progress reports.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Avoid dangling-pointer usage in pg_basebackup progress reports.
Date: 2022-02-17 20:03:47
Message-ID: E1nKn0F-0007yX-EP@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Avoid dangling-pointer usage in pg_basebackup progress reports.

Ill-considered refactoring in 23a1c6578 led to progress_filename
sometimes pointing to data that had gone out of scope. The most
bulletproof fix is to hang onto a copy of whatever's passed in.
Compared to the work spent elsewhere per file, that's not very
expensive, plus we can skip it except in verbose logging mode.

Per buildfarm.

Discussion: https://postgr.es/m/20220212211316.GK31460@telsasoft.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/62cb7427d1e491faf8612a82c2e3711a8cd65422

Modified Files
--------------
src/bin/pg_basebackup/pg_basebackup.c | 21 ++++++++++++++++-----
1 file changed, 16 insertions(+), 5 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Amit Kapila 2022-02-18 02:27:19 pgsql: Fix a comment in worker.c.
Previous Message Robert Haas 2022-02-17 15:59:37 Re: pgsql: pg_upgrade: Preserve relfilenodes and tablespace OIDs.