diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2015-01-24 13:25:22 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2015-01-24 13:25:22 -0500 |
| commit | 91964c3ed1c49d9a8670d3f85a660181cc541c7c (patch) | |
| tree | 56ed1eac4dc206d5adc41a45648ad51da471f5cc /src/backend/utils/adt/network_gist.c | |
| parent | d51d4ff311d01de8521acedb0a6f7c242648a231 (diff) | |
| download | postgresql-91964c3ed1c49d9a8670d3f85a660181cc541c7c.tar.gz postgresql-91964c3ed1c49d9a8670d3f85a660181cc541c7c.zip | |
Fix unsafe coding in ReorderBufferCommit().
"iterstate" must be marked volatile since it's changed inside the PG_TRY
block and then used in the PG_CATCH stanza. Noted by Mark Wilding of
Salesforce. (We really need to see if we can't get the C compiler to warn
about this.)
Also, reset iterstate to NULL after the mainline ReorderBufferIterTXNFinish
call, to ensure the PG_CATCH block doesn't try to do that a second time.
Diffstat (limited to 'src/backend/utils/adt/network_gist.c')
0 files changed, 0 insertions, 0 deletions