]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Avoid masking EOF (no-password-supplied) conditions in auth.c.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 3 Jan 2024 22:40:38 +0000 (17:40 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 3 Jan 2024 22:40:38 +0000 (17:40 -0500)
commita0d016393be5732f4422268050db2f111bb84674
treeeffb69f64e5ea3d803ce6240ffb5c8f61611cfc4
parent272f857aed43a662f8f93df3d3b6d5798a05df61
Avoid masking EOF (no-password-supplied) conditions in auth.c.

CheckPWChallengeAuth() would return STATUS_ERROR if the user does not
exist or has no password assigned, even if the client disconnected
without responding to the password challenge (as libpq often will,
for example).  We should return STATUS_EOF in that case, and the
lower-level functions do, but this code level got it wrong since the
refactoring done in 7ac955b34.  This breaks the intent of not logging
anything for EOF cases (cf. comments in auth_failed()) and might
also confuse users of ClientAuthentication_hook.

Per report from Liu Lang.  Back-patch to all supported versions.

Discussion: https://postgr.es/m/b725238c-539d-cb09-2bff-b5e6cb2c069c@esgyn.cn
src/backend/libpq/auth.c