Re: get_dn

[Peter M. <pet...@ma...>]

Hi

On Tuesday 02 July 2002 20:22, you wrote:
> #check for search error
>      if ($error =3D=3D $results->code()){
> =09 =09  die "Critical LDAP search failed:$error,$results->filter(0)";
> =09=09  }
here, you compare the result code with the variable error,
and if they match, you bail out. Is this intended ? I doubt it.
May be =3D instead of =3D=3D would serve better

> =09 if ($results->count()) {
>               die"User entry not found for filter:&results->filter";
>      }
The condition here is fulfilled if you have atleast one entry.
It should be reversed !

> #get DN
> =09 $ent =3D $results -> shift_entry();
> =09 $dn =3D $ent -> dn();     #line 39

dn() has to fail here, because you only arrive here in case of errors.


Yours
Peter
--=20
Peter Marschall     |   eMail: pet...@ma...
Scheffelstra=DFe 15   |          pet...@is...
97072 W=FCrzburg      |   Tel:   0931/14721
PGP:  D7 FF 20 FE E6 6B 31 74  D1 10 88 E0 3C FE 28 35