nss-pam-ldapd branch master updated. 0.8.12-84-ga9aea20

[Commits of the nss-pam-ldapd project <nss-pam-ldapd-commits [at] lists.arthurdejong.org>]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "nss-pam-ldapd".

The branch, master has been updated
       via  a9aea20f136026d6553295727350c88fe2431e0c (commit)
       via  bfdf7cd045e4f622de75d5409063c8e91d61e3e3 (commit)
       via  1c6d856721974390b14ff7f0ee3bde8b37418ff1 (commit)
      from  d54243ab8eab01afad0b7877f7ae7bef310b091d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://arthurdejong.org/git/nss-pam-ldapd/commit/?id=a9aea20f136026d6553295727350c88fe2431e0c

commit a9aea20f136026d6553295727350c88fe2431e0c
Author: Arthur de Jong <arthur@arthurdejong.org>
Date:   Tue Feb 12 23:25:50 2013 +0100

    extra sanity check to ensure not too many file descriptors are open

diff --git a/nslcd/nslcd.c b/nslcd/nslcd.c
index 4be063a..ee3e029 100644
--- a/nslcd/nslcd.c
+++ b/nslcd/nslcd.c
@@ -297,6 +297,11 @@ static int create_socket(const char *filename)
     log_log(LOG_ERR, "cannot create socket: %s", strerror(errno));
     exit(EXIT_FAILURE);
   }
+  if (sock >= FD_SETSIZE)
+  {
+    log_log(LOG_ERR, "socket file descriptor number too high (%d)", sock);
+    exit(EXIT_FAILURE);
+  }
   /* remove existing named socket */
   if (unlink(filename) < 0)
   {

http://arthurdejong.org/git/nss-pam-ldapd/commit/?id=bfdf7cd045e4f622de75d5409063c8e91d61e3e3

commit bfdf7cd045e4f622de75d5409063c8e91d61e3e3
Author: Arthur de Jong <arthur@arthurdejong.org>
Date:   Sat Feb 23 17:30:24 2013 +0100

    clarify NSLCD_ACTION_SERVICE_* request parameter description

diff --git a/nslcd.h b/nslcd.h
index 24b1f14..456e2df 100644
--- a/nslcd.h
+++ b/nslcd.h
@@ -173,7 +173,9 @@
 #define NSLCD_ACTION_RPC_BYNUMBER      0x000a0002
 #define NSLCD_ACTION_RPC_ALL           0x000a0008
 
-/* Service (/etc/services) information requests. Result values are:
+/* Service (/etc/services) information requests. The BYNAME and BYNUMBER
+   requests contain an extra protocol string in the request which, if not
+   blank, will filter the services by this protocol. Result values are:
      STRING      service name
      STRINGLIST  service aliases
      INT32       service (port) number

http://arthurdejong.org/git/nss-pam-ldapd/commit/?id=1c6d856721974390b14ff7f0ee3bde8b37418ff1

commit 1c6d856721974390b14ff7f0ee3bde8b37418ff1
Author: Arthur de Jong <arthur@arthurdejong.org>
Date:   Sat Feb 23 17:29:54 2013 +0100

    allow names with one character in default validnames option and allow 
parentheses (taken from Fedora packages)

diff --git a/man/nslcd.conf.5.xml b/man/nslcd.conf.5.xml
index b43baa3..67b9b7a 100644
--- a/man/nslcd.conf.5.xml
+++ b/man/nslcd.conf.5.xml
@@ -733,7 +733,7 @@
        characters and the 'i' flag may be appended at the end to indicate
        that the match should be case-insensetive.
        The default value is
-       <literal>/^[a-z0-9._@$][a-z0-9._@$ \\~-]*[a-z0-9._@$~-]$/i</literal>
+       <literal>/^[a-z0-9._@$()]([a-z0-9._@$() 
\\~-]*[a-z0-9._@$()~-])?$/i</literal>
       </para>
      </listitem>
     </varlistentry>
diff --git a/nslcd/cfg.c b/nslcd/cfg.c
index afb4af7..018cd78 100644
--- a/nslcd/cfg.c
+++ b/nslcd/cfg.c
@@ -1048,7 +1048,7 @@ static void cfg_defaults(struct ldap_config *cfg)
   cfg->nss_min_uid = 0;
   cfg->validnames_str = NULL;
   handle_validnames(__FILE__, __LINE__, "",
-                    "/^[a-z0-9._@$][a-z0-9._@$ \\~-]*[a-z0-9._@$~-]$/i",
+                    "/^[a-z0-9._@$()]([a-z0-9._@$() 
\\~-]*[a-z0-9._@$()~-])?$/i",
                     cfg);
   cfg->ignorecase = 0;
   for (i = 0; i < NSS_LDAP_CONFIG_MAX_AUTHZ_SEARCHES; i++)
diff --git a/tests/test_common.c b/tests/test_common.c
index f1db304..29c899a 100644
--- a/tests/test_common.c
+++ b/tests/test_common.c
@@ -2,7 +2,7 @@
    test_common.c - simple test for the common module
    This file is part of the nss-pam-ldapd library.
 
-   Copyright (C) 2008, 2009, 2011, 2012 Arthur de Jong
+   Copyright (C) 2008, 2009, 2011, 2012, 2013 Arthur de Jong
 
    This library is free software; you can redistribute it and/or
    modify it under the terms of the GNU Lesser General Public
@@ -39,6 +39,8 @@ static void test_isvalidname(void)
   assert(!isvalidname("\\foo\\bar"));
   assert(!isvalidname("foo\\bar\\"));
   assert(isvalidname("me"));    /* try short name */
+  assert(isvalidname("f"));
+  assert(isvalidname("(foo bar)"));
 }
 
 /* the main program... */

-----------------------------------------------------------------------

Summary of changes:
 man/nslcd.conf.5.xml |    2 +-
 nslcd.h              |    4 +++-
 nslcd/cfg.c          |    2 +-
 nslcd/nslcd.c        |    5 +++++
 tests/test_common.c  |    4 +++-
 5 files changed, 13 insertions(+), 4 deletions(-)


hooks/post-receive
-- 
nss-pam-ldapd
-- 
To unsubscribe send an email to
nss-pam-ldapd-commits-unsubscribe@lists.arthurdejong.org or see
http://lists.arthurdejong.org/nss-pam-ldapd-commits/