From 7e80ceb49b6f69960587a81339e94748a99c36b3 Mon Sep 17 00:00:00 2001
From: Stephen Gran <steve@lobefin.net>
Date: Tue, 19 May 2009 00:19:49 +0100
Subject: [PATCH] And a little tidy up Signed-off-by: Stephen Gran
 <steve@lobefin.net>

---
 dsa-nagios-checks/checks/dsa-check-running-kernel | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/dsa-nagios-checks/checks/dsa-check-running-kernel b/dsa-nagios-checks/checks/dsa-check-running-kernel
index 757459b..d1d65ae 100755
--- a/dsa-nagios-checks/checks/dsa-check-running-kernel
+++ b/dsa-nagios-checks/checks/dsa-check-running-kernel
@@ -50,7 +50,6 @@ get_avail() {
 	local prefix=linux
 
 	local kervers=$(uname -r)
-	local kernelversion=$(apt-cache policy ${prefix}-image-${kervers} 2>/dev/null | grep '^  Installed:' | awk '{print $2}')
 
 	local metavers=''
 
@@ -77,24 +76,24 @@ get_avail() {
 		return 2
 	fi
 
-	local last=0
+	local imagename=0
 	for vers in $(apt-cache --no-all-versions depends ${prefix}-image-${metavers} | grep Depends | awk '{print $2}' | sort -u); do
-		if dpkg --compare-versions $vers gt $last; then
-			last=$vers
+		if dpkg --compare-versions $vers gt $imagename; then
+			imagename=$vers
 		fi
 	done
-	local imagename=$last
 
 	if [ -z "$imagename" ] || [ "$imagename" = 0 ]; then
 		return 2
 	fi
 
-        if [ "$imagename" != "${prefix}-image-${kervers}" ]; then
-	        echo "$imagename" != "${prefix}-image-${kervers}"
+	if [ "$imagename" != "${prefix}-image-${kervers}" ]; then
+		echo "$imagename" != "${prefix}-image-${kervers}"
 		return 1
 	fi
 
 	local availvrs=$(apt-cache policy ${imagename} 2>/dev/null | grep '^  Candidate' | awk '{print $2}')
+	local kernelversion=$(apt-cache policy ${prefix}-image-${kervers} 2>/dev/null | grep '^  Installed:' | awk '{print $2}')
 
 	if [ "$availvrs" = "$kernelversion" ]; then
 		return 0
-- 
2.20.1