diff options
author | Alexander Korotkov <akorotkov@postgresql.org> | 2019-09-20 01:10:56 +0300 |
---|---|---|
committer | Alexander Korotkov <akorotkov@postgresql.org> | 2019-09-20 01:19:08 +0300 |
commit | 8c8a267201bebb0edeaab2a76b7d3bcc9739924f (patch) | |
tree | 2d8b34940beeeee24fb2e5825ee593575e8dc069 /src/backend/utils/adt/jsonfuncs.c | |
parent | 6cae9d2c10e151f741e7bc64a8b70bb2615c367c (diff) | |
download | postgresql-8c8a267201bebb0edeaab2a76b7d3bcc9739924f.tar.gz postgresql-8c8a267201bebb0edeaab2a76b7d3bcc9739924f.zip |
Fix freeing old values in index_store_float8_orderby_distances()
6cae9d2c10 has added an error in freeing old values in
index_store_float8_orderby_distances() function. It looks for old value in
scan->xs_orderbynulls[i] after setting a new value there.
This commit fixes that. Also it removes short-circuit in handling
distances == NULL situation. Now distances == NULL will be treated the same
way as array with all null distances. That is, previous values will be freed
if any.
Reported-by: Tom Lane, Nikita Glukhov
Discussion: https://postgr.es/m/CAPpHfdu2wcoAVAm3Ek66rP%3Duo_C-D84%2B%2Buf1VEcbyi_caBXWCA%40mail.gmail.com
Discussion: https://postgr.es/m/426580d3-a668-b9d1-7b8e-f74d1a6524e0%40postgrespro.ru
Backpatch-through: 12
Diffstat (limited to 'src/backend/utils/adt/jsonfuncs.c')
0 files changed, 0 insertions, 0 deletions