diff options
| author | Noah Misch <noah@leadboat.com> | 2016-04-06 23:41:43 -0400 |
|---|---|---|
| committer | Noah Misch <noah@leadboat.com> | 2016-04-06 23:41:43 -0400 |
| commit | f2b1b3079ce9d2965f6e450585f24d18cdf5647b (patch) | |
| tree | 7db788c4736fad71d68c79565f578679c50e5be8 /src/backend/libpq/auth.c | |
| parent | 33d3fc5e2aac32fcf356c09cee4bfded6613a1f3 (diff) | |
Standardize GetTokenInformation() error reporting.
Commit c22650cd6450854e1a75064b698d7dcbb4a8821a sparked a discussion
about diverse interpretations of "token user" in error messages. Expel
old and new specimens of that phrase by making all GetTokenInformation()
callers report errors the way GetTokenUser() has been reporting them.
These error conditions almost can't happen, so users are unlikely to
observe this change.
Reviewed by Tom Lane and Stephen Frost.
Diffstat (limited to 'src/backend/libpq/auth.c')
| -rw-r--r-- | src/backend/libpq/auth.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/backend/libpq/auth.c b/src/backend/libpq/auth.c index 2751183c3e6..f21056e2445 100644 --- a/src/backend/libpq/auth.c +++ b/src/backend/libpq/auth.c @@ -1242,8 +1242,8 @@ pg_SSPI_recvauth(Port *port) if (!GetTokenInformation(token, TokenUser, NULL, 0, &retlen) && GetLastError() != 122) ereport(ERROR, - (errmsg_internal("could not get token user size: error code %lu", - GetLastError()))); + (errmsg_internal("could not get token information buffer size: error code %lu", + GetLastError()))); tokenuser = malloc(retlen); if (tokenuser == NULL) @@ -1252,8 +1252,8 @@ pg_SSPI_recvauth(Port *port) if (!GetTokenInformation(token, TokenUser, tokenuser, retlen, &retlen)) ereport(ERROR, - (errmsg_internal("could not get token user: error code %lu", - GetLastError()))); + (errmsg_internal("could not get token information: error code %lu", + GetLastError()))); CloseHandle(token); |