[Patch] checksumming-related buglets inpg_verify_checksums/pg_basebackup TAP tests - Mailing list pgsql-hackers

From Michael Banck
Subject [Patch] checksumming-related buglets inpg_verify_checksums/pg_basebackup TAP tests
Date
Msg-id 1550153276.796.35.camel@credativ.de
Whole thread Raw
Responses Re: [Patch] checksumming-related buglets inpg_verify_checksums/pg_basebackup TAP tests  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
Hi,

while hacking on pg_verify_checksums and looking at hexdumps, I noticed
that the TAP tests of pg_verify_checksums (and pg_basebackup from which
it was copy-pasted) actually write "305c305c[...]" (i.e. literal
backslashes and number 0s) instead of "000[...]" into the to-be-
corrupted relfilenodes due to wrong quoting.

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 addident. Oops, my bad, patch attached.


Michael

-- 
Michael Banck
Projektleiter / Senior Berater
Tel.: +49 2166 9901-171
Fax:  +49 2166 9901-100
Email: michael.banck@credativ.de

credativ GmbH, HRB Mönchengladbach 12080
USt-ID-Nummer: DE204566209
Trompeterallee 108, 41189 Mönchengladbach
Geschäftsführung: Dr. Michael Meskes, Jörg Folz, Sascha Heuer

Unser Umgang mit personenbezogenen Daten unterliegt
folgenden Bestimmungen: https://www.credativ.de/datenschutz
Attachment

pgsql-hackers by date:

Previous
From: Sergei Kornilov
Date:
Subject: Re: pg11.1: dsa_area could not attach to segment
Next
From: Bruce Momjian
Date:
Subject: Re: Protect syscache from bloating with negative cache entries