diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2017-03-12 16:01:33 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2017-03-12 16:01:33 -0400 |
| commit | 766f7fd613adbceaf1b40803793e10dc487f8596 (patch) | |
| tree | 667802f0b0d171fb9d4bcd81166e46ede6e5fea4 /src/backend/libpq/crypt.c | |
| parent | 835cc1136745e8cf02d3d0231b5b7c7a543df5df (diff) | |
| download | postgresql-766f7fd613adbceaf1b40803793e10dc487f8596.tar.gz postgresql-766f7fd613adbceaf1b40803793e10dc487f8596.zip | |
Add "break"s to make it clearer what will happen in a nested switch.
This could only matter if the guessed_type variable had a value that wasn't
a member of the PasswordType enum; but just in case, let's be sure that
control falls out to reach the elog(ERROR) at the end of the function.
Per gripe from Coverity.
Diffstat (limited to 'src/backend/libpq/crypt.c')
| -rw-r--r-- | src/backend/libpq/crypt.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/backend/libpq/crypt.c b/src/backend/libpq/crypt.c index bd3e936d38c..9f0ae15b00e 100644 --- a/src/backend/libpq/crypt.c +++ b/src/backend/libpq/crypt.c @@ -162,6 +162,7 @@ encrypt_password(PasswordType target_type, const char *role, case PASSWORD_TYPE_MD5: return pstrdup(password); } + break; case PASSWORD_TYPE_SCRAM: switch (guessed_type) @@ -178,6 +179,7 @@ encrypt_password(PasswordType target_type, const char *role, case PASSWORD_TYPE_SCRAM: return pstrdup(password); } + break; } /* |