NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitdd4e0a3

committed

Fix oversight in pg_dump's handling of extension configuration tables.

If an extension has not been selected to be dumped (perhaps because ofa --schema or --table switch), the contents of its configuration tablessurely should not get dumped either. Per gripe fromHubert Depesz Lubaczewski.

1 parente565eff commitdd4e0a3Copy full SHA for dd4e0a3

File tree

1 file changed

-2

lines changed

src/bin/pg_dump
- pg_dump.c

1 file changed

-2

lines changed

`‎src/bin/pg_dump/pg_dump.c`

Lines changed: 7 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -13794,13 +13794,18 @@ getExtensionMembership(ExtensionInfo extinfo[], int numExtensions)`
`13794`	`13794`	`*/`
`13795`	`13795`	`for (i=0;i<numExtensions;i++)`
`13796`	`13796`	`{`
`13797`		`-char*extconfig=extinfo[i].extconfig;`
`13798`		`-char*extcondition=extinfo[i].extcondition;`
	`13797`	`+ExtensionInfo*curext=&(extinfo[i]);`
	`13798`	`+char*extconfig=curext->extconfig;`
	`13799`	`+char*extcondition=curext->extcondition;`
`13799`	`13800`	`char**extconfigarray=NULL;`
`13800`	`13801`	`char**extconditionarray=NULL;`
`13801`	`13802`	`intnconfigitems;`
`13802`	`13803`	`intnconditionitems;`
`13803`	`13804`
	`13805`	`+/* Tables of not-to-be-dumped extensions shouldn't be dumped */`
	`13806`	`+if (!curext->dobj.dump)`
	`13807`	`+continue;`
	`13808`	`+`
`13804`	`13809`	`if (parsePGArray(extconfig,&extconfigarray,&nconfigitems)&&`
`13805`	`13810`	`parsePGArray(extcondition,&extconditionarray,&nconditionitems)&&`
`13806`	`13811`	`nconfigitems==nconditionitems)`

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!

Commitdd4e0a3

File tree

1 file changed

1 file changed

`‎src/bin/pg_dump/pg_dump.c`

0 commit comments