From d98ed080bb31fd3d46281127871b7886288686d9 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Thu, 8 Jun 2023 11:24:31 -0400 Subject: Fix small overestimation of base64 encoding output length. pg_base64_enc_len() and its clones overestimated the output length by up to 2 bytes, as a result of sloppy thinking about where to divide. No callers require a precise estimate, so this has no consequences worse than palloc'ing a byte or two more than necessary. We might as well get it right though. This bug is very ancient, dating to commit 79d78bb26 which added encode.c. (The other instances were presumably copied from there.) Still, it doesn't quite seem worth back-patching. Oleg Tselebrovskiy Discussion: https://postgr.es/m/f94da55286a63022150bc266afdab754@postgrespro.ru --- src/common/base64.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/common/base64.c') diff --git a/src/common/base64.c b/src/common/base64.c index 2943ac76520..ec4eb49382c 100644 --- a/src/common/base64.c +++ b/src/common/base64.c @@ -224,7 +224,7 @@ int pg_b64_enc_len(int srclen) { /* 3 bytes will be converted to 4 */ - return (srclen + 2) * 4 / 3; + return (srclen + 2) / 3 * 4; } /* -- cgit v1.2.3