Re: [Patch] checksumming-related buglets in pg_verify_checksums/pg_basebackup TAP tests

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Michael Banck <michael(dot)banck(at)credativ(dot)de>
Cc: PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Magnus Hagander <magnus(at)hagander(dot)net>
Subject: Re: [Patch] checksumming-related buglets in pg_verify_checksums/pg_basebackup TAP tests
Date: 2019-02-18 05:25:03
Message-ID: 20190218052503.GL15532@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Feb 14, 2019 at 03:07:56PM +0100, Michael Banck wrote:
> This also revealed a second bug in the pg_basebackup test suite where
> the offset for the corruption in the second file was wrong, so it
> actually never got corrupted, and the tests only passed due to the above
> twice than expected number of written bytes. The write() probably
> overflowed into an adjacent block so that the total number of corrupted
> blocks was as expected by accident. Oops, my bad, patch attached.

Fixed and back-patched where adapted, thanks! ee9e145 was a first
shot for a fix in pg_basebackup tests, but it has missed one seek()
call.
--
Michael

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Kuroda, Hayato 2019-02-18 05:39:09 RE: Prevent extension creation in temporary schemas
Previous Message David Rowley 2019-02-18 05:15:46 Re: 2019-03 CF Summary / Review - Tranche #2