dsa-check-zone-rrsig-expiration-many: fix use of uninitialized value in numeric gt (>)

We have a state count array, and we assign each state (ok, warn, etc.) a
nagios error code.  one of the states we use internally is "unsigned",
which is not an error but did not have an integer exit code.  Give it 0
now.

diff --git a/dsa-nagios-checks/checks/dsa-check-zone-rrsig-expiration-many b/dsa-nagios-checks/checks/dsa-check-zone-rrsig-expiration-many
index ec99484..674dd1c 100755 (executable)
--- a/dsa-nagios-checks/checks/dsa-check-zone-rrsig-expiration-many
+++ b/dsa-nagios-checks/checks/dsa-check-zone-rrsig-expiration-many
@@ -190,6 +190,7 @@ my %state_mapping = (
        'unknown' => 255,
        'critical' => 2,
        'warn' => 1,
+       'unsigned' => 0,
        'ok' => 0 );
 
 for my $state (@$states) {

diff --git a/dsa-nagios-checks/debian/changelog b/dsa-nagios-checks/debian/changelog
index c5d80fe..4237d0e 100644 (file)
--- a/dsa-nagios-checks/debian/changelog
+++ b/dsa-nagios-checks/debian/changelog
@@ -2,6 +2,8 @@ dsa-nagios-checks (117) UNRELEASED; urgency=medium
 
   * dsa-check-hpssacli: add --ignore-cache
   * add: dsa-check-unbound-anchors
+  * dsa-check-zone-rrsig-expiration-many: fix use of uninitialized value
+    with unsigned zones.
 
  -- Peter Palfrader <weasel@debian.org>  Sun, 11 Mar 2018 09:06:13 +0100