Bug 247192 - adminutil: admldapBuildInfoCbk() is not properly setting the LDAP handle
Summary: adminutil: admldapBuildInfoCbk() is not properly setting the LDAP handle
Alias: None
Product: 389
Classification: Retired
Component: Admin   
(Show other bugs)
Version: 1.1.0
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nathan Kinder
QA Contact: Orla Hegarty
Depends On:
Blocks: 240316 FDS1.1.0
TreeView+ depends on / blocked
Reported: 2007-07-05 21:06 UTC by Nathan Kinder
Modified: 2008-01-03 18:02 UTC (History)
2 users (show)

Fixed In Version: 1.1.3-1.fc7
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-07-25 05:22:08 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
CVS Diffs (2.90 KB, patch)
2007-07-05 21:06 UTC, Nathan Kinder
no flags Details | Diff

Description Nathan Kinder 2007-07-05 21:06:04 UTC
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.

The attached diffs address 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.

Comment 1 Nathan Kinder 2007-07-05 21:06:04 UTC
Created attachment 158620 [details]
CVS Diffs

Comment 2 Rich Megginson 2007-07-05 21:09:09 UTC

Comment 3 Nathan Kinder 2007-07-05 21:13:38 UTC
Checked into adminutil (HEAD).  Thanks for the review Rich!

Checking in include/libadminutil/distadm.h;
/cvs/dirsec/adminutil/include/libadminutil/distadm.h,v  <--  distadm.h
new revision: 1.5; previous revision: 1.4
Checking in lib/libadminutil/admutil.c;
/cvs/dirsec/adminutil/lib/libadminutil/admutil.c,v  <--  admutil.c
new revision: 1.9; previous revision: 1.8
Checking in lib/libadminutil/distadm.c;
/cvs/dirsec/adminutil/lib/libadminutil/distadm.c,v  <--  distadm.c
new revision: 1.4; previous revision: 1.3

Comment 4 Fedora Update System 2007-07-25 05:21:49 UTC
adminutil-1.1.3-1.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.

Note You need to log in before you can comment on or make changes to this bug.