[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: SimplePAMApps 0.54preB



Dave Kinchlea wrote:
> Here is a patch for the HOME problem, I also note that PAM_RHOST is not
> being set correctly, I will submit a different patch if/when I fix it.

> -	if (!pw_dir || pw_dir == '\0') {
> +	if (!pw_dir || *pw_dir == '\0') {

Oops!

> -	    fprintf(stderr, ". changing to " DEFAULT_HOME "\n");
> +	    fprintf(stderr, ". changing to %s \n", pw_dir);

No, login is changing directory to DEFAULT_HOME... as pw_dir is not
available.

> -    if (pam_misc_setenv(pamh, "HOME", DEFAULT_HOME, 0) != PAM_SUCCESS) {
> +    if (pam_misc_setenv(pamh, "HOME", pw_dir, 0) != PAM_SUCCESS) {

Oops. This piece of code should have gone up a few lines (inside the brace),
then it would have been correct. Thanks for finding the problem.

Cheers

Andrew
-- 
               Linux-PAM, libpwdb, Orange-Linux and Linux-GSS
                  http://parc.power.net/morgan/index.html
       [ For those that prefer FTP  ---  ftp://ftp.lalug.org/morgan ]



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index] []