diff options
| author | Michael Paquier <michael@paquier.xyz> | 2024-12-10 12:54:14 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2024-12-10 12:54:14 +0900 |
| commit | 416d1bd66c2859b2a40d265b77299c1f1d5dfc5b (patch) | |
| tree | 5687b9777efcca88dec24c1e7e6250c6e4c24e8c | |
| parent | a12b0f3a9dc08ed18711fdcd38763cb3f6bc9f69 (diff) | |
Fix outdated comment of scram_build_secret()
This routine documented that "iterations" would use a default value if
set to 0 by the caller. However, the iteration should always be set by
the caller to a value strictly more than 0, as documented by an
assertion.
Oversight in b577743000cd, that has made the iteration count of SCRAM
configurable.
Author: Matheus Alcantara
Discussion: https://postgr.es/m/ac858943-4743-44cd-b4ad-08a0c10cbbc8@gmail.com
Backpatch-through: 16
| -rw-r--r-- | src/common/scram-common.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/common/scram-common.c b/src/common/scram-common.c index b611bb8fe71..51cc12e8c3e 100644 --- a/src/common/scram-common.c +++ b/src/common/scram-common.c @@ -200,8 +200,7 @@ scram_ServerKey(const uint8 *salted_password, * * The password should already have been processed with SASLprep, if necessary! * - * If iterations is 0, default number of iterations is used. The result is - * palloc'd or malloc'd, so caller is responsible for freeing it. + * The result is palloc'd or malloc'd, so caller is responsible for freeing it. * * On error, returns NULL and sets *errstr to point to a message about the * error details. |