NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitfe30365

committed

Fix race condition in statext_store().

Must hold some lock on the pg_statistic_ext_data catalog *before*we look up the tuple we aim to replace. Otherwise a concurrentVACUUM FULL or similar operation could move it to a different TID,leaving us trying to replace the wrong tuple.Back-patch to v12 where this got broken.Credit goes to Dean Rasheed; I'm just doing the clerical work.Discussion:https://postgr.es/m/CAEZATCU0zHMDiQV0g8P2U+YSP9C1idUPrn79DajsbonwkN0xvQ@mail.gmail.com

1 parent33cd0e5 commitfe30365Copy full SHA for fe30365

File tree

1 file changed

-3

lines changed

src/backend/statistics
- extended_stats.c

1 file changed

-3

lines changed

`‎src/backend/statistics/extended_stats.c`

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -493,12 +493,14 @@ statext_store(Oid statOid,`
`493`	`493`	`MVNDistinctndistinct,MVDependenciesdependencies,`
`494`	`494`	`MCVListmcv,VacAttrStats*stats)`
`495`	`495`	`{`
	`496`	`+Relationpg_stextdata;`
`496`	`497`	`HeapTuplestup,`
`497`	`498`	`oldtup;`
`498`	`499`	`Datumvalues[Natts_pg_statistic_ext_data];`
`499`	`500`	`boolnulls[Natts_pg_statistic_ext_data];`
`500`	`501`	`boolreplaces[Natts_pg_statistic_ext_data];`
`501`		`-Relationpg_stextdata;`
	`502`	`+`
	`503`	`+pg_stextdata=table_open(StatisticExtDataRelationId,RowExclusiveLock);`
`502`	`504`
`503`	`505`	`memset(nulls, true,sizeof(nulls));`
`504`	`506`	`memset(replaces, false,sizeof(replaces));`
`@@ -542,8 +544,6 @@ statext_store(Oid statOid,`
`542`	`544`	`elog(ERROR,"cache lookup failed for statistics object %u",statOid);`
`543`	`545`
`544`	`546`	`/* replace it */`
`545`		`-pg_stextdata=table_open(StatisticExtDataRelationId,RowExclusiveLock);`
`546`		`-`
`547`	`547`	`stup=heap_modify_tuple(oldtup,`
`548`	`548`	`RelationGetDescr(pg_stextdata),`
`549`	`549`	`values,`

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!

Commitfe30365

File tree

1 file changed

1 file changed

`‎src/backend/statistics/extended_stats.c`

0 commit comments