diff options
author | Jakub Hrozek <jhrozek@redhat.com> | 2012-04-03 18:42:43 -0400 |
---|---|---|
committer | Stephen Gallagher <sgallagh@redhat.com> | 2012-04-18 10:23:55 -0400 |
commit | bd09ead65cded3207cf228c44a31bbc87c2979bd (patch) | |
tree | 91d7abf452011afdbef9c42a3eb7d59fa8768a83 /src/providers/dp_pam_data_util.c | |
parent | 7329723a5d8ce46f146e78dabfad5c94b3eed505 (diff) | |
download | sssd-bd09ead65cded3207cf228c44a31bbc87c2979bd.tar.gz sssd-bd09ead65cded3207cf228c44a31bbc87c2979bd.tar.xz sssd-bd09ead65cded3207cf228c44a31bbc87c2979bd.zip |
Prevent printing NULL from DEBUG messages
Diffstat (limited to 'src/providers/dp_pam_data_util.c')
-rw-r--r-- | src/providers/dp_pam_data_util.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/src/providers/dp_pam_data_util.c b/src/providers/dp_pam_data_util.c index 7ac50c4e1..889c47f00 100644 --- a/src/providers/dp_pam_data_util.c +++ b/src/providers/dp_pam_data_util.c @@ -45,6 +45,8 @@ } \ } while(0) +#define PAM_SAFE_ITEM(item) item ? item : "not set" + static const char *pamcmd2str(int cmd) { switch (cmd) { case SSS_PAM_AUTHENTICATE: @@ -138,12 +140,12 @@ failed: void pam_print_data(int l, struct pam_data *pd) { DEBUG(l, ("command: %s\n", pamcmd2str(pd->cmd))); - DEBUG(l, ("domain: %s\n", pd->domain)); - DEBUG(l, ("user: %s\n", pd->user)); - DEBUG(l, ("service: %s\n", pd->service)); - DEBUG(l, ("tty: %s\n", pd->tty)); - DEBUG(l, ("ruser: %s\n", pd->ruser)); - DEBUG(l, ("rhost: %s\n", pd->rhost)); + DEBUG(l, ("domain: %s\n", PAM_SAFE_ITEM(pd->domain))); + DEBUG(l, ("user: %s\n", PAM_SAFE_ITEM(pd->user))); + DEBUG(l, ("service: %s\n", PAM_SAFE_ITEM(pd->service))); + DEBUG(l, ("tty: %s\n", PAM_SAFE_ITEM(pd->tty))); + DEBUG(l, ("ruser: %s\n", PAM_SAFE_ITEM(pd->ruser))); + DEBUG(l, ("rhost: %s\n", PAM_SAFE_ITEM(pd->rhost))); DEBUG(l, ("authtok type: %d\n", pd->authtok_type)); DEBUG(l, ("authtok size: %d\n", pd->authtok_size)); DEBUG(l, ("newauthtok type: %d\n", pd->newauthtok_type)); |