Fix harmless LC_COLLATE[_MASK] confusion.

macdice · macdice · commit c1bb534bae43 · 2024-08-19T22:21:59.000+12:00
Commit ca051d8 called newlocale(LC_COLLATE, ...) instead of newlocale(LC_COLLATE_MASK, ...), in code reached only on FreeBSD. They have the same value on that OS, explaining why it worked. Fix. Back-patch to 14, where ca051d8 landed.
diff --git a/src/backend/utils/adt/pg_locale.c b/src/backend/utils/adt/pg_locale.c
@@ -1722,7 +1722,7 @@ get_collation_actual_version(char collprovider, const char *collcollate)
 		locale_t	loc;
 
 		/* Look up FreeBSD collation version. */
-		loc = newlocale(LC_COLLATE, collcollate, NULL);
+		loc = newlocale(LC_COLLATE_MASK, collcollate, NULL);
 		if (loc)
 		{
 			collversion =

Original file line number	Diff line number	Diff line change
`@@ -1722,7 +1722,7 @@ get_collation_actual_version(char collprovider, const char *collcollate)`
`1722`	`1722`	`locale_t loc;`
`1723`	`1723`
`1724`	`1724`	`/* Look up FreeBSD collation version. */`
`1725`		`- loc = newlocale(LC_COLLATE, collcollate, NULL);`
	`1725`	`+ loc = newlocale(LC_COLLATE_MASK, collcollate, NULL);`
`1726`	`1726`	`if (loc)`
`1727`	`1727`	`{`
`1728`	`1728`	`collversion =`