Fixes for custom_query.
authorGreg Sabino Mullane <[email protected]>
Mon, 3 Jan 2011 19:04:30 +0000 (14:04 -0500)
committerGreg Sabino Mullane <[email protected]>
Mon, 3 Jan 2011 19:04:30 +0000 (14:04 -0500)
check_postgres.pl

index 968c1c6bfd4ca15e609509f52367b1f0115ed902..de56cbea052cbc82cb0445c02284352d3d80822b 100755 (executable)
@@ -3155,6 +3155,10 @@ sub check_custom_query {
             $goodrow++;
             $db->{perf} .= " $msg";
             my $gotmatch = 0;
+            if (! defined $data) {
+                add_unknown msg('custom-invalid');
+                return;
+            }
             if (length $critical) {
                 if (($valtype eq 'string' and $data eq $critical)
                     or
@@ -3167,7 +3171,7 @@ sub check_custom_query {
             if (length $warning and ! $gotmatch) {
                 if (($valtype eq 'string' and $data eq $warning)
                     or
-                    ($reverse ? $data <= $warning : $data >= $warning)) {
+                    (length $data and $reverse ? $data <= $warning : $data >= $warning)) {
                     add_warning "$data";
                     $gotmatch = 1;
                 }
@@ -6937,7 +6941,7 @@ WHERE xact_start IS NOT NULL $USERWHERECLAUSE
     ## Use of skip_item means we may have no matches
     if ($maxdb eq '?') {
         if ($USERWHERECLAUSE) { ## needed?
-            add_unknown msg('fixme-nomatch');
+            add_unknown msg('txntime-none');
         }
         else {
             add_ok msg('txntime-none');