public inbox for [email protected]help / color / mirror / Atom feed
pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo 6+ messages / 1 participants [nested] [flat]
* pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo @ 2026-02-14 20:17 Noah Misch <[email protected]> 0 siblings, 0 replies; 6+ messages in thread From: Noah Misch @ 2026-02-14 20:17 UTC (permalink / raw) To: [email protected] pg_mblen_range, pg_mblen_with_len: Valgrind after encoding ereport. The prior order caused spurious Valgrind errors. They're spurious because the ereport(ERROR) non-local exit discards the pointer in question. pg_mblen_cstr() ordered the checks correctly, but these other two did not. Back-patch to v14, like commit 1e7fe06c10c0a8da9dd6261a6be8d405dc17c728. Reviewed-by: Thomas Munro <[email protected]> Discussion: https://postgr.es/m/[email protected] Backpatch-through: 14 Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/4644f8b23bb8cd5cf3454bcd69bc28a5fd5623a9 Modified Files -------------- src/backend/utils/mb/mbutils.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) ^ permalink raw reply [nested|flat] 6+ messages in thread
* pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo @ 2026-02-14 20:17 Noah Misch <[email protected]> 0 siblings, 0 replies; 6+ messages in thread From: Noah Misch @ 2026-02-14 20:17 UTC (permalink / raw) To: [email protected] pg_mblen_range, pg_mblen_with_len: Valgrind after encoding ereport. The prior order caused spurious Valgrind errors. They're spurious because the ereport(ERROR) non-local exit discards the pointer in question. pg_mblen_cstr() ordered the checks correctly, but these other two did not. Back-patch to v14, like commit 1e7fe06c10c0a8da9dd6261a6be8d405dc17c728. Reviewed-by: Thomas Munro <[email protected]> Discussion: https://postgr.es/m/[email protected] Backpatch-through: 14 Branch ------ REL_16_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/45eb47230d93a652873e9b0f94e2f1b04181a8cc Modified Files -------------- src/backend/utils/mb/mbutils.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) ^ permalink raw reply [nested|flat] 6+ messages in thread
* pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo @ 2026-02-14 20:17 Noah Misch <[email protected]> 0 siblings, 0 replies; 6+ messages in thread From: Noah Misch @ 2026-02-14 20:17 UTC (permalink / raw) To: [email protected] pg_mblen_range, pg_mblen_with_len: Valgrind after encoding ereport. The prior order caused spurious Valgrind errors. They're spurious because the ereport(ERROR) non-local exit discards the pointer in question. pg_mblen_cstr() ordered the checks correctly, but these other two did not. Back-patch to v14, like commit 1e7fe06c10c0a8da9dd6261a6be8d405dc17c728. Reviewed-by: Thomas Munro <[email protected]> Discussion: https://postgr.es/m/[email protected] Backpatch-through: 14 Branch ------ REL_15_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/5b305ebcc9b56b053415997625c6425aa7b878b2 Modified Files -------------- src/backend/utils/mb/mbutils.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) ^ permalink raw reply [nested|flat] 6+ messages in thread
* pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo @ 2026-02-14 20:17 Noah Misch <[email protected]> 0 siblings, 0 replies; 6+ messages in thread From: Noah Misch @ 2026-02-14 20:17 UTC (permalink / raw) To: [email protected] pg_mblen_range, pg_mblen_with_len: Valgrind after encoding ereport. The prior order caused spurious Valgrind errors. They're spurious because the ereport(ERROR) non-local exit discards the pointer in question. pg_mblen_cstr() ordered the checks correctly, but these other two did not. Back-patch to v14, like commit 1e7fe06c10c0a8da9dd6261a6be8d405dc17c728. Reviewed-by: Thomas Munro <[email protected]> Discussion: https://postgr.es/m/[email protected] Backpatch-through: 14 Branch ------ REL_14_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/44fc85bbf99029e83592d4992c1d700951e59875 Modified Files -------------- src/backend/utils/mb/mbutils.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) ^ permalink raw reply [nested|flat] 6+ messages in thread
* pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo @ 2026-02-14 20:17 Noah Misch <[email protected]> 0 siblings, 0 replies; 6+ messages in thread From: Noah Misch @ 2026-02-14 20:17 UTC (permalink / raw) To: [email protected] pg_mblen_range, pg_mblen_with_len: Valgrind after encoding ereport. The prior order caused spurious Valgrind errors. They're spurious because the ereport(ERROR) non-local exit discards the pointer in question. pg_mblen_cstr() ordered the checks correctly, but these other two did not. Back-patch to v14, like commit 1e7fe06c10c0a8da9dd6261a6be8d405dc17c728. Reviewed-by: Thomas Munro <[email protected]> Discussion: https://postgr.es/m/[email protected] Backpatch-through: 14 Branch ------ REL_18_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/4174e41b9e9ee552fff3501e73a1897c9f1eecaa Modified Files -------------- src/backend/utils/mb/mbutils.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) ^ permalink raw reply [nested|flat] 6+ messages in thread
* pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo @ 2026-02-14 20:17 Noah Misch <[email protected]> 0 siblings, 0 replies; 6+ messages in thread From: Noah Misch @ 2026-02-14 20:17 UTC (permalink / raw) To: [email protected] pg_mblen_range, pg_mblen_with_len: Valgrind after encoding ereport. The prior order caused spurious Valgrind errors. They're spurious because the ereport(ERROR) non-local exit discards the pointer in question. pg_mblen_cstr() ordered the checks correctly, but these other two did not. Back-patch to v14, like commit 1e7fe06c10c0a8da9dd6261a6be8d405dc17c728. Reviewed-by: Thomas Munro <[email protected]> Discussion: https://postgr.es/m/[email protected] Backpatch-through: 14 Branch ------ REL_17_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/8e73530f1546ff6a052786845fce689b70ec2909 Modified Files -------------- src/backend/utils/mb/mbutils.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) ^ permalink raw reply [nested|flat] 6+ messages in thread
end of thread, other threads:[~2026-02-14 20:17 UTC | newest] Thread overview: 6+ messages (download: mbox mbox.gz follow: Atom feed) -- links below jump to the message on this page -- 2026-02-14 20:17 pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo Noah Misch <[email protected]> 2026-02-14 20:17 pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo Noah Misch <[email protected]> 2026-02-14 20:17 pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo Noah Misch <[email protected]> 2026-02-14 20:17 pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo Noah Misch <[email protected]> 2026-02-14 20:17 pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo Noah Misch <[email protected]> 2026-02-14 20:17 pgsql: pg_mblen_range, pg_mblen_with_len: Valgrind after encoding erepo Noah Misch <[email protected]>
This inbox is served by agora; see mirroring instructions for how to clone and mirror all data and code used for this inbox