SVN commit 507710 by ossi: thanks for the reminder ... as expected, (at least linux-)pam ignores the pamh argument to pam_strerror, so it can be safely set to null. M +1 -1 client.c --- branches/KDE/3.5/kdebase/kdm/backend/client.c #507709:507710 @@ -299,7 +299,7 @@ pamh = 0; pam_bail2: ReInitErrorLog(); - LogError( "PAM error: %s\n", pam_strerror( 0, pretc ) ); /* XXX */ + LogError( "PAM error: %s\n", pam_strerror( 0, pretc ) ); V_RET_FAIL( 0 ); } if ((td->displayType & d_location) == dForeign) {