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

Commit78f8c67

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 parent37e0f13 commit78f8c67

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
@@ -1990,6 +1990,8 @@ GetLockConflicts(const LOCKTAG *locktag, LOCKMODE lockmode)
19901990
* on this lockable object.
19911991
*/
19921992
LWLockRelease(partitionLock);
1993+
vxids[count].backendId=InvalidBackendId;
1994+
vxids[count].localTransactionId=InvalidLocalTransactionId;
19931995
returnvxids;
19941996
}
19951997

@@ -2035,6 +2037,8 @@ GetLockConflicts(const LOCKTAG *locktag, LOCKMODE lockmode)
20352037
if (count>MaxBackends)/* should never happen */
20362038
elog(PANIC,"too many conflicting locks found");
20372039

2040+
vxids[count].backendId=InvalidBackendId;
2041+
vxids[count].localTransactionId=InvalidLocalTransactionId;
20382042
returnvxids;
20392043
}
20402044

0 commit comments

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