diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2015-09-20 16:48:44 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2015-09-20 16:48:44 -0400 |
| commit | 7e6e3bdd3c36d77b8a611dbf5e8c72164bc6108b (patch) | |
| tree | d6e1774a6fc562abbea479e0502c602b8c92f40b /src/backend/parser/parse_param.c | |
| parent | f6b701c0b41a55a167ae0ae6f21e068e57005039 (diff) | |
Be more wary about partially-valid LOCALLOCK data in RemoveLocalLock().
RemoveLocalLock() must consider the possibility that LockAcquireExtended()
failed to palloc the initial space for a locallock's lockOwners array.
I had evidently meant to cope with this hazard when the code was originally
written (commit 1785acebf2ed14fd66955e2d9a55d77a025f418d), but missed that
the pfree needed to be protected with an if-test. Just to make sure things
are left in a clean state, reset numLockOwners as well.
Per low-memory testing by Andreas Seltenreich. Back-patch to all supported
branches.
Diffstat (limited to 'src/backend/parser/parse_param.c')
0 files changed, 0 insertions, 0 deletions