From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> |
Cc: | Michael Banck <michael(dot)banck(at)credativ(dot)de>, Andres Freund <andres(at)anarazel(dot)de>, Postgres hackers <pgsql-hackers(at)postgresql(dot)org>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Magnus Hagander <magnus(at)hagander(dot)net> |
Subject: | Re: pgsql: Add TAP tests for pg_verify_checksums |
Date: | 2018-10-14 22:41:38 |
Message-ID: | 20181014224138.GA15473@paquier.xyz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Sun, Oct 14, 2018 at 10:24:55AM -0400, Andrew Dunstan wrote:
> [snip]
Thanks a lot for the review, Andrew!
> This code now seems redundant:
>
> if (strcmp(fn, ".") == 0 ||
> strcmp(fn, "..") == 0)
> return true;
Indeed. I have renamed skipfile() to isRelFileName on the way, which
looks better in the context.
> I would probably reverse the order of these two tests. It might not make any
> difference, assuming fn is never an empty string, but it seems more logical
> to me.
>
> + /* good to go if only digits */
> + if (fn[pos] == '\0')
> + return false;
> + /* leave if no digits */
> + if (pos == 0)
> + return true;
No objections to that. Done.
> It also looks to me like the check for a segment number doesn't ensure
> there is at least one digit, so "123." might pass, but I could be
> wrong. In any case, there isn't a test for that, and there probably
> should be.
You are right here. This name pattern has been failing. Fixed in the
attached. I have added "123_" and "123_." as extra patterns to check.
What do you think about the updated version attached?
--
Michael
Attachment | Content-Type | Size |
---|---|---|
checksum-whitelist-v3.patch | text/x-diff | 5.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2018-10-15 01:11:42 | Re: pgsql: Add TAP tests for pg_verify_checksums |
Previous Message | Alexander Korotkov | 2018-10-14 22:12:50 | pgsql: Add missed tag in bloom.sgml |
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2018-10-14 22:43:39 | Re: [RFC] Removing "magic" oids |
Previous Message | Tom Lane | 2018-10-14 22:34:50 | Re: [RFC] Removing "magic" oids |