postgres.git / summary / log / commit / refs

commit    eea28a3cb36951ee439288d8e400621ec301484f
Author:   Michael Paquier <michael@paquier.xyz>
Date:     Thu Aug 01 00:38:29 2019 +0000
First released in: 13.0, 9.4.24, 9.5.19, 9.6.15, 10.10, 11.5, 12.0

    Fix handling of previous password hooks in passwordcheck
    
    When piling up loading of modules using check_password_hook_type,
    loading passwordcheck would remove any trace of a previously-loaded
    hook.  Unloading the module would also cause previous hooks to be
    entirely gone.
    
    Reported-by: Rafael Castro
    Author: Michael Paquier
    Reviewed-by: Daniel Gustafsson
    Discussion: https://postgr.es/m/15932-78f48f9ef166778c@postgresql.org
    Backpatch-through: 9.4


 contrib/passwordcheck/passwordcheck.c | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/contrib/passwordcheck/passwordcheck.c b/contrib/passwordcheck/passwordcheck.c
index 405896d40a9..a46403c701f 100644
--- a/contrib/passwordcheck/passwordcheck.c
+++ b/contrib/passwordcheck/passwordcheck.c
@@ -27,10 +27,14 @@
 
 PG_MODULE_MAGIC;
 
+/* Saved hook value in case of unload */
+static check_password_hook_type prev_check_password_hook = NULL;
+
 /* passwords shorter than this will be rejected */
 #define MIN_PWD_LENGTH 8
 
 extern void _PG_init(void);
+extern void _PG_fini(void);
 
 /*
  * check_password
@@ -63,6 +67,11 @@ check_password(const char *username,
 	bool		pwd_has_letter,
 				pwd_has_nonletter;
 
+	if (prev_check_password_hook)
+		prev_check_password_hook(username, password,
+								 password_type, validuntil_time,
+								 validuntil_null);
+
 	switch (password_type)
 	{
 		case PASSWORD_TYPE_MD5:
@@ -144,5 +153,16 @@ void
 _PG_init(void)
 {
 	/* activate password checks when the module is loaded */
+	prev_check_password_hook = check_password_hook;
 	check_password_hook = check_password;
 }
+
+/*
+ * Module unload function
+ */
+void
+_PG_fini(void)
+{
+	/* uninstall hook */
+	check_password_hook = prev_check_password_hook;
+}


[parent: 4e10b6f82355]