diff options
| author | Björn Jacke <bj@sernet.de> | 2009-05-26 15:40:21 +0200 |
|---|---|---|
| committer | Björn Jacke <bj@sernet.de> | 2009-05-26 16:23:11 +0200 |
| commit | caeae66c5bb3a4a600b5c90be97d98a323d1dc2f (patch) | |
| tree | fdff8e7b6a21b09ddb03865eb948cea3e5eafeb0 /source3/pam_smbpass/support.h | |
| parent | 6b7bd3757005286c528d3bb0c6790e0d8651eaaf (diff) | |
| download | samba-caeae66c5bb3a4a600b5c90be97d98a323d1dc2f.tar.gz samba-caeae66c5bb3a4a600b5c90be97d98a323d1dc2f.tar.bz2 samba-caeae66c5bb3a4a600b5c90be97d98a323d1dc2f.zip | |
s3:pam_smbpass: don't call openlog() or closelog() from pam_smbpass
Patch from Steve Langasek with tiny fixes by me to make it apply to master.
Also see Debian bug #434372 and bugzilla #4831.
Calling openlog() or closelog() inside a pam module is not good as these
functions are not stackable and no program won't re-do openlog() just because a
pam module might have called closelog().
Diffstat (limited to 'source3/pam_smbpass/support.h')
| -rw-r--r-- | source3/pam_smbpass/support.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/source3/pam_smbpass/support.h b/source3/pam_smbpass/support.h index 87f1690a60..7ee77042d7 100644 --- a/source3/pam_smbpass/support.h +++ b/source3/pam_smbpass/support.h @@ -1,8 +1,8 @@ /* syslogging function for errors and other information */ -extern void _log_err(int, const char *, ...); +extern void _log_err(pam_handle_t *, int, const char *, ...); /* set the control flags for the UNIX module. */ -extern int set_ctrl(int, int, const char **); +extern int set_ctrl(pam_handle_t *, int, int, const char **); /* generic function for freeing pam data segments */ extern void _cleanup(pam_handle_t *, void *, int); @@ -12,7 +12,7 @@ extern void _cleanup(pam_handle_t *, void *, int); * evidence of old token around for later stack analysis. */ -extern char *smbpXstrDup(const char *); +extern char *smbpXstrDup(pam_handle_t *,const char *); /* ************************************************************** * * Useful non-trivial functions * |