diff options
| author | Tobias Stoeckmann <tobias@stoeckmann.org> | 2023-11-11 12:02:04 +0100 |
|---|---|---|
| committer | Dmitry V. Levin <ldv@strace.io> | 2023-11-12 19:20:21 +0000 |
| commit | e3e3b06fbdb917eaff24e1a6860654b99b99faa0 (patch) | |
| tree | 44fd0f521a029e3d794137f749f834c570e333f8 /libpam/pam_handlers.c | |
| parent | 3f6189918f088e353119944e24fe1f725ee502a7 (diff) | |
| download | pam-e3e3b06fbdb917eaff24e1a6860654b99b99faa0.tar.gz pam-e3e3b06fbdb917eaff24e1a6860654b99b99faa0.tar.bz2 pam-e3e3b06fbdb917eaff24e1a6860654b99b99faa0.zip | |
libpam: simplify IF_NO_PAMH
The first argument of IF_NO_PAMH is supposed to be the name of the
function which was called with pamh being NULL.
With __FUNCTION__ the name can be inserted automatically by the
compiler which is also already done with D macro.
Fixes a bug in which _pam_drop_env erroneously logs with the
function name _pam_make_env.
Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
Diffstat (limited to 'libpam/pam_handlers.c')
| -rw-r--r-- | libpam/pam_handlers.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/libpam/pam_handlers.c b/libpam/pam_handlers.c index 6c91c193..fdc2b8a3 100644 --- a/libpam/pam_handlers.c +++ b/libpam/pam_handlers.c @@ -392,7 +392,7 @@ int _pam_init_handlers(pam_handle_t *pamh) int retval; D(("called.")); - IF_NO_PAMH("_pam_init_handlers",pamh,PAM_SYSTEM_ERR); + IF_NO_PAMH(pamh,PAM_SYSTEM_ERR); /* Return immediately if everything is already loaded */ if (pamh->handlers.handlers_loaded) { @@ -796,7 +796,7 @@ int _pam_add_handler(pam_handle_t *pamh int mod_type = PAM_MT_FAULTY_MOD; D(("called.")); - IF_NO_PAMH("_pam_add_handler",pamh,PAM_SYSTEM_ERR); + IF_NO_PAMH(pamh,PAM_SYSTEM_ERR); D(("adding type %d, handler_type %d, module `%s'", type, handler_type, mod_path)); @@ -954,7 +954,7 @@ int _pam_free_handlers(pam_handle_t *pamh) struct loaded_module *mod; D(("called.")); - IF_NO_PAMH("_pam_free_handlers",pamh,PAM_SYSTEM_ERR); + IF_NO_PAMH(pamh,PAM_SYSTEM_ERR); mod = pamh->handlers.module; |