[pve-devel] applied: [PATCH access-control] fix #1612: give authenticate_user_ldap the realm on second server

Wolfgang Bumiller w.bumiller at proxmox.com
Thu Jan 11 11:39:56 CET 2018


applied

On Thu, Jan 11, 2018 at 09:49:31AM +0100, Dominik Csapak wrote:
> this was missing and lead to 'use of unitialized value $realm...'
> and a not working second server if a bindpw was defined
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  PVE/Auth/LDAP.pm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/PVE/Auth/LDAP.pm b/PVE/Auth/LDAP.pm
> index 310234a..9f08504 100755
> --- a/PVE/Auth/LDAP.pm
> +++ b/PVE/Auth/LDAP.pm
> @@ -145,7 +145,7 @@ sub authenticate_user {
>      my $err = $@;
>      return 1 if !$err;
>      die $err if !$config->{server2};
> -    &$authenticate_user_ldap($config, $config->{server2}, $username, $password); 
> +    &$authenticate_user_ldap($config, $config->{server2}, $username, $password, $realm);
>  }
>  
>  1;
> -- 
> 2.11.0




More information about the pve-devel mailing list