From 6b1a837f69d00d265bee4b57ba2d320f1463f131 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Thu, 7 Jan 2016 11:26:54 -0500 Subject: [PATCH] Remove vestigial CHECK_FOR_INTERRUPTS call. Commit e710b65c inserted code in md5_crypt_verify to disable and later re-enable interrupts, with a CHECK_FOR_INTERRUPTS call as part of the second step, to process any interrupts that had been held off. Commit 6647248e removed the interrupt disable/re-enable code, but left behind the CHECK_FOR_INTERRUPTS, even though this is now an entirely random, pointless place for one. md5_crypt_verify doesn't run long enough to need such a check, and if it did, this would still be the wrong place to put one. --- src/backend/libpq/crypt.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/backend/libpq/crypt.c b/src/backend/libpq/crypt.c index f3c59e5303..d79f5a2496 100644 --- a/src/backend/libpq/crypt.c +++ b/src/backend/libpq/crypt.c @@ -81,8 +81,6 @@ md5_crypt_verify(const Port *port, const char *role, char *client_pass, return STATUS_ERROR; /* empty password */ } - CHECK_FOR_INTERRUPTS(); - /* * Compare with the encrypted or plain password depending on the * authentication method being used for this connection. (We do not -- 2.40.0