[Fedora-directory-devel] Please Review: (247192) adminutil: admldapBuildInfoCbk() is not properly setting the LDAP handle

[Nathan Kinder]

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=247192
Resolves: bug 247192
Bug Description: The admldapBuildInfoCbk() function is supposed to
  create an LDAP handle.  This handle is not getting created since
  it is attempting to bind as the sie DN, which will always fail.
  This causes a problem for other functions which require a valid
  LDAP handle.  One of the functions that this causes a problem for
  in particular is the admldapGetUserDirectoryInfo() function.

  The admldapGetUserDirectoryInfo function is used to get info about
  the users and groups directory.  This is used at Admin Server startup
  time as well as by the ugdsconfig CGI that is called by the Admin
  Server Console.  When used by the Console, we need to pass the userdn
  used to authenticate to the Admin Server through to the Directory
  Server.  This approach does not work for the startup of the Admin
  Server since there is no userdn set at that time.  This is ok since
  mod_admserv will use cached info if there is no LDAP handle present.
Reviewed by: ???
Files: see diff
Branch: HEAD
Fix Description: The afix addresses this issue by overriding the sie DN
  with the userdn when when setting up the LDAP handle.  We only override
  the sie DN if the userdn is initialized.
Platforms tested: FC6
Flag Day: no
Doc impact: no
https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=158620&action=...