libpq code should use libpq_gettext(), not _()

petere · petere · commit 28ec31678767 · 2022-08-25T20:46:58.000+02:00
Fix some wrong use and install a safeguard against future mistakes.
diff --git a/src/interfaces/libpq/fe-auth-scram.c b/src/interfaces/libpq/fe-auth-scram.c
@@ -922,7 +922,7 @@ pg_fe_scram_build_secret(const char *password, const char **errstr)
 	rc = pg_saslprep(password, &prep_password);
 	if (rc == SASLPREP_OOM)
 	{
-		*errstr = _("out of memory");
+		*errstr = libpq_gettext("out of memory");
 		return NULL;
 	}
 	if (rc == SASLPREP_SUCCESS)
@@ -931,7 +931,7 @@ pg_fe_scram_build_secret(const char *password, const char **errstr)
 	/* Generate a random salt */
 	if (!pg_strong_random(saltbuf, SCRAM_DEFAULT_SALT_LEN))
 	{
-		*errstr = _("could not generate random salt");
+		*errstr = libpq_gettext("could not generate random salt");
 		free(prep_password);
 		return NULL;
 	}
diff --git a/src/interfaces/libpq/libpq-int.h b/src/interfaces/libpq/libpq-int.h
@@ -881,6 +881,11 @@ extern char *libpq_ngettext(const char *msgid, const char *msgid_plural, unsigne
 #define libpq_gettext(x) (x)
 #define libpq_ngettext(s, p, n) ((n) == 1 ? (s) : (p))
 #endif
+/*
+ * libpq code should use the above, not _(), since that would use the
+ * surrounding programs's message catalog.
+ */
+#undef _
 
 /*
  * These macros are needed to let error-handling code be portable between

Original file line number	Diff line number	Diff line change
`@@ -922,7 +922,7 @@ pg_fe_scram_build_secret(const char password, const char *errstr)`
`922`	`922`	`rc = pg_saslprep(password, &prep_password);`
`923`	`923`	`if (rc == SASLPREP_OOM)`
`924`	`924`	`{`
`925`		`- *errstr = _("out of memory");`
	`925`	`+ *errstr = libpq_gettext("out of memory");`
`926`	`926`	`return NULL;`
`927`	`927`	`}`
`928`	`928`	`if (rc == SASLPREP_SUCCESS)`
`@@ -931,7 +931,7 @@ pg_fe_scram_build_secret(const char password, const char *errstr)`
`931`	`931`	`/* Generate a random salt */`
`932`	`932`	`if (!pg_strong_random(saltbuf, SCRAM_DEFAULT_SALT_LEN))`
`933`	`933`	`{`
`934`		`- *errstr = _("could not generate random salt");`
	`934`	`+ *errstr = libpq_gettext("could not generate random salt");`
`935`	`935`	`free(prep_password);`
`936`	`936`	`return NULL;`
`937`	`937`	`}`