Bug#727177: Upgrade of libnss-ldap to 265-1 causes important binaries to segfault

2013-10-26 Thread Arthur de Jong
On Thu, 2013-10-24 at 10:25 -0400, Klee Dienes wrote:
> The issue is that libnss-ldap is ending up with a dependency on
> __libc_lock_lock, which was removed from glibc.

Thanks for the pointer to the patch. I thought I tested the release
before uploading but apparently I was mistaken. Sorry about that.

Preparing another upload with this fix (I'll run some tests this time).

> A few other thoughts:
> 
> * It might be nice to build nss-ldap with -Wimplicit -Werror or
> something along those lines.  FTBFS is much better than fail-to-boot.
> * 'sudo' just crashes on null pointer dereference; 'su' complains
> about the link error.  It'd be much better if the client apps would
> just ignore the missing nsswitch module.

Patches are always welcome, however, libnss-ldap currently doesn't have
a maintainer so if someone who actually regularly uses this package
would step up to become maintainer that would help.

I personally use nss-pam-ldapd which is easier to maintain and contains
almost all features nss_ldap has but I understand some people still
prefer nss_ldap so I uploaded a new release that should fix some bugs.

Kind regards,

-- 
-- arthur - adej...@debian.org - http://people.debian.org/~adejong --


signature.asc
Description: This is a digitally signed message part


Bug#732077: libnss-ldap: Fails to install via chroot

2013-12-21 Thread Arthur de Jong
On Fri, 2013-12-13 at 17:36 +0100, Olivier Sallou wrote:
> I face an issue when installing libnss-ldap in a chroot environment
> 
> Error message is:
> 
> [597853.361845] DEBUG: Setting up libnss-ldap:amd64 (264-2.5) ...
> [598058.244944] DEBUG: locale: Cannot set LC_CTYPE to default locale: No such 
> file or directory
> [598058.34198] DEBUG: locale: Cannot set LC_MESSAGES to default locale: No 
> such file or directory
> [598058.418989] DEBUG: locale: Cannot set LC_ALL to default locale: No such 
> file or directory
> [598219.187021] DEBUG: invoke-rc.d: unknown initscript, /etc/init.d/nscd not 
> found.
> 
> It fails because invoke-rc.d does not find /etc/init.d/nscd.

Can you reproduce this with version 265-3? Amongst other things it
includes a number of fixes related to the restarting of nscd if it is
installed and this should be one of them.

Thanks,

-- 
-- arthur - adej...@debian.org - http://people.debian.org/~adejong --


signature.asc
Description: This is a digitally signed message part


Bug#732077: ok in sid

2013-12-30 Thread Arthur de Jong
Control: fixed -1 libnss-ldap/265-3

On Mon, 2013-12-30 at 13:06 +0100, Olivier Sallou wrote:
> I tested as per requested in a sid chroot with v265-3 and install is
> fine.

Thanks,

Marking the bug as fixed in sid. I'm not sure this bug is worthy of an
update in a wheezy point release.

For those wanting to back-port this fix, the nscd restarting logic of
265-3 in the postinst is:

# Restart (u)nscd to have it load the new NSS module
if [ -s /etc/init.d/nscd ]; then
  if pidof nscd > /dev/null; then
if which invoke-rc.d >/dev/null 2>&1; then
  invoke-rc.d nscd restart || true
else
  /etc/init.d/nscd restart || true
fi
  fi
elif [ -s /etc/init.d/unscd ]; then
  if pidof nscd > /dev/null; then
if which invoke-rc.d >/dev/null 2>&1; then
  invoke-rc.d unscd restart || true
else
  /etc/init.d/unscd restart || true
fi
  fi
fi

-- 
-- arthur - adej...@debian.org - http://people.debian.org/~adejong --


signature.asc
Description: This is a digitally signed message part


Bug#692472: ncpfs - Fails with Cannot convert kernel release "3.6-trunk-amd64" to number

2012-11-10 Thread Arthur de Jong
Control: tags -1 + patch

On Tue, 2012-11-06 at 16:25 +0100, Bastian Blank wrote:
> ncpmount fails on current Debian kernels with:
> | Cannot convert kernel release "3.6-trunk-amd64" to number
> 
> It expects three digits, but this is not longer the case.

Attached is a patch, suitable for being dropped in debian/patches that
short-circuits the test altogether. I haven't tested it because I
haven't used IPX in a few years.

-- 
-- arthur - adej...@debian.org - http://people.debian.org/~adejong --
Description: avoid doing a kernel version check to handle 3.x kernels
 This patch takes out the kernel version checks that were in place in
 ncpfs. The checks in place have some special handling for kernels older
 than 2.5.31. Since no-one is expected to use these old kernels any more
 it is easiest to remove the checks.
Author: Arthur de Jong 
Bug-Debian: http://bugs.debian.org/692472

--- a/sutil/ncpm_common.c
+++ b/sutil/ncpm_common.c
@@ -244,6 +244,8 @@ static int load_ncpfs(void)
 #endif	/* MOUNT2 */
 
 static int getmountver(void) {
+/*
+  Commented out because this is only useful for kernels before 2.5.31.
 	struct utsname name;
 	int maj, mid, rev;
 	int ver;
@@ -261,6 +263,7 @@ static int getmountver(void) {
 		return 3;
 	if (ver < 0x2051F)
 		return 4;
+*/
 	return 5;
 }
 


signature.asc
Description: This is a digitally signed message part


Bug#692471: ncpfs - Fails to build to times

2012-11-10 Thread Arthur de Jong
Control: tags -1 + patch

On Tue, 2012-11-06 at 16:24 +0100, Bastian Blank wrote:
> | dpkg-source: info: local changes detected, the modified files are:
> |  ncpfs-2.2.6/include/private/libncp-atomic.h

Attached is a trivial patch which fixes this issue.

-- 
-- arthur - adej...@debian.org - http://people.debian.org/~adejong --
--- debian/rules.orig	2012-11-10 22:43:30.0 +0100
+++ debian/rules	2012-11-10 22:53:13.0 +0100
@@ -38,6 +38,7 @@
 	dh_testroot
 	rm -f build-stamp install-stamp po/de.gmo
 	rm -f config.sub config.guess
+	rm -f include/private/libncp-atomic.h
 
 	# Add here commands to clean up after the build process.
 	[ ! -f Makefile ] || $(MAKE) distclean


signature.asc
Description: This is a digitally signed message part