[PATCH] squid: Update-suggestion for initscript

[Matthias Fischer <matthias.fischer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 3363 bytes --]

Path fix for squid.

Changed loop detecting if 'squid' is still running.

Added '/bin/rm -f /var/log/cache/swap.state' after killing 'squid'.

Changed flush option to '/bin/rm -f /var/log/cache/swap.state'.

Signed-off-by: Matthias Fischer <matthias.fischer(a)ipfire.org>
---
 src/initscripts/init.d/squid | 47 +++++++++++++++++++++++++++-----------------
 1 file changed, 29 insertions(+), 18 deletions(-)

diff --git a/src/initscripts/init.d/squid b/src/initscripts/init.d/squid
index c641c7d..eecad12 100644
--- a/src/initscripts/init.d/squid
+++ b/src/initscripts/init.d/squid
@@ -33,21 +33,22 @@ transparent() {
 			CONN_TYPE=`echo "$LINE" | awk -F, '{ print $5 }'`
 			if [ "$CONN_TYPE" != "net" ]; then
 				continue
-			fi  
-  		iptables -t nat -A SQUID -i $1 -p tcp -d `echo "$LINE" | awk -F, '{ print $13 }'` --dport 80 -j RETURN
+			fi
+		iptables -t nat -A SQUID -i $1 -p tcp -d `echo "$LINE" | awk -F, '{ print $13 }'` --dport 80 -j RETURN
 		done < $FILE
-		
+
 		if [ "$RED_TYPE" == "STATIC" ]; then
 			iptables -t nat -A SQUID -i $1 -p tcp -d $RED_NETADDRESS/$RED_NETMASK --dport 80 -j RETURN
 		fi
-		
+
 		iptables -t nat -A SQUID -i $1 -p tcp -d $LOCALIP --dport 80 -j RETURN
-		
+
 		iptables -t nat -A SQUID -i $1 -p tcp --dport 80 -j REDIRECT --to-port "${TRANSPARENT_PORT}"
 }
 
 case "$1" in
 	start)
+
 		getpids "squid"
 
 		if [ -n "${pidlist}" ]; then
@@ -93,14 +94,16 @@ case "$1" in
 		;;
 
 	stop)
+
 			iptables -t nat -F SQUID
 		if [ -e /var/run/squid.pid ]; then
 			boot_mesg "Stopping Squid Proxy Server..."
-			squid -k shutdown >/dev/null 2>&1
+			/usr/sbin/squid -k shutdown >/dev/null 2>&1
 			evaluate_retval
 
 			# Stop squidGuard, updxlrator, squidclamav
-			# and redirect_wrappers.			
+			# and redirect_wrappers.
+
 			killproc /usr/bin/squidGuard >/dev/null &
 			killproc /usr/sbin/updxlrator >/dev/null &
 			killproc /usr/bin/squidclamav >/dev/null &
@@ -109,22 +112,30 @@ case "$1" in
 			# Wait until all redirectors have been stopped.
 			wait
 
-			# If squid is still running, wait up to 30 seconds
+			# If squid is still running, wait up to 120 seconds
 			# before we go on to kill it.
-			counter=30
 
-			while [ ${counter} -gt 0 ]; do
-				statusproc /usr/sbin/squid >/dev/null && break;
+			n=0
+			while /usr/sbin/squid -k check && [ $n -lt 120 ]; do
 				sleep 1
-				counter=$(( ${counter} - 1))
+				echo -n .
+				n=`/usr/bin/expr $n + 1`
+
 			done
 
-			# Kill squid service, if still running.
+# Kill squid service, if still running.
 			killproc /usr/sbin/squid >/dev/null
 
-			# Trash remain pid file from squid.
-			rm -rf /var/run/squid.pid	
+# Delete - damaged - /var/log/cache/swap.state
+			/bin/rm -f /var/log/cache/swap.state
+
+		fi
+
+		# Delete remaining pid file from squid if it STILL exists.
+		if [ -e '/var/run/squid.pid' ]; then
+			rm -rf /var/run/squid.pid
 		fi
+
 		;;
 
 	restart)
@@ -141,14 +152,14 @@ case "$1" in
 		statusproc /usr/sbin/squid
 		statusproc /usr/lib/squid/unlinkd
 		;;
-		
+
 	flush)
 		$0 stop
-		echo > /var/log/cache/swap.state
-		chown squid.squid /var/log/cache/swap.state
+		/bin/rm -f /var/log/cache/swap.state
 		sleep 1
 		$0 start
 		;;
+
 	setperms)
 		chown -R nobody.squid /var/updatecache/
 		;;
-- 
2.8.2