pgsql: Correct some code in pg_restore when reading the header of a tar

From: neilc(at)svr1(dot)postgresql(dot)org (Neil Conway)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Correct some code in pg_restore when reading the header of a tar
Date: 2005-06-22 02:12:19
Message-ID: 20050622021219.911C4528B4@svr1.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Correct some code in pg_restore when reading the header of a tar archive:

(1) The code doesn't initialize `sum', so the initial "does the checksum
match?" test is wrong.

(2) The loop that is intended to check for a "null block" just checks
the first byte of the tar block 512 times, rather than each of the
512 bytes one time (!), which I'm guessing was the intent.

It was only through sheer luck that this worked in the first place.

Per Coverity static analysis performed by EnterpriseDB.

Tags:
----
REL7_4_STABLE

Modified Files:
--------------
pgsql/src/bin/pg_dump:
pg_backup_tar.c (r1.38 -> r1.38.2.1)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/bin/pg_dump/pg_backup_tar.c.diff?r1=1.38&r2=1.38.2.1)

Browse pgsql-committers by date

  From Date Subject
Next Message Neil Conway 2005-06-22 07:28:48 pgsql: Fix bug in CONTINUE statement for PL/pgSQL: when we continue a
Previous Message Neil Conway 2005-06-22 02:02:09 pgsql: Correct some code in pg_restore when reading the header of a tar