Ubuntu
samba package

tdb2.so undefined symbol: dyn_get_STATEDIR (source code typo)

Bug #789097 reported by Dave L
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
samba (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: samba

This error occurs if "idmap backend = tdb2" is specified in smb.conf

The source of the error is this patchfile
fhs-filespath.patch

The offending patch is at the bottom of the file, as follows

===================================================================
--- a/source3/winbindd/idmap_tdb2.c
+++ b/source3/winbindd/idmap_tdb2.c
@@ -107,7 +107,7 @@
  if (db_path == NULL) {
   /* fall back to the private directory, which, despite
      its name, is usually on shared storage */
- db_path = talloc_asprintf(NULL, "%s/idmap2.tdb", lp_private_dir());
+ db_path = talloc_asprintf(NULL, "%s/idmap2.tdb", dyn_get_STATEDIR());
  }
  NT_STATUS_HAVE_NO_MEMORY(db_path);

The correct function name is get_dyn_STATEDIR, not dyn_get_STATEDIR.

I've tested it can be fixed by changing this accordingly.

ProblemType: Bug
DistroRelease: Ubuntu 11.04
Package: samba 2:3.5.8~dfsg-1ubuntu2.2
ProcVersionSignature: Ubuntu 2.6.38-8.42-generic 2.6.38.2
Uname: Linux 2.6.38-8-generic x86_64
Architecture: amd64
BothFailedConnect: Yes
Date: Fri May 27 12:28:59 2011
InstallationMedia: Ubuntu 9.10 "Karmic Koala" - Release amd64 (20091027)
ProcEnviron:
 LANGUAGE=en_GB:en
 PATH=(custom, user)
 LANG=en_GB.UTF-8
 SHELL=/bin/bash
SambaServerRegression: No
SmbConfIncluded: No
SourcePackage: samba
UpgradeStatus: Upgraded to natty on 2011-05-09 (17 days ago)

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

Which version is this for?

Regards
chuck

Changed in samba (Ubuntu):
importance: Undecided → Low
status: New → Incomplete
Revision history for this message
Dave L (westy3) wrote :

samba_3.5.8~dfsg-1ubuntu2.2

And the previous version was the same.

Revision history for this message
Christian Perrier (bubulle) wrote : Re: [Samba] idmap backend = tdb2 is broken in Ubuntu / Debian

Quoting Dave Lawrence (<email address hidden>):
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> The tdb2 backend for idmap is broken in Ubuntu and presumably debian.
>
> As reported here
> https://bugs.launchpad.net/bugs/789097
>
> Careful how you read it, the quoted patch is the source of the bug, NOT
> the fix.

Thanks for pointing this out (I actually missed the BR in LP, even
though I receive these bug reports).

Brynnen Owen mentioned this to us, recently, indeed.

The fix has been committed in the Debian package SVN. It will reach
Debian unstable, then Ubuntu, when we upload a new release.

Given that the fix is trivial but has important consequences for
anybody wanting to use the TDB2 backend, I'll ask for permission to
upload a fix in squeeze too.

Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.3 KiB)

This bug was fixed in the package samba - 2:3.5.8~dfsg-5ubuntu1

---------------
samba (2:3.5.8~dfsg-5ubuntu1) oneiric; urgency=low

  * Merge from debian unstable. Remaining changes:
    + debian/patches/VERSION.patch:
      - set SAMBA_VERSION_SUFFIX to Ubuntu.
    + debian/smb.conf:
      - add "(Samba, Ubuntu)" to server string.
      - comment out the default [homes] share, and add a comment about
        "valid users = %S" to show users how to restrict access to
        \\server\username to only username.
      - Set 'usershare allow guests', so that usershare admins are
        allowed to create public shares in addition to authenticated
        ones.
      - add map to guest = Bad user, maps bad username to guest access.
    + debian/samba-common.config:
      - Do not change priority to high if dhclient3 is installed.
      - Use priority medium instead of high for the workgroup question.
    + debian/mksmbpasswd.awk:
      - Do not add user with UID less than 1000 to smbpasswd
    + debian/control:
      - Make libwbclient0 replace/conflict with hardy's likewise-open.
      - Don't build against or suggest ctdb.
      - Add dependency on samba-common-bin to samba.
      - Add cuups breaks to push the package to aslo upgrade cups (LP: #639768)
    + debian/rules:
      - enable "native" PIE hardening.
      - Add BIND_NOW to maximize benefit of RELRO hardening.
    + Add ufw integration:
      - Created debian/samba.ufw.profile
      - debian/rules, debian/samba.dirs, debian/samba.files: install
        profile
      - debian/control: have samba suggest ufw
    + Add apport hook:
      - Created debian/source_samba.py.
      - debian/rules, debian/samba.dirs, debian/samba-common-bin.files: install
    + Switch to upstart:
      - Add debian/samba.{nmbd,smbd}.upstart.
      - Don't ship the /etc/network/if-up.d file.
    + debian/samba.postinst:
      - Fixed bashism.
      - Avoid scary pdbedit warnings on first import.
    + debian/samba-common.postinst: Add more informative error message for
      the case where smb.conf was manually deleted
    + debian/samba.logrotate: Make it upstart compatible
    + debian/samba-common.dhcp: Fix typo to get a proper parsing in
      /etc/samba/dhcp.
    + debian/patches/fix-upstream-lp-738968.patch: Fix connection to EMC
      Celerra NAS version 5.6.50. (LP: #738968)
    + debian/patches/fix-debuglevel-name-conflict.patch: Fix issue with the
      "debug_level" variable name in libnss_wins.so conflicting with the "debug_level"
      global variable that some applications. (LP: #529714)

samba (2:3.5.8~dfsg-5) unstable; urgency=low

  * Fix "tdb2.so undefined symbol: dyn_get_STATEDIR" by fixing a typo
    in fhs-filespath.patch. Closes: #629183, LP: #789097

samba (2:3.5.8~dfsg-4) unstable; urgency=low

  [ Debconf translations ]
  * Spanish (Omar Campagne). Closes: #627813
  * Swedish (Martin Bagge / brother). Closes: #627849
  * Brazilian Portuguese (Adriano Rafael Gomes). Closes: #627866

  [ Christian Perrier ]
  * bug_601406_fix-perl-path-in-example.patch: fix path to perl
    binary in example file. Closes: #601406

samba (2:3.5.8~dfsg-3) unstable; urgency=low

  [ Debconf translatio...

Read more...

Changed in samba (Ubuntu):
status: Incomplete → Fix Released
Revision history for this message
Christian Perrier (bubulle) wrote :

Quoting Christian PERRIER (<email address hidden>):

> The fix has been committed in the Debian package SVN. It will reach
> Debian unstable, then Ubuntu, when we upload a new release.

A new package fixing this was uploaded in Debian unstable on Sunday
June 5th. Package version is 2:3.5.8~dfsg-5.

For Ubuntu, a resync with Debian unstable should fix the problem in
the currently developed version "whatever funky name it might have" (I
can't cope with Ubuntu release names!). A fix is probably needed for
Ubuntu 11.04. I think that the diff between Debian 2:3.5.8~dfsg-4 and
-5 should be OK. I'm not sure whether the problem is also in the
current Ubuntu LTS.

I'm awaiting for the Debian release managers approval for a fix to be
uploaded for Debian stable.

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

Other bug subscribers

Remote bug watches

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