NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commitb2a3d70

committed

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-loadedhook. Unloading the module would also cause previous hooks to beentirely gone.Reported-by: Rafael CastroAuthor: Michael PaquierReviewed-by: Daniel GustafssonDiscussion:https://postgr.es/m/15932-78f48f9ef166778c@postgresql.orgBackpatch-through: 9.4

1 parent07b3908 commitb2a3d70Copy full SHA for b2a3d70

File tree

1 file changed

+20

-0

lines changed

contrib/passwordcheck
- passwordcheck.c

1 file changed

+20

-0

lines changed

`‎contrib/passwordcheck/passwordcheck.c`

Lines changed: 20 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -26,10 +26,14 @@`
`26`	`26`
`27`	`27`	`PG_MODULE_MAGIC;`
`28`	`28`
	`29`	`+/* Saved hook value in case of unload */`
	`30`	`+staticcheck_password_hook_typeprev_check_password_hook=NULL;`
	`31`	`+`
`29`	`32`	`/* passwords shorter than this will be rejected */`
`30`	`33`	`#defineMIN_PWD_LENGTH 8`
`31`	`34`
`32`	`35`	`externvoid_PG_init(void);`
	`36`	`+externvoid_PG_fini(void);`
`33`	`37`
`34`	`38`	`/*`
`35`	`39`	`* check_password`
`@@ -55,6 +59,11 @@ check_password(const char *username,`
`55`	`59`	`Datumvaliduntil_time,`
`56`	`60`	`boolvaliduntil_null)`
`57`	`61`	`{`
	`62`	`+if (prev_check_password_hook)`
	`63`	`+prev_check_password_hook(username,shadow_pass,`
	`64`	`+password_type,validuntil_time,`
	`65`	`+validuntil_null);`
	`66`	`+`
`58`	`67`	`if (password_type!=PASSWORD_TYPE_PLAINTEXT)`
`59`	`68`	`{`
`60`	`69`	`/*`
`@@ -133,5 +142,16 @@ void`
`133`	`142`	`_PG_init(void)`
`134`	`143`	`{`
`135`	`144`	`/* activate password checks when the module is loaded */`
	`145`	`+prev_check_password_hook=check_password_hook;`
`136`	`146`	`check_password_hook=check_password;`
`137`	`147`	`}`
	`148`	`+`
	`149`	`+/*`
	`150`	`+ * Module unload function`
	`151`	`+ */`
	`152`	`+void`
	`153`	`+_PG_fini(void)`
	`154`	`+{`
	`155`	`+/* uninstall hook */`
	`156`	`+check_password_hook=prev_check_password_hook;`
	`157`	`+}`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitb2a3d70

File tree

1 file changed

1 file changed

`‎contrib/passwordcheck/passwordcheck.c`

0 commit comments