NotificationsYou must be signed in to change notification settings
Fork28
Star153

Commitd57d615

committed

Add missing check_collation_set call to bpcharne().

We should throw an error for indeterminate collation, but bpcharne()was missing that logic, resulting in a much less user-friendly error(either an assertion failure or "cache lookup failed for collation 0").Per report from Manuel Rigger. Back-patch to v12 where the mistakecame in, evidently in commit5e1963f. (Before non-deterministiccollations, this function wasn't collation sensitive.)Discussion:https://postgr.es/m/CA+u7OA4HOjtymxAbuGNh4-X_2R0Lw5n01tzvP8E5-i-2gQXYWA@mail.gmail.com

1 parent0cafdd0 commitd57d615Copy full SHA for d57d615

File tree

1 file changed

-0

lines changed

src/backend/utils/adt
- varchar.c

1 file changed

-0

lines changed

`‎src/backend/utils/adt/varchar.c‎`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -784,6 +784,8 @@ bpcharne(PG_FUNCTION_ARGS)`
`784`	`784`	`boolresult;`
`785`	`785`	`Oidcollid=PG_GET_COLLATION();`
`786`	`786`
	`787`	`+check_collation_set(collid);`
	`788`	`+`
`787`	`789`	`len1=bcTruelen(arg1);`
`788`	`790`	`len2=bcTruelen(arg2);`
`789`	`791`

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!

Commitd57d615

File tree

1 file changed

1 file changed

`‎src/backend/utils/adt/varchar.c‎`

0 commit comments