From 15d7d6d97b4e5d653c56f2bc5fe1ccdd5f777ee2 Mon Sep 17 00:00:00 2001 From: Mike Gabriel Date: Sun, 6 May 2018 13:15:57 +0200 Subject: src/pam-x2go.c: Update some comments. --- src/pam-x2go.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/pam-x2go.c b/src/pam-x2go.c index 74ca68d..f9ae42b 100644 --- a/src/pam-x2go.c +++ b/src/pam-x2go.c @@ -61,8 +61,8 @@ get_item (pam_handle_t * pamh, int type) /* Check to see if we just have the value. If we do, great let's dup it some we're consistently allocating memory */ if ((type == PAM_USER) || (type == PAM_AUTHTOK)) { - /* If it's not a session type we can use the PAM functions because the PAM - functions don't support session type */ + /* If it's not an X2Go parameter type we can use the PAM functions because the PAM + functions don't support X2Go parameters */ char * value = NULL; if (pam_get_item(pamh, type, (const void **)&value) == PAM_SUCCESS && value != NULL) { return value; @@ -158,11 +158,12 @@ get_item (pam_handle_t * pamh, int type) char * retval = NULL; if (promptval != NULL) { /* Can't believe it really would be at this point, but let's be sure */ if ((type == PAM_USER) || (type == PAM_AUTHTOK)) { - /* We can only use the PAM functions if it's not the session type */ + /* We can only use the PAM functions for types supported by PAM */ pam_set_item(pamh, type, (const void *)promptval); /* We're returning the value saved by PAM so we can clear promptval */ pam_get_item(pamh, type, (const void **)&retval); } + /* Here we deal with all X2Go specific parameter types */ if (type == PAM_TYPE_X2GO_USER) { /* The remote user can be saved globally */ if (global_x2go_user != NULL) { -- cgit v1.2.3