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

Commit17792bf

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 parented12700 commit17792bf

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
@@ -2804,6 +2804,8 @@ GetLockConflicts(const LOCKTAG *locktag, LOCKMODE lockmode)
28042804
* on this lockable object.
28052805
*/
28062806
LWLockRelease(partitionLock);
2807+
vxids[count].backendId=InvalidBackendId;
2808+
vxids[count].localTransactionId=InvalidLocalTransactionId;
28072809
returnvxids;
28082810
}
28092811

@@ -2857,6 +2859,8 @@ GetLockConflicts(const LOCKTAG *locktag, LOCKMODE lockmode)
28572859
if (count>MaxBackends)/* should never happen */
28582860
elog(PANIC,"too many conflicting locks found");
28592861

2862+
vxids[count].backendId=InvalidBackendId;
2863+
vxids[count].localTransactionId=InvalidLocalTransactionId;
28602864
returnvxids;
28612865
}
28622866

0 commit comments

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