Relevant BUGIDs:

Purpose of commit: bugfix

Commit summary:
---------------

Fix some bugs on older distributions

2006-08-31  Thorsten Kukuk  <kukuk@thkukuk.de>

        * modules/pam_namespace/Makefile.am: Don't install
        manual page if we don't build module.

        * m4/ld-as-needed.m4: Don't set LDFLAGS if check failed.
        * m4/ld-O1: Likewise.

diff --git a/ChangeLog b/ChangeLog
index 16ba5969b8139eedfbf87aa456d0378acd4da7f5..748d69536d3b0e0856c38b8962b1567cca575436 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2006-08-31  Thorsten Kukuk  <kukuk@thkukuk.de>
+
+       * modules/pam_namespace/Makefile.am: Don't install
+       manual page if we don't build module.
+
+       * m4/ld-as-needed.m4: Don't set LDFLAGS if check failed.
+       * m4/ld-O1: Likewise.
+
 2006-08-30  Tomas Mraz  <t8m@centrum.cz>
 
        * modules/pam_access/pam_access.8.xml: All services supported.

diff --git a/examples/xsh.c b/examples/xsh.c
index 08b4af11ed8f7d3713064d1164768a055a672177..ef4dca0c640e99997f6195813e6c04d6ad8fe80d 100644 (file)
--- a/examples/xsh.c
+++ b/examples/xsh.c
@@ -1,7 +1,3 @@
-/*
- * $Id$
- */
-
 /* Andrew Morgan (morgan@kernel.org) -- an example application
  * that invokes a shell, based on blank.c */
 
@@ -143,7 +139,7 @@ int main(int argc, char **argv)
                  (const char *)username);
 
          /* this is always a really bad thing for security! */
-         system("/bin/sh");
+         retcode = system("/bin/sh");
 
          /* close a session for the user --- `0' could be PAM_SILENT
           * it is possible that this pam_close_call is in another program..

diff --git a/m4/ld-O1.m4 b/m4/ld-O1.m4
index c8630740210c2e216e229c257bd92050e8156bd1..6efd3f34199733efb56865a8e75f12b3405abdf6 100644 (file)
--- a/m4/ld-O1.m4
+++ b/m4/ld-O1.m4
@@ -14,11 +14,11 @@ EOF
                                  -Wl,-O1 1>&AS_MESSAGE_LOG_FD])
   then
     pam_cv_ld_O1=yes
+    LDFLAGS="$LDFLAGS -Wl,-O1"
   else
     pam_cv_ld_O1=no
   fi
   rm -f conftest*])
   AC_SUBST(pam_cv_ld_as_needed)
-  LDFLAGS="$LDFLAGS -Wl,-O1"
   ]
 )

diff --git a/m4/ld-as-needed.m4 b/m4/ld-as-needed.m4
index cd81f91b0812f62a05985ffdae93dbb4809152c6..5ba6e0f9971116b85c272f065932eca84a158d0a 100644 (file)
--- a/m4/ld-as-needed.m4
+++ b/m4/ld-as-needed.m4
@@ -14,11 +14,11 @@ EOF
                                  -Wl,--as-needed 1>&AS_MESSAGE_LOG_FD])
   then
     pam_cv_ld_as_needed=yes
+    LDFLAGS="$LDFLAGS -Wl,--as-needed"
   else
     pam_cv_ld_as_needed=no
   fi
   rm -f conftest*])
   AC_SUBST(pam_cv_ld_as_needed)
-  LDFLAGS="$LDFLAGS -Wl,--as-needed"
   ]
 )

diff --git a/modules/pam_access/pam_access.8 b/modules/pam_access/pam_access.8
index de86275fc91d3f30d6fe0dd36ad63060b62963c9..b613e323745e7b0aeb66a3a2143497cb97af63f8 100644 (file)
--- a/modules/pam_access/pam_access.8
+++ b/modules/pam_access/pam_access.8
@@ -1,11 +1,11 @@
 .\"     Title: pam_access
 .\"    Author: 
-.\" Generator: DocBook XSL Stylesheets v1.70.1 <http://docbook.sf.net/>
-.\"      Date: 06/21/2006
+.\" Generator: DocBook XSL Stylesheets vsnapshot_2006\-08\-24_0226 <http://docbook.sf.net/>
+.\"      Date: 08/31/2006
 .\"    Manual: Linux\-PAM Manual
 .\"    Source: Linux\-PAM Manual
 .\"
-.TH "PAM_ACCESS" "8" "06/21/2006" "Linux\-PAM Manual" "Linux\-PAM Manual"
+.TH "PAM_ACCESS" "8" "08/31/2006" "Linux\-PAM Manual" "Linux\-PAM Manual"
 .\" disable hyphenation
 .nh
 .\" disable justification (adjust text to left margin only)
@@ -46,11 +46,7 @@ This option modifies the list separator character that pam_access will recognize
 will cause the default ` ' (space) and `\\t' (tab) characters to be treated as part of a list element value and `,' becomes the only list element separator. Doing this may be useful on a system with group information obtained from a Windows domain, where the default built\-in groups "Domain Users", "Domain Admins" contain a space.
 .SH "MODULE SERVICES PROVIDED"
 .PP
-The
-\fBauth\fR
-and
-\fBaccount\fR
-services are supported.
+All services are supported.
 .SH "RETURN VALUES"
 .TP 3n
 PAM_SUCCESS

diff --git a/modules/pam_namespace/Makefile.am b/modules/pam_namespace/Makefile.am
index 9c7c2f737c6be2898f29f907d9472ac656904520..bbcb2f702cbc7722dc1114098b40bd315677f6fe 100644 (file)
--- a/modules/pam_namespace/Makefile.am
+++ b/modules/pam_namespace/Makefile.am
@@ -1,13 +1,11 @@
 #
-# Copyright (c) 2006 Red Hat, Inc. 
+# Copyright (c) 2006 Red Hat, Inc.
 #
 
 CLEANFILES = *~
 MAN5 = namespace.conf.5
 MAN8 = pam_namespace.8
 
-man_MANS = $(MAN5) $(MAN8)
-
 XMLS = README.xml namespace.conf.5.xml pam_namespace.8.xml
 
 if ENABLE_REGENERATE_MAN
@@ -15,7 +13,7 @@ noinst_DATA = README
 -include $(top_srcdir)/Make.xml.rules
 endif
 
-EXTRA_DIST = README namespace.conf namespace.init $(man_MANS) $(XMLS) tst-pam_namespace
+EXTRA_DIST = README namespace.conf namespace.init $(MAN5) $(MAN8) $(XMLS) tst-pam_namespace
 
 noinst_HEADERS = md5.h
 
@@ -37,4 +35,5 @@ pam_namespace_la_SOURCES = pam_namespace.c pam_namespace.h md5.c md5.h
 secureconf_DATA = namespace.conf
 secureconf_SCRIPTS = namespace.init
 TESTS = tst-pam_namespace
+man_MANS = $(MAN5) $(MAN8)
 endif

diff --git a/modules/pam_succeed_if/pam_succeed_if.8 b/modules/pam_succeed_if/pam_succeed_if.8
index c94c5f43b21677969d8ef1b7af405db93bc9edb7..30af456cb73385fc92889339c07f451d55503576 100644 (file)
--- a/modules/pam_succeed_if/pam_succeed_if.8
+++ b/modules/pam_succeed_if/pam_succeed_if.8
@@ -1,11 +1,11 @@
 .\"     Title: pam_succeed_if
 .\"    Author: 
-.\" Generator: DocBook XSL Stylesheets v1.70.1 <http://docbook.sf.net/>
-.\"      Date: 08/05/2006
+.\" Generator: DocBook XSL Stylesheets vsnapshot_2006\-08\-24_0226 <http://docbook.sf.net/>
+.\"      Date: 08/31/2006
 .\"    Manual: Linux\-PAM
 .\"    Source: Linux\-PAM
 .\"
-.TH "PAM_SUCCEED_IF" "8" "08/05/2006" "Linux\-PAM" "Linux\-PAM"
+.TH "PAM_SUCCEED_IF" "8" "08/31/2006" "Linux\-PAM" "Linux\-PAM"
 .\" disable hyphenation
 .nh
 .\" disable justification (adjust text to left margin only)
@@ -101,11 +101,7 @@ User is not in given group.
 (user,host) is not in given netgroup.
 .SH "MODULE SERVICES PROVIDED"
 .PP
-The
-\fBauth\fR
-and
-\fBaccount\fR
-services are supported.
+All services are supported.
 .SH "RETURN VALUES"
 .TP 3n
 PAM_SUCCESS