diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2017-12-11 16:33:20 -0500 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2017-12-11 16:33:43 -0500 |
commit | 05f239e4a19f3388c73df093243d4b5b4c4e7f56 (patch) | |
tree | 6b78cae17a5d227485fcb54045550fd0ec58df4a /src/backend/access/gist/gistutil.c | |
parent | 470de6a24d37d630515c5112e1bebe03a742b2c8 (diff) | |
download | postgresql-05f239e4a19f3388c73df093243d4b5b4c4e7f56.tar.gz postgresql-05f239e4a19f3388c73df093243d4b5b4c4e7f56.zip |
Fix corner-case coredump in _SPI_error_callback().
I noticed that _SPI_execute_plan initially sets spierrcontext.arg = NULL,
and only fills it in some time later. If an error were to happen in
between, _SPI_error_callback would try to dereference the null pointer.
This is unlikely --- there's not much between those points except
push-snapshot calls --- but it's clearly not impossible. Tweak the
callback to do nothing if the pointer isn't set yet.
It's been like this for awhile, so back-patch to all supported branches.
Diffstat (limited to 'src/backend/access/gist/gistutil.c')
0 files changed, 0 insertions, 0 deletions