mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix comment in fe-auth-scram.c

Browse Source 
The frontend-side routine in charge of building a SCRAM verifier
mentioned that the restrictions applying to SASLprep on the password
with the encoding are described at the top of fe-auth-scram.c, but this
information is in auth-scram.c.

This is wrong since 8f8b9be, so backpatch all the way down as this is an
important documentation bit.

Spotted while reviewing a different patch.

Backpatch-through: 11
This commit is contained in:
Michael Paquier 2022-11-30 08:38:33 +09:00
parent 6e8ad11521
commit 4c7bb4dccb
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/interfaces/libpq/fe-auth-scram.c
View File

@ -834,7 +834,8 @@ pg_fe_scram_build_secret(const char *password)
/* /*
* Normalize the password with SASLprep. If that doesn't work, because * Normalize the password with SASLprep. If that doesn't work, because
* the password isn't valid UTF-8 or contains prohibited characters, just * the password isn't valid UTF-8 or contains prohibited characters, just
* proceed with the original password. (See comments at top of file.) * proceed with the original password. (See comments at the top of
* auth-scram.c.)
*/ */
rc = pg_saslprep(password, &prep_password); rc = pg_saslprep(password, &prep_password);
if (rc == SASLPREP_OOM) if (rc == SASLPREP_OOM)