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_private.h | |
| 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_private.h')
| -rw-r--r-- | libpam/pam_private.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libpam/pam_private.h b/libpam/pam_private.h index 508527cf..576edcfb 100644 --- a/libpam/pam_private.h +++ b/libpam/pam_private.h @@ -287,9 +287,9 @@ void _pam_parse_control(int *control_array, char *tok); * else */ -#define IF_NO_PAMH(X,pamh,ERR) \ +#define IF_NO_PAMH(pamh,ERR) \ if ((pamh) == NULL) { \ - syslog(LOG_ERR, _PAM_SYSTEM_LOG_PREFIX " " X ": NULL pam handle passed"); \ + syslog(LOG_ERR, _PAM_SYSTEM_LOG_PREFIX " %s: NULL pam handle passed", __FUNCTION__); \ return ERR; \ } |