summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHeikki Linnakangas2017-04-06 14:41:48 +0000
committerHeikki Linnakangas2017-04-06 14:41:48 +0000
commit07044efe00762bdd04c4d392adb8f6425b13369b (patch)
treed85c1fc6cac5d3b28c976668b29554c01bdbd343
parentcd0cebaf7d1ab04427d4045edf7121a8f3753d8b (diff)
Remove bogus SCRAM_ITERATION_LEN constant.
It was not used for what the comment claimed, at all. It was actually used as the 'base' argument to strtol(), when reading the iteration count. We don't need a constant for base-10, so remove it.
-rw-r--r--src/backend/libpq/auth-scram.c2
-rw-r--r--src/include/common/scram-common.h3
-rw-r--r--src/interfaces/libpq/fe-auth-scram.c2
3 files changed, 2 insertions, 5 deletions
diff --git a/src/backend/libpq/auth-scram.c b/src/backend/libpq/auth-scram.c
index bcc8d03ef5..14ddc8bd54 100644
--- a/src/backend/libpq/auth-scram.c
+++ b/src/backend/libpq/auth-scram.c
@@ -476,7 +476,7 @@ parse_scram_verifier(const char *verifier, char **salt, int *iterations,
if ((p = strtok(NULL, ":")) == NULL)
goto invalid_verifier;
errno = 0;
- *iterations = strtol(p, &p, SCRAM_ITERATION_LEN);
+ *iterations = strtol(p, &p, 10);
if (*p || errno != 0)
goto invalid_verifier;
diff --git a/src/include/common/scram-common.h b/src/include/common/scram-common.h
index 7c98cc74d6..6740069eee 100644
--- a/src/include/common/scram-common.h
+++ b/src/include/common/scram-common.h
@@ -31,9 +31,6 @@
/* length of salt when generating new verifiers */
#define SCRAM_SALT_LEN 10
-/* number of bytes used when sending iteration number during exchange */
-#define SCRAM_ITERATION_LEN 10
-
/* default number of iterations when generating verifier */
#define SCRAM_ITERATIONS_DEFAULT 4096
diff --git a/src/interfaces/libpq/fe-auth-scram.c b/src/interfaces/libpq/fe-auth-scram.c
index a7bb30a141..818ade4993 100644
--- a/src/interfaces/libpq/fe-auth-scram.c
+++ b/src/interfaces/libpq/fe-auth-scram.c
@@ -444,7 +444,7 @@ read_server_first_message(fe_scram_state *state, char *input,
/* read_attr_value() has generated an error string */
return false;
}
- state->iterations = strtol(iterations_str, &endptr, SCRAM_ITERATION_LEN);
+ state->iterations = strtol(iterations_str, &endptr, 10);
if (*endptr != '\0' || state->iterations < 1)
{
printfPQExpBuffer(errormessage,