Bug 247192 - adminutil: admldapBuildInfoCbk() is not properly setting the LDAP handle
Summary: adminutil: admldapBuildInfoCbk() is not properly setting the LDAP handle
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: 389
Classification: Retired
Component: Admin
Version: 1.1.0
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Nathan Kinder
QA Contact: Orla Hegarty
URL:
Whiteboard:
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
Clone Of:
Environment:
Last Closed: 2007-07-25 05:22:08 UTC
Embargoed:


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
Ok.

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
done
Checking in lib/libadminutil/admutil.c;
/cvs/dirsec/adminutil/lib/libadminutil/admutil.c,v  <--  admutil.c
new revision: 1.9; previous revision: 1.8
done
Checking in lib/libadminutil/distadm.c;
/cvs/dirsec/adminutil/lib/libadminutil/distadm.c,v  <--  distadm.c
new revision: 1.4; previous revision: 1.3
done

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.