Minor cleanup of backend SCRAM code. Free each SASL message after sending it. It's not a lot of wasted memory, and it's short-lived, but the authentication code in general tries to pfree() stuff, so let's follow the example.
Adding the pfree() revealed a little bug in build_server_first_message(). It attempts to keeps a copy of the sent message, but it was missing a pstrdup(), so the pointer started to dangle, after adding the pfree() into CheckSCRAMAuth(). Reword comments and debug messages slightly, while we're at it. Reviewed by Michael Paquier. Discussion: https://www.postgresql.org/message-id/[email protected] Branch ------ master Details ------- http://git.postgresql.org/pg/commitdiff/00707fa58275e370dc445fa7e1130085aa04f37b Modified Files -------------- src/backend/libpq/auth-scram.c | 10 +++++----- src/backend/libpq/auth.c | 12 +++++++----- 2 files changed, 12 insertions(+), 10 deletions(-) -- Sent via pgsql-committers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-committers
