diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2022-12-21 17:51:50 -0500 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2022-12-21 17:51:50 -0500 |
commit | d35f1d485c392ce058382794742ce246e47643ec (patch) | |
tree | 342c62b3be90cd86ea4ab872ad35b83f16f42d76 /src/backend/executor/nodeSubplan.c | |
parent | e39856d1f027a91f20e7c8cac7a5c844b74c4962 (diff) | |
download | postgresql-d35f1d485c392ce058382794742ce246e47643ec.tar.gz postgresql-d35f1d485c392ce058382794742ce246e47643ec.zip |
Fix contrib/seg to be more wary of long input numbers.
seg stores the number of significant digits in an input number
in a "char" field. If char is signed, and the input is more than
127 digits long, the count can read out as negative causing
seg_out() to print garbage (or, if you're really unlucky,
even crash).
To fix, clamp the digit count to be not more than FLT_DIG.
(In theory this loses some information about what the original
input was, but it doesn't seem like useful information; it would
not survive dump/restore in any case.)
Also, in case there are stored values of the seg type containing
bad data, add a clamp in seg_out's restore() subroutine.
Per bug #17725 from Robins Tharakan. It's been like this
forever, so back-patch to all supported branches.
Discussion: https://postgr.es/m/17725-0a09313b67fbe86e@postgresql.org
Diffstat (limited to 'src/backend/executor/nodeSubplan.c')
0 files changed, 0 insertions, 0 deletions