Commit9598afa

committed

Fix one overflow and one signedness error, caused by the patch to calculate

OLDSERXID_MAX_PAGE based on BLCKSZ. MSVC compiler warned about these.

1 parent7544064 commit9598afaCopy full SHA for 9598afa

File tree

-2

lines changed

-2

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -311,7 +311,7 @@ static SlruCtlData OldSerXidSlruCtlData;`
`311`	`311`	`* transactions and the maximum that SLRU supports.`
`312`	`312`	`*/`
`313`	`313`	`#defineOLDSERXID_MAX_PAGEMin(SLRU_PAGES_PER_SEGMENT * 0x10000 - 1, \`
`314`		`-(MaxTransactionId + 1) / OLDSERXID_ENTRIESPERPAGE - 1)`
	`314`	`+(MaxTransactionId) / OLDSERXID_ENTRIESPERPAGE)`
`315`	`315`
`316`	`316`	`#defineOldSerXidNextPage(page) (((page) >= OLDSERXID_MAX_PAGE) ? 0 : (page) + 1)`
`317`	`317`
`@@ -767,7 +767,7 @@ OldSerXidPagePrecedesLogically(int p, int q)`
`767`	`767`	`diff=p-q;`
`768`	`768`	`if (diff >= ((OLDSERXID_MAX_PAGE+1) /2))`
`769`	`769`	`diff-=OLDSERXID_MAX_PAGE+1;`
`770`		`-elseif (diff<-((OLDSERXID_MAX_PAGE+1) /2))`
	`770`	`+elseif (diff<-((int) (OLDSERXID_MAX_PAGE+1) /2))`
`771`	`771`	`diff+=OLDSERXID_MAX_PAGE+1;`
`772`	`772`	`returndiff<0;`
`773`	`773`	`}`

Comments

(0)