oysteikt/heimdal
0
0
Fork 0
Code Issues 5 Pull Requests Releases Activity

Address code review comments (use _krb5_homedir_access())

Browse Source
This commit is contained in:
Nicolas Williams
2011-12-10 14:02:34 -06:00
parent 8e04b6dce2
commit 3109770484
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
lib/krb5/kuserok.c
View File

@@ -552,6 +552,10 @@ kuserok_user_k5login_plug_f(void *plug_ctx, krb5_context context,
profile_dir = k5login_dir; profile_dir = k5login_dir;
if (profile_dir == NULL) { if (profile_dir == NULL) {
/* Don't deadlock with gssd or anything of the sort */
if (!_krb5_homedir_access(context))
return KRB5_PLUGIN_NO_HANDLE;
#ifdef POSIX_GETPWNAM_R #ifdef POSIX_GETPWNAM_R
if (getpwnam_r(luser, &pw, pwbuf, sizeof(pwbuf), &pwd) != 0) { if (getpwnam_r(luser, &pw, pwbuf, sizeof(pwbuf), &pwd) != 0) {
krb5_set_error_message(context, errno, "User unknown (getpwnam_r())"); krb5_set_error_message(context, errno, "User unknown (getpwnam_r())");
@@ -560,6 +564,7 @@ kuserok_user_k5login_plug_f(void *plug_ctx, krb5_context context,
#else #else
pwd = getpwnam (luser); pwd = getpwnam (luser);
#endif #endif
if (pwd == NULL) { if (pwd == NULL) {
krb5_set_error_message(context, errno, "User unknown (getpwnam())"); krb5_set_error_message(context, errno, "User unknown (getpwnam())");
return KRB5_PLUGIN_NO_HANDLE; return KRB5_PLUGIN_NO_HANDLE;