Fixed error in rtt/rta calculation in case of system clock problems
Time differences are now set to 0 in case of backward timejumps so there are no wrap-around problems any more.
The RTA calculation hopefully gets a more accurate value in these cases also.
Time differences are now set to 0 in case of backward timejumps so there are no wrap-around problems any more.
The RTA calculation hopefully gets a more accurate value in these cases also.
Fixed testcases for check_snmp, check_ifstatus and check_ifoperstatus when no snmp community is specified
The checks for invalid/unavailable hosts need a complete command line.
They failed on hosts where no community was specified. Now, an invalid community is used so
the tests can run on hosts w/o snmp host/community in NPTest.cache. Execution will fail anyway.
The checks for invalid/unavailable hosts need a complete command line.
They failed on hosts where no community was specified. Now, an invalid community is used so
the tests can run on hosts w/o snmp host/community in NPTest.cache. Execution will fail anyway.
15 years agoFurther test parameters and skip blocks for check_ifoperstatus.t. getTestparam change...
Further test parameters and skip blocks for check_ifoperstatus.t. getTestparam changes for check_ifstatus.t and ifoperstatus.t
Added snmp_interface and snmp_ifxtable test parameters to check_ifoperstatus.t.
Also getTestParameter calls for both plugins were converted to use three arguments
instead of undocumented >3 arg behaviour which led to redundant data in NPTest.cache.
Added snmp_interface and snmp_ifxtable test parameters to check_ifoperstatus.t.
Also getTestParameter calls for both plugins were converted to use three arguments
instead of undocumented >3 arg behaviour which led to redundant data in NPTest.cache.
Merge branch 'master' of ssh://PsychoTrahe@repo.or.cz/srv/git/nagiosplugins
renamed check_ifoperstatus' newly introdced -x to -P so it fits check_ifstatus
check_ifstatus already used -x so both plugins can use same short option, now.
to specify privacy protocol.
check_ifstatus already used -x so both plugins can use same short option, now.
to specify privacy protocol.
Fixed SNMPv3 behaviour of check_ifstatus. Added -P to define privprotocol (#2343438 - Robin Schroeder)
check_ifstatus didn't function correctly with SNMPv3. This is fixed now.
Created argument-hash for SNMP session creation. This removes redundant code.
Session creation was moved out of process_arguments() and now takes place
after setting the timeout handler.
Additionally the -P argument was added to specify the privprotocol.
check_ifstatus didn't function correctly with SNMPv3. This is fixed now.
Created argument-hash for SNMP session creation. This removes redundant code.
Session creation was moved out of process_arguments() and now takes place
after setting the timeout handler.
Additionally the -P argument was added to specify the privprotocol.
Added testcases for check_ifoperstatus
Added testcases for check_ifoperstatus. No SNMPv3 testing
included for now.
Added testcases for check_ifoperstatus. No SNMPv3 testing
included for now.
minor fixes to check_ifoperstatus
Minor output fixes for check_ifoperstatus.
Fixed typo so that -n now implies -I as documented.
Minor output fixes for check_ifoperstatus.
Fixed typo so that -n now implies -I as documented.
Use a more "natural" way of writing argument lists in extra-opts tests.
check_ifoperstatus is now more user friendly in case of missing arguments
Added proper usage messages in case of missing arguments.
Removed accidently committed Data::Dumper call.
Minor whitespace fixes.
Added proper usage messages in case of missing arguments.
Removed accidently committed Data::Dumper call.
Minor whitespace fixes.
Whitespace and indent fixes for check_ifoperstatus
15 years agoFixed SNMPv3 behaviour of check_ifoperstatus. Added -x to define privprotocol (#23434...
Fixed SNMPv3 behaviour of check_ifoperstatus. Added -x to define privprotocol (#2343438 - Robin Schroeder)
check_ifoperstatus didn't function correctly with SNMPv3. This is fixed now.
Created argument-hash for SNMP session creation. This removes redundant code.
Session creation was moved out of process_arguments() and now takes place
after setting the timeout handler.
Additionally the -x argument was added to specify the privprotocol.
check_ifoperstatus didn't function correctly with SNMPv3. This is fixed now.
Created argument-hash for SNMP session creation. This removes redundant code.
Session creation was moved out of process_arguments() and now takes place
after setting the timeout handler.
Additionally the -x argument was added to specify the privprotocol.
Fix the memory allocation for the thresholds data
Allocate the appropriate amount of memory for storing the thresholds
data. Before, we allocated the amount of memory required for storing a
_pointer_ to the thresholds data. This crashed (at least) check_mysql
when using its "-S" option on FreeBSD/amd64 (as reported and analyzed by
Nikita Kalabukhov - 2797757).
Signed-off-by: Holger Weiss <holger@zedat.fu-berlin.de>
Allocate the appropriate amount of memory for storing the thresholds
data. Before, we allocated the amount of memory required for storing a
_pointer_ to the thresholds data. This crashed (at least) check_mysql
when using its "-S" option on FreeBSD/amd64 (as reported and analyzed by
Nikita Kalabukhov - 2797757).
Signed-off-by: Holger Weiss <holger@zedat.fu-berlin.de>
Don't use the plain char type for data buffers
Buffers which are used for holding arbitrary data should be referenced
using pointers to void and accessed using pointers to unsigned char.
Signed-off-by: Holger Weiss <holger@zedat.fu-berlin.de>
Buffers which are used for holding arbitrary data should be referenced
using pointers to void and accessed using pointers to unsigned char.
Signed-off-by: Holger Weiss <holger@zedat.fu-berlin.de>
Whitespace indent fixes for check_pgsql
Fixed check_icmp compiler warnings by including float.h
Self defined DBL_MAX caused compiler warnings. Now float.h
which defines DBL_MAX is included.
Self defined DBL_MAX caused compiler warnings. Now float.h
which defines DBL_MAX is included.
Fixed --help output for warn and critical thresholds (sf.net #2796624, debian #530553)
check_icmp wrongly calculated the current thresholds in --help output.
check_icmp wrongly calculated the current thresholds in --help output.
Added verbose output to check_pgsql (#2799281 - Jun Kuriyama)
check_pgsql displayed the -v flag in help output, but didn't
accept the verbosity argument. Added -v and basic verbose output.
check_pgsql displayed the -v flag in help output, but didn't
accept the verbosity argument. Added -v and basic verbose output.
check_fping now supports passing target timeout and interval to fping (#2347686 - Martin Foster)
Added support for passing timeout and interval length to check_fping.
To protect compatibility, both options use fping's defaults if they
are not specified on the command line.
Added support for passing timeout and interval length to check_fping.
To protect compatibility, both options use fping's defaults if they
are not specified on the command line.
check_snmp: Make use of standard threshold functions
This patch makes use of standard threshold functions. This allows using
doubles as thresholds.
Since SNMP supports only integers, double precision numbers are only
printed when parsed from a STRING type.
In addition, support for printing properly Timeticks type has been added,
and the code has been thoroughly cleaned.
This patch makes use of standard threshold functions. This allows using
doubles as thresholds.
Since SNMP supports only integers, double precision numbers are only
printed when parsed from a STRING type.
In addition, support for printing properly Timeticks type has been added,
and the code has been thoroughly cleaned.
Fix whitespaces in check_snmp
Fixed wrong perfdata label for output traffic in check_mrtgtraf
check_mrtgtraf used the label "in" for both input and output traffic.
This fixed, now.
check_mrtgtraf used the label "in" for both input and output traffic.
This fixed, now.
Removed -n option in check_smtp's help and usage output (#2782784)
Check_smtp's help showed the -n/--nocommand options to not send an
SMTP command. This usage is covered by check_tcp and is thus not needed
in check_smtp.
Check_smtp's help showed the -n/--nocommand options to not send an
SMTP command. This usage is covered by check_tcp and is thus not needed
in check_smtp.
Whitespace changes only
Fix a few minor typos
Signed-off-by: Holger Weiss <holger@zedat.fu-berlin.de>
Signed-off-by: Holger Weiss <holger@zedat.fu-berlin.de>
Talk about Git instead of CVS
Update the Developer Guidelines to reflect the fact that our development
is done using Git instead of CVS these days (as noted by Chris Pepper on
the "nagiosplug-devel" list).
While at it, apply another few minor cleanups.
Signed-off-by: Holger Weiss <holger@zedat.fu-berlin.de>
Update the Developer Guidelines to reflect the fact that our development
is done using Git instead of CVS these days (as noted by Chris Pepper on
the "nagiosplug-devel" list).
While at it, apply another few minor cleanups.
Signed-off-by: Holger Weiss <holger@zedat.fu-berlin.de>
Remove comments regarding former requirements
Don't explain why Libtool and Gettext are no longer needed. Libtool
actually is a requirement, as (at least) the M4 macros provided by
Libtool are needed for our Autotools setup. And the fact that Gettext
is no longer needed probably isn't that interesting for new developers.
Signed-off-by: Holger Weiss <holger@zedat.fu-berlin.de>
Don't explain why Libtool and Gettext are no longer needed. Libtool
actually is a requirement, as (at least) the M4 macros provided by
Libtool are needed for our Autotools setup. And the fact that Gettext
is no longer needed probably isn't that interesting for new developers.
Signed-off-by: Holger Weiss <holger@zedat.fu-berlin.de>
Fixed typo in check_disk
15 years agoFixed crash from check_nt when -H unspecified or given multiple times (sf.net #273175...
Fixed crash from check_nt when -H unspecified or given multiple times (sf.net #2731755, debian #521097)
check_nt: allow returning UNKNOWN on timeouts (-u)
Whitespace and intend cleanup
Update check_http short help with new stycky and stickyport options.
check_http: die when an invalid --onredirect option is specified
check_http: add --onredirect=stickyport - also follow the same port
Add the timegm Gnulib module
negate: make timeouts configurable
Make alarm handler customisable
negate: Fix not printing the real timeout in ALRM handler when timeout is left to default
tests/check_http.t: Add various timeout-related tests
Fixed check_http not timing-out on redirects
15 years agoRevert "Add timezone support and fix checks around cert expiration" (keep the expirat...
Revert "Add timezone support and fix checks around cert expiration" (keep the expiration fix)
This reverts commit d41a33a434558189300113c28b26e2d3d681d390.
This reverts commit d41a33a434558189300113c28b26e2d3d681d390.
Add suggested multi-oid test (John A. Barbuto)
check_snmp: Fix potential buffer overflow - enforce MAX_OIDS limit (John A. Barbuto)
Fix logging of STDERR in tinderbox tests phase
Add timezone support and fix checks around cert expiration
Check for existence of pre-instaleld pst3 for Solaris tests in t/check_procs.t
Fix check_snmp test on solaris
Update extended check_http tests for 0d78171...
Also add some tests, and make debug mode loop infinitely.
Also add some tests, and make debug mode loop infinitely.
check_http fixes:
- Fix broken printf on 500 errors
- Add bytes to normal output (accidentally removed in 0d78171)
- Fix broken printf on 500 errors
- Add bytes to normal output (accidentally removed in 0d78171)
check_snmp fixes:
- Fix broken compilation caused by typo in command ending
- Remove extra whitespaces at EOL
- Fix invalid host test on Ubuntu Hardy (and possibly others)
- Fix broken compilation caused by typo in command ending
- Remove extra whitespaces at EOL
- Fix invalid host test on Ubuntu Hardy (and possibly others)
Ignore stderr messages unless return code is non-zero or there is no output
Create expired cert for testing purposes. Updated tests to check
expired and unexpired certificates
expired and unexpired certificates
Merge branch 'master' of ssh://repo.or.cz/srv/git/nagiosplugins
Added inline tests for HTTPS using HTTP::Daemon::SSL
check_snmp long help missing retries entry (#2608223)
Fix for check_snmp long help, reported by Richard Edward Horner
Fix for check_snmp long help, reported by Richard Edward Horner
Updated string ids, due to typo fixes
Devel doc: NP-VERSION-GEN needs to be updated as well
Small correction in NEWS entry
Update the devel notes for Git
Revert "Moving project administration documents to nagiosplugins.org site"
This reverts commit f4a52594818e7c3634a397ac1f86ef87a68bd121.
This reverts commit f4a52594818e7c3634a397ac1f86ef87a68bd121.
Add NEWS entry for last commit
Fix aberrant behaviours in check_http:
check_http used to ignore many parameters with check combinations - sometimes
even time thresholds(!!)
With this commit as long as a response is received *all* checks are performed
on it.
This fixes #1460312 plus a number of other issues I discovered over time.
check_http used to ignore many parameters with check combinations - sometimes
even time thresholds(!!)
With this commit as long as a response is received *all* checks are performed
on it.
This fixes #1460312 plus a number of other issues I discovered over time.
Make extract_value more generic:
Add a separator argument passed using defines.
Add a separator argument passed using defines.
Remove the commented out blocks
The die tests are implemented in the test_*3.t files
The die tests are implemented in the test_*3.t files
Fix paths in test_ini tests
Add die tests for parse_ini
Return UNKNOWN if none of the default files can be found.
It makes no sense to return unknown if an implicit section is not found, but
go on if the file is missing
It makes no sense to return unknown if an implicit section is not found, but
go on if the file is missing
Add die tests for extra-opts
Move check_ntp's extract_value to utils_base.c.
This function can also be used to parse performance data strings which
could be useful in the future.
This function can also be used to parse performance data strings which
could be useful in the future.
Accidentally included built executables - also updated .gitignore
Some platform are missing setenv
Instead of adding the gnulib module for solving tests only I split the test
in two, setting environment from Perl for the later.
Instead of adding the gnulib module for solving tests only I split the test
in two, setting environment from Perl for the later.
Fix (or try at least) for IRIX compile to tap tests
Gnulib says: "$(LTLIBINTL) when linking with libtool, $(LIBINTL) otherwise"
Everywhere else we use LTLIBINTL so I assume the tests should use it as well.
Gnulib says: "$(LTLIBINTL) when linking with libtool, $(LIBINTL) otherwise"
Everywhere else we use LTLIBINTL so I assume the tests should use it as well.
Trick for check_users test without users logged-in
Build extra-opts in tinderbox so they can be tested
Re-apply "Cleanup; fix the way libraries are linked; Make-It-Simple(tm)"
This reverts commit 7449b16b1781c645751e2c05b93893e93630fb2a.
This reverts commit 7449b16b1781c645751e2c05b93893e93630fb2a.
Trying out new patch for IRIX 6.5 from Bruno Haible
See: http://article.gmane.org/gmane.comp.lib.gnulib.bugs/16341
See: http://article.gmane.org/gmane.comp.lib.gnulib.bugs/16341
Revert "Trying out a patch for IRIX 11"
This reverts commit d190c5d25865a7414eea4fe6409a94842fba220e.
This reverts commit d190c5d25865a7414eea4fe6409a94842fba220e.
Trying out a patch for IRIX 11
See: http://lists.gnu.org/archive/html/bug-gnulib/2009-01/msg00098.html
See: http://lists.gnu.org/archive/html/bug-gnulib/2009-01/msg00098.html
Removing AC_FUNC_ALLOCA from configure.in (added in the gnulib sync commit)
I added AC_FUNC_ALLOCA because of a mistake on my part (which has been fixed
before the final commit). Removing it now.
I added AC_FUNC_ALLOCA because of a mistake on my part (which has been fixed
before the final commit). Removing it now.
Typo
Sync with gnulib
Clean up last commit - remove the __STDC_VERSION__ check in tap.h
HP-UX refuses to compile tap with the check for __STDC_VERSION__ >= 199901L
even though it can.
HP-UX refuses to compile tap with the check for __STDC_VERSION__ >= 199901L
even though it can.
Trying to see if HP-UX will compile without the __STDC_VERSION__ check in tap.h
Make /lib before /lib/tests to satisfy dependencies (building tests fails on IRIX)
Fix test from last commit
basename check in test_utils.c was accidentaly added again.
In 538b97c I reverted a serie of libtap patch by Ton Voon; it included
the removal of the useless basename check.
Relevant portion of the changelog from which it was removed (f31cd40):
Removed basename test as we should use base_name from GNUlib which is
tested separately
In 538b97c I reverted a serie of libtap patch by Ton Voon; it included
the removal of the useless basename check.
Relevant portion of the changelog from which it was removed (f31cd40):
Removed basename test as we should use base_name from GNUlib which is
tested separately
Revert "Cleanup; fix the way libraries are linked; Make-It-Simple(tm)"
Trying to see if this has any relation to the tinderbox failure. I doubt it
and will revert back as soon as it's tested.
This reverts commit a97e5fbda21a05574b0e3c5a5118402ebfbbd13c.
Trying to see if this has any relation to the tinderbox failure. I doubt it
and will revert back as soon as it's tested.
This reverts commit a97e5fbda21a05574b0e3c5a5118402ebfbbd13c.
Try to make IRIX happy (this fixes a similar warning in Solaris)