Enable idmap_ldap support in winbind (missing ldap.so)

Bug #397203 reported by Aaron J. Zirbes
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
samba (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Binary package hint: winbind

Winbind has lost LDAP support because ldap.so is missing from /usr/lib/samba/idmap/ldap.so

I discovered this when trying to use winbind to map to LDAP: /var/log/samba/log.winbind-idmap reported the following:

[2009/07/08 17:10:26, 5] lib/module.c:smb_probe_module(111)
  Probing module 'ldap'
[2009/07/08 17:10:26, 5] lib/module.c:smb_probe_module(130)
  Probing module 'ldap': Trying to load from /usr/lib/samba/idmap/ldap.so
[2009/07/08 17:10:26, 3] lib/module.c:do_smb_load_module(48)
  Error loading module '/usr/lib/samba/idmap/ldap.so': /usr/lib/samba/idmap/ldap.so: cannot open shared object file: No such file or directory
[2009/07/08 17:10:26, 1] winbindd/idmap.c:idmap_alloc_init(578)
  could not find idmap alloc module ldap:ldap://ldap-north.healthstudies.umn.edu/

Revision history for this message
Chuck Short (zulcss) wrote :

Hi,

Thanks for the bug report, which version of samba are you using?

Thanks
chuck

Changed in samba (Ubuntu):
status: New → Incomplete
Revision history for this message
Aaron J. Zirbes (ajz) wrote :

I was unable to do and svn commit to the samba package on alioth yet, but here's the fix:

svn diff debian/rules
Index: rules
===================================================================
--- rules (revision 2940)
+++ rules (working copy)
@@ -48,7 +48,7 @@
   --with-pam_smbpass \
   --with-libsmbclient \
   --with-winbind \
- --with-shared-modules=idmap_rid,idmap_ad,idmap_adex,idmap_hash,idmap_tdb2 \
+ --with-shared-modules=idmap_ldap,idmap_rid,idmap_ad,idmap_adex,idmap_hash,idmap_tdb2 \
   --with-automount \
   --with-ldap \
   --with-ads \

svn diff debian/winbind.files
Index: winbind.files
===================================================================
--- winbind.files (revision 2940)
+++ winbind.files (working copy)
@@ -6,6 +6,7 @@
 usr/lib/samba/idmap/adex.so
 usr/lib/samba/idmap/hash.so
 usr/lib/samba/idmap/tdb2.so
+usr/lib/samba/idmap/ldap.so
 usr/share/man/man1/wbinfo.1
 usr/share/man/man1/ntlm_auth.1
 usr/share/man/man8/pam_winbind.8

Aaron J. Zirbes (ajz)
Changed in samba (Ubuntu):
status: Incomplete → In Progress
Revision history for this message
Aaron J. Zirbes (ajz) wrote :

My version of samba is 2:3.3.2-1ubuntu3 on Ubuntu Server 9.04

I did an SVN checkout of the debian samba package, and the problem is there as well.

I'm not sure where to post my fix to get this pushed upstream to the debian package, but the diffs above fix it.

Should I report this bug via debian? I could install a debian system, and run reportbug if needed.

Changed in samba (Ubuntu):
status: In Progress → Confirmed
Revision history for this message
Thierry Carrez (ttx) wrote :

By "Winbind has lost LDAP support" you mean it used to work with previous Ubuntu versions ? Trying to figure if it's a regression or a feature request.

Revision history for this message
Aaron J. Zirbes (ajz) wrote : Re: [Bug 397203] Re: ldap.so missing from package

I guess I've only gotten it running on Redhat, OpenSUSE, and FreeBSD hosts. It may have never
worked in debian/ubuntu builds.

--
Aaron

Thierry Carrez wrote:
> By "Winbind has lost LDAP support" you mean it used to work with
> previous Ubuntu versions ? Trying to figure if it's a regression or a
> feature request.
>

Thierry Carrez (ttx)
summary: - ldap.so missing from package
+ Enable idmap_ldap support in winbind (missing ldap.so)
Changed in samba (Ubuntu):
importance: Undecided → Wishlist
Revision history for this message
MarianoAbsatz (el-baby) wrote :

I guess in previous versions what happened was that winbindd wasn't split in different modules, so everything (including ldap) worked from the same binaries.

In newer versions you need the modules for each idmap backend, and ubuntu doesn't have ldap nor tdb :-(

Revision history for this message
MarianoAbsatz (el-baby) wrote :

I just downloaded the current source package for jaunty and it not only lacks idmap_ldap, but also idmap_tdb2 support.

FWIW, I made similar changes to the patch posted by Aaron on July and left a step by step log (copy&paste instructions) of how to build and re-install winbind in my wiki: http://wiki.clueless.com.ar/SambaLdap/RecompilarSamba

Revision history for this message
Chuck Short (zulcss) wrote :

This has been added for lucid. Thanks for the bug report.

Regards
chuck

Changed in samba (Ubuntu):
status: Confirmed → Fix Released
Revision history for this message
MarianoAbsatz (el-baby) wrote :

Any chance this gets into jaunty-updates (or -backports) or at least karmic-updates?... I wouldn't like to have to manually build the package every time samba gets updated :-(

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.