Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit8251acf

Browse files
committed
Properly terminate the array returned by GetLockConflicts().
GetLockConflicts() has for a long time not properly terminated thereturned array. During normal processing the returned array is zeroinitialized which, while not pretty, is sufficient to be recognized asa invalid virtual transaction id. But the HotStandby case is more thanaesthetically broken: The allocated (and reused) array is neitherzeroed upon allocation, nor reinitialized, nor terminated.Not having a terminating element means that the end of the array willnot be recognized and that recovery conflict handling will thus readahead into adjacent memory. Only terminating when hitting memorycontent that looks like a invalid virtual transaction id. Luckilythis seems so far not have caused significant problems, besides makingrecovery conflict more expensive.Discussion: 20150127142713.GD29457@awork2.anarazel.deBackpatch into all supported branches.
1 parent61729e9 commit8251acf

File tree

1 file changed

+4
-0
lines changed
  • src/backend/storage/lmgr

1 file changed

+4
-0
lines changed

‎src/backend/storage/lmgr/lock.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2738,6 +2738,8 @@ GetLockConflicts(const LOCKTAG *locktag, LOCKMODE lockmode)
27382738
* on this lockable object.
27392739
*/
27402740
LWLockRelease(partitionLock);
2741+
vxids[count].backendId=InvalidBackendId;
2742+
vxids[count].localTransactionId=InvalidLocalTransactionId;
27412743
returnvxids;
27422744
}
27432745

@@ -2791,6 +2793,8 @@ GetLockConflicts(const LOCKTAG *locktag, LOCKMODE lockmode)
27912793
if (count>MaxBackends)/* should never happen */
27922794
elog(PANIC,"too many conflicting locks found");
27932795

2796+
vxids[count].backendId=InvalidBackendId;
2797+
vxids[count].localTransactionId=InvalidLocalTransactionId;
27942798
returnvxids;
27952799
}
27962800

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp