From: Erez Zadok <ezk@cs.sunysb.edu>
Date: Tue, 3 May 2005 03:14:31 +0000 (+0000)
Subject: * doc/am-utils.texi (amd): document proper use of
X-Git-Tag: am-utils-6_1_rc2~4
X-Git-Url: https://git.fsl.cs.sunysb.edu/?a=commitdiff_plain;h=4a376296b1555dcfb3cb832fb5278f9dc0142917;p=am-utils-6.2.git

* doc/am-utils.texi (amd): document proper use of
/etc/hosts.allow.  Don't spawn a new Amd process because Amd will
get confused if someone else waitpid's on its children (which
often do important un/mount work).  Issue reported by Francis
Montagnac <Francis.Montagnac at sophia.inria.fr>.
---

diff --git a/ChangeLog b/ChangeLog
index e243ae8f..c106e258 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2005-05-02  Erez Zadok  <ezk@cs.sunysb.edu>
 
+	* doc/am-utils.texi (amd): document proper use of
+	/etc/hosts.allow.  Don't spawn a new Amd process because Amd will
+	get confused if someone else waitpid's on its children (which
+	often do important un/mount work).  Issue reported by Francis
+	Montagnac <Francis.Montagnac at sophia.inria.fr>.
+
 	* minor new ports: powerpc64-unknown-linux-rhel4, and
 	powerpc64-unknown-linux-suse9.
 
diff --git a/NEWS b/NEWS
index d17369c1..95dd7fa8 100644
--- a/NEWS
+++ b/NEWS
@@ -22,6 +22,7 @@ version (10.3.x) than uname(3) reports.
 	* fix double-free in type:=nfsx
 	* timeo and retrans shouldn't be set for type:=toplvl
 	* fix inconsistency in handling filehandle generation number
+	* document proper use of hosts.allow (don't spawn)
 
 *** Notes specific to am-utils version 6.1-rc1
 
diff --git a/doc/am-utils.texi b/doc/am-utils.texi
index ebc1c0ff..f1e64cc2 100644
--- a/doc/am-utils.texi
+++ b/doc/am-utils.texi
@@ -38,7 +38,7 @@
 @c
 @c      %W% (Berkeley) %G%
 @c
-@c $Id: am-utils.texi,v 1.101 2005/04/17 03:05:54 ezk Exp $
+@c $Id: am-utils.texi,v 1.102 2005/05/03 03:14:31 ezk Exp $
 @c
 @setfilename am-utils.info
 
@@ -4972,8 +4972,14 @@ and this line to @code{/etc/hosts.deny}:
 amd: ALL
 @end example
 
-Consult the man pages for @b{hosts_access}(5) for more information on
-using the tcpwrappers access-control library.
+Consult the man pages for @b{hosts_access}(5) for more information on using
+the tcpwrappers access-control library.
+
+Note that in particular, you should not configure your @code{hosts.allow}
+file to spawn a command for @i{Amd}: that will cause @i{Amd} to not be able
+to @code{waitpid} on the child process ID of any background un/mount that
+@i{Amd} issued, resulting in a confused @i{Amd} that does not know what
+happened to those background un/mount requests.
 
 @c ----------------------------------------------------------------
 @node vendor Parameter, , use_tcpwrappers Parameter, Global Parameters