Hi,

Attached is v4 of this patch.

As promised in the strtoint() thread, now that Tom's
commit 426551c8cf49 has landed and been backpatched
through PG 14 (REL_18_STABLE: ec65cbc00b2c), this
version switches the rounds= parse back to strtoint(),
relying on its new saturating behavior on overflow.

The delta against v3 is exactly one line in
crypt-sha.c (strtoi64() -> strtoint()); the clamping,
the NOTICE print cap, and all regression tests are
unchanged from v3. I verified it locally against
current master: the pgcrypto suite passes 25/25, and
the overflow battery and NOTICE probe output are
identical to v3.

Since the strtoint() change is backpatched, this
version should backpatch to REL_18_STABLE cleanly as
well.

Testing was done with AI assistance.

Regards,
Prashant Chinnam

Attachment: v4-0001-pgcrypto-Fix-sha-crypt-rounds-values-that-do-not-fit.patch
Description: v4-0001-pgcrypto-Fix-sha-crypt-rounds-values-that-do-not-fit.patch

Reply via email to