]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix corner case with 16kB-long decompression in pgcrypto, take 2
authorMichael Paquier <michael@paquier.xyz>
Mon, 27 Jul 2020 06:59:13 +0000 (15:59 +0900)
committerMichael Paquier <michael@paquier.xyz>
Mon, 27 Jul 2020 06:59:13 +0000 (15:59 +0900)
commit8a60f541f3e4ef2177fa07e8f196b330df617e34
tree40afd84a6a239a5b1b7b4274b21e52f7e653651b
parentccf964a801007a5fe7f9bef4cfb55b1263d23b1e
Fix corner case with 16kB-long decompression in pgcrypto, take 2

A compressed stream may end with an empty packet.  In this case
decompression finishes before reading the empty packet and the
remaining stream packet causes a failure in reading the following
data.  This commit makes sure to consume such extra data, avoiding a
failure when decompression the data.  This corner case was reproducible
easily with a data length of 16kB, and existed since e94dd6a.  A cheap
regression test is added to cover this case based on a random,
incompressible string.

The first attempt of this patch has allowed to find an older failure
within the compression logic of pgcrypto, fixed by b9b6105.  This
involved SLES 15 with z390 where a custom flavor of libz gets used.
Bonus thanks to Mark Wong for providing access to the specific
environment.

Reported-by: Frank Gagnepain
Author: Kyotaro Horiguchi, Michael Paquier
Reviewed-by: Tom Lane
Discussion: https://postgr.es/m/16476-692ef7b84e5fb893@postgresql.org
Backpatch-through: 9.5
contrib/pgcrypto/expected/pgp-compression.out
contrib/pgcrypto/pgp-compress.c
contrib/pgcrypto/sql/pgp-compression.sql