pgsql: Avoid crashes in contrib/intarray gist__int_ops (bug #15518)

From: Andrew Gierth <rhodiumtoad(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Avoid crashes in contrib/intarray gist__int_ops (bug #15518)
Date: 2018-11-24 08:53:34
Message-ID: E1gQTh0-0007Ba-Mn@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Avoid crashes in contrib/intarray gist__int_ops (bug #15518)

1. Integer overflow in internal_size could result in memory corruption
in decompression since a zero-length array would be allocated and then
written to. This leads to crashes or corruption when traversing an
index which has been populated with sufficiently sparse values. Fix by
using int64 for computations and checking for overflow.

2. Integer overflow in g_int_compress could cause pessimal merge
choices, resulting in unnecessarily large ranges (which would in turn
trigger issue 1 above). Fix by using int64 again.

3. Even without overflow, array sizes could become large enough to
cause unexplained memory allocation errors. Fix by capping the sizes
to a safe limit and report actual errors pointing at gist__intbig_ops
as needed.

4. Large inputs to the compression function always consist of large
runs of consecutive integers, and the compression loop was processing
these one at a time in an O(N^2) manner with a lot of overhead. The
expected runtime of this function could easily exceed 6 months for a
single call as a result. Fix by performing a linear-time first pass,
which reduces the worst case to something on the order of seconds.

Backpatch all the way, since this has been wrong forever.

Per bug #15518 from report from irc user "dymk", analysis and patch by
me.

Discussion: https://postgr.es/m/15518-799e426c3b4f8358@postgresql.org

Branch
------
REL9_5_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/f0bfc7a2b13a6cd48b6cea253b0e9f918e457c6d

Modified Files
--------------
contrib/intarray/_int_gist.c | 71 +++++++++++++++++++++++++++++++++++++++-----
contrib/intarray/_int_tool.c | 12 +++++---
2 files changed, 71 insertions(+), 12 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Christoph Berg 2018-11-24 14:38:45 Re: pgsql: Add WL_EXIT_ON_PM_DEATH pseudo-event.
Previous Message Tom Lane 2018-11-24 04:49:33 pgsql: Adjust new test case for more portability.