diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2025-03-08 11:24:22 -0500 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2025-03-08 11:25:01 -0500 |
commit | f1c1bafcdf76dbcdfcba506813bea5d939341631 (patch) | |
tree | 1fe2bb1aee9186a77d70f1a6a07930e3f15249ff /src/backend/executor/nodeAgg.c | |
parent | 308d0d443770a0dfe49e7e253792dc6b3a149fb0 (diff) |
Clear errno before calling strtol() in spell.c.
Per POSIX, a caller of strtol() that wishes to check for errors must
set errno to 0 beforehand. Several places in spell.c neglected that,
so that they risked delivering a false overflow error in case errno
had been ERANGE already. Given the lack of field reports, this case
may be unreachable at present --- but it's surely trouble waiting to
happen, so fix it.
Author: Jacob Brazeal <jacob.brazeal@gmail.com>
Discussion: https://postgr.es/m/CA+COZaBhsq6EromFm+knMJfzK6nTpG23zJ+K2=nfUQQXcj_xcQ@mail.gmail.com
Backpatch-through: 13
Diffstat (limited to 'src/backend/executor/nodeAgg.c')
0 files changed, 0 insertions, 0 deletions