Commite06dd0c

robertmhaas

authored and

pull[bot]

committed

Fix brain fade ine530be2.

The BoolGetDatum() call ended up in the wrong place. It should beapplied when we, err, want to convert a bool to a datum.Thanks to Tom Lane for noticing this.Discussion:http://postgr.es/m/2511599.1658861964@sss.pgh.pa.us

1 parent56e68f5 commite06dd0cCopy full SHA for e06dd0c

File tree

1 file changed

-2

lines changed

src/backend/commands
- user.c

1 file changed

-2

lines changed

`‎src/backend/commands/user.c‎`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -693,14 +693,14 @@ AlterRole(ParseState pstate, AlterRoleStmt stmt)`
`693`	`693`	`*/`
`694`	`694`	`if (dissuper)`
`695`	`695`	`{`
`696`		`-boolshould_be_super=BoolGetDatum(boolVal(dissuper->arg));`
	`696`	`+boolshould_be_super=boolVal(dissuper->arg);`
`697`	`697`
`698`	`698`	`if (!should_be_super&&roleid==BOOTSTRAP_SUPERUSERID)`
`699`	`699`	`ereport(ERROR,`
`700`	`700`	`(errcode(ERRCODE_INSUFFICIENT_PRIVILEGE),`
`701`	`701`	`errmsg("permission denied: bootstrap user must be superuser")));`
`702`	`702`
`703`		`-new_record[Anum_pg_authid_rolsuper-1]=should_be_super;`
	`703`	`+new_record[Anum_pg_authid_rolsuper-1]=BoolGetDatum(should_be_super);`
`704`	`704`	`new_record_repl[Anum_pg_authid_rolsuper-1]= true;`
`705`	`705`	`}`
`706`	`706`

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

Commite06dd0c

File tree

1 file changed

1 file changed

`‎src/backend/commands/user.c‎`

0 commit comments