NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitee4ddcb

committed

Fix targetRelation initializiation in prepsecurity

In6f9bd50, we modifiedexpand_security_quals() to tell expand_security_qual() about when thecurrent RTE was the targetRelation. Unfortunately, that commitinitialized the targetRelation variable used outside of the loop overthe RTEs instead of at the start of it.This patch moves the variable and the initialization of it into theloop, where it should have been to begin with.Pointed out by Dean Rasheed.Back-patch to 9.4 as the original commit was.

1 parent8abb3cd commitee4ddcbCopy full SHA for ee4ddcb

File tree

1 file changed

-2

lines changed

src/backend/optimizer/prep
- prepsecurity.c

1 file changed

-2

lines changed

`‎src/backend/optimizer/prep/prepsecurity.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,6 @@ expand_security_quals(PlannerInfo root, List tlist)`
`63`	`63`	`Query*parse=root->parse;`
`64`	`64`	`intrt_index;`
`65`	`65`	`ListCell*cell;`
`66`		`-booltargetRelation= false;`
`67`	`66`
`68`	`67`	`/*`
`69`	`68`	`* Process each RTE in the rtable list.`
`@@ -74,7 +73,8 @@ expand_security_quals(PlannerInfo root, List tlist)`
`74`	`73`	`rt_index=0;`
`75`	`74`	`foreach(cell,parse->rtable)`
`76`	`75`	`{`
`77`		`-RangeTblEntryrte= (RangeTblEntry)lfirst(cell);`
	`76`	`+booltargetRelation= false;`
	`77`	`+RangeTblEntryrte= (RangeTblEntry)lfirst(cell);`
`78`	`78`
`79`	`79`	`rt_index++;`
`80`	`80`

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!

Commitee4ddcb

File tree

1 file changed

1 file changed

`‎src/backend/optimizer/prep/prepsecurity.c`

0 commit comments