Commit75bcba6

committed

Remove extra check_stack_depth() from dropconstraint_internal()

The second check was added byd57b7cc without taking into account thereis already a check sinceb0f7dd9.Reported-by: Ashutosh Bapat, Alexander LakhinDiscussion:https://postgr.es/m/CAExHW5sBZWDjeBUFs_ehEDM%2BuhWxTiBkPbLiat7ZjWkb-DWQWw%40mail.gmail.com

1 parentfcd210d commit75bcba6Copy full SHA for 75bcba6

File tree

-3

lines changed

-3

lines changed

Lines changed: 0 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -12738,9 +12738,6 @@ dropconstraint_internal(Relation rel, HeapTuple constraintTup, DropBehavior beha`
`12738`	`12738`	`/* Guard against stack overflow due to overly deep inheritance tree. */`
`12739`	`12739`	`check_stack_depth();`
`12740`	`12740`
`12741`		`-/* since this function recurses, it could be driven to stack overflow */`
`12742`		`-check_stack_depth();`
`12743`		`-`
`12744`	`12741`	`/* At top level, permission check was done in ATPrepCmd, else do it */`
`12745`	`12742`	`if (recursing)`
`12746`	`12743`	`ATSimplePermissions(AT_DropConstraint, rel, ATT_TABLE \| ATT_FOREIGN_TABLE);`

Comments

(0)