diff options
| author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2017-04-13 17:44:15 +0300 |
|---|---|---|
| committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2017-04-13 17:44:15 +0300 |
| commit | 00707fa58275e370dc445fa7e1130085aa04f37b (patch) | |
| tree | ac34d654e3d719bc3eb2ff9f5d13b9b1da2b902f /src/backend/regex/regexport.c | |
| parent | 3d5facfd9ab66c819ed583b2614b0560405a6aa2 (diff) | |
| download | postgresql-00707fa58275e370dc445fa7e1130085aa04f37b.tar.gz postgresql-00707fa58275e370dc445fa7e1130085aa04f37b.zip | |
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/6490b975-5ee1-6280-ac1d-af975b19fb9a@iki.fi
Diffstat (limited to 'src/backend/regex/regexport.c')
0 files changed, 0 insertions, 0 deletions