| From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
|---|---|
| To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | pgsql: Fix heap-buffer-overflow in pglz_decompress() on corrupt input. |
| Date: | 2026-04-10 13:32:37 |
| Message-ID: | E1wBByO-000Iyp-24@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Fix heap-buffer-overflow in pglz_decompress() on corrupt input.
When decoding a match tag, pglz_decompress() reads 2 bytes (or 3
for extended-length matches) from the source buffer before checking
whether enough data remains. The existing bounds check (sp > srcend)
occurs after the reads, so truncated compressed data that ends
mid-tag causes a read past the allocated buffer.
Fix by validating that sufficient source bytes are available before
reading each part of the match tag. The post-read sp > srcend
check is no longer needed and is removed.
Found by fuzz testing with libFuzzer and AddressSanitizer.
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/2b5ba2a0a141f621f61cb732d776dc78269f879b
Modified Files
--------------
src/common/pg_lzcompress.c | 27 +++++++++++++++++++--------
1 file changed, 19 insertions(+), 8 deletions(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrew Dunstan | 2026-04-10 13:57:30 | pgsql: Revert "Add built-in fuzzing harnesses for security testing." |
| Previous Message | Aleksander Alekseev | 2026-04-10 10:35:39 | Re: pgsql: Allow autovacuum to use parallel vacuum workers. |