From 12e2a3abe7ca640a7cef2630039c06964f779f17 Mon Sep 17 00:00:00 2001 From: Andrew Deason Date: Wed, 8 Feb 2012 16:03:29 -0600 Subject: [PATCH] RedHat: Fail openafs-client 'stop' on rmmod error Currently, the openafs-client RPM init script ignores any error reported by rmmod. If 'umount /afs' succeeds but rmmod does not, the client may panic the machine if the client is started again (from e.g. running the 'restart' init script method), since afsd will try to initialize AFS with a libafs that has been shut down. So, do not ignore errors from 'rmmod', and instead fail the 'stop' method from the init script if we get an error. Change-Id: Id4a07703fb4df69ad3a6a3569c91e48f73a0d309 Reviewed-on: http://gerrit.openafs.org/6709 Tested-by: BuildBot Reviewed-by: Jeffrey Altman Reviewed-by: Derrick Brashear --- src/packaging/RedHat/openafs-client.init | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/packaging/RedHat/openafs-client.init b/src/packaging/RedHat/openafs-client.init index f07ff1662..caaf89095 100644 --- a/src/packaging/RedHat/openafs-client.init +++ b/src/packaging/RedHat/openafs-client.init @@ -53,8 +53,11 @@ stop() { RETVAL=$? echo if [ $RETVAL -eq 0 ] ; then - rm -f /var/lock/subsys/openafs-client rmmod openafs + RETVAL=$? + fi + if [ $RETVAL -eq 0 ] ; then + rm -f /var/lock/subsys/openafs-client fi return $RETVAL } -- 2.39.5