diff options
| author | Michael Paquier <michael@paquier.xyz> | 2022-01-14 11:25:39 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2022-01-14 11:25:39 +0900 |
| commit | ad5b6f248ab288c3252d8122d12a1eb410d4a0b6 (patch) | |
| tree | dd914d58c9e73ef85bad9978e7c249462374a404 /src/backend/commands/user.c | |
| parent | 4aee39ddb8fa748a6beea2bc1b48882990c226a7 (diff) | |
| download | postgresql-ad5b6f248ab288c3252d8122d12a1eb410d4a0b6.tar.gz postgresql-ad5b6f248ab288c3252d8122d12a1eb410d4a0b6.zip | |
Revert error handling improvements for cryptohashes
This reverts commits ab27df2, af8d530 and 3a0cced, that introduced
pg_cryptohash_error(). In order to make the core code able to pass down
the new error types that this introduced, some of the MD5-related
routines had to be reworked, causing an ABI breakage, but we found that
some external extensions rely on them. Maintaining compatibility
outweights the error report benefits, so just revert the change in v14.
Reported-by: Laurenz Albe
Discussion: https://postgr.es/m/9f0c0a96d28cf14fc87296bbe67061c14eb53ae8.camel@cybertec.at
Diffstat (limited to 'src/backend/commands/user.c')
| -rw-r--r-- | src/backend/commands/user.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/commands/user.c b/src/backend/commands/user.c index a415100c3b1..65bb7339589 100644 --- a/src/backend/commands/user.c +++ b/src/backend/commands/user.c @@ -393,7 +393,7 @@ CreateRole(ParseState *pstate, CreateRoleStmt *stmt) if (password) { char *shadow_pass; - const char *logdetail = NULL; + char *logdetail; /* * Don't allow an empty password. Libpq treats an empty password the @@ -835,7 +835,7 @@ AlterRole(AlterRoleStmt *stmt) if (password) { char *shadow_pass; - const char *logdetail = NULL; + char *logdetail; /* Like in CREATE USER, don't allow an empty password. */ if (password[0] == '\0' || |