Re: Fix a host of strto*() bugs

On Sun Aug 2, 2026 at 10:26 AM UTC, Michael Paquier wrote:
> On Fri, Jul 31, 2026 at 05:03:38PM +0900, Michael Paquier wrote:
>> @@ -961,7 +961,7 @@ precheck_tar_backup_file(verifier_context *context, char *relpath,
>> * Report an error if we didn't consume at least one character, if the
>> * result is 0, or if the value is too large to be a valid OID.
>> */
>> - if (suffix == NULL || num <= 0 || num > OID_MAX)
>> + if (suffix == relpath || num <= 0 || num > OID_MAX)
>>
>> Because that's something part of the C standard. My Linux man page
>> has a reference to that, as well:
>> "If there were no digits at all, strtoul() stores the original value
>> of nptr in *endptr (and returns 0)."
>
> 0002 and 0005 have been merged together, and applied as 355814931141.

Here are some re-spins of the previous uncommitted patches. They have
been rebased on master. I am really curious how the formatting got
messed up initially. Thanks for committing the the two patches.

--
Tristan Partin
PostgreSQL Contributors Team
AWS (https://aws.amazon.com)