Commitd4b3440

tglsfdc

authored and

pull[bot]

committed

Fix data-corruption hazard in WAL-logged CREATE DATABASE.

RelationCopyStorageUsingBuffer thought it could skip copyingempty pages, but of course that does not work at all, becausesubsequent blocks will be out of place.Also fix it to acquire share lock on the source buffer. It *might*be safe to not do that, but it's not very certain, and I don't thinkthis code deserves any benefit of the doubt.Dilip Kumar, per complaint from meDiscussion:https://postgr.es/m/3679800.1659654066@sss.pgh.pa.us

1 parentcb31e2a commitd4b3440Copy full SHA for d4b3440

File tree

1 file changed

-7

lines changed

src/backend/storage/buffer
- bufmgr.c

1 file changed

-7

lines changed

`‎src/backend/storage/buffer/bufmgr.c`

Lines changed: 3 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -3741,23 +3741,19 @@ RelationCopyStorageUsingBuffer(Relation src, Relation dst, ForkNumber forkNum,`
`3741`	`3741`	`srcBuf=ReadBufferWithoutRelcache(src->rd_locator,forkNum,blkno,`
`3742`	`3742`	`RBM_NORMAL,bstrategy_src,`
`3743`	`3743`	`permanent);`
	`3744`	`+LockBuffer(srcBuf,BUFFER_LOCK_SHARE);`
`3744`	`3745`	`srcPage=BufferGetPage(srcBuf);`
`3745`		`-if (PageIsNew(srcPage)\|\|PageIsEmpty(srcPage))`
`3746`		`-{`
`3747`		`-ReleaseBuffer(srcBuf);`
`3748`		`-continue;`
`3749`		`-}`
`3750`	`3746`
`3751`	`3747`	`/* Use P_NEW to extend the destination relation. */`
`3752`	`3748`	`dstBuf=ReadBufferWithoutRelcache(dst->rd_locator,forkNum,P_NEW,`
`3753`	`3749`	`RBM_NORMAL,bstrategy_dst,`
`3754`	`3750`	`permanent);`
`3755`	`3751`	`LockBuffer(dstBuf,BUFFER_LOCK_EXCLUSIVE);`
	`3752`	`+dstPage=BufferGetPage(dstBuf);`
`3756`	`3753`
`3757`	`3754`	`START_CRIT_SECTION();`
`3758`	`3755`
`3759`	`3756`	`/* Copy page data from the source to the destination. */`
`3760`		`-dstPage=BufferGetPage(dstBuf);`
`3761`	`3757`	`memcpy(dstPage,srcPage,BLCKSZ);`
`3762`	`3758`	`MarkBufferDirty(dstBuf);`
`3763`	`3759`
`@@ -3768,7 +3764,7 @@ RelationCopyStorageUsingBuffer(Relation src, Relation dst, ForkNumber forkNum,`
`3768`	`3764`	`END_CRIT_SECTION();`
`3769`	`3765`
`3770`	`3766`	`UnlockReleaseBuffer(dstBuf);`
`3771`		`-ReleaseBuffer(srcBuf);`
	`3767`	`+UnlockReleaseBuffer(srcBuf);`
`3772`	`3768`	`}`
`3773`	`3769`	`}`
`3774`	`3770`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitd4b3440

File tree

1 file changed

1 file changed

`‎src/backend/storage/buffer/bufmgr.c`

0 commit comments