tyler.durden
/
centreon-plugins
mirror of https://github.com/centreon/centreon-plugins.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix #4250

This commit is contained in:
Quentin Garnier 2013-02-27 15:50:52 +01:00
parent 0024aafa11
commit 3725c9abe0
1 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/check_centreon_snmp_traffic
View File

@ -71,6 +71,9 @@ my %OPTION = (
"disable-warn-state" => undef "disable-warn-state" => undef
); );
# Catch UNKNOWN From GetOptions (like -i without value)
$SIG{'__WARN__'} = sub { print $_[0]; exit($ERRORS{'UNKNOWN'}); };
Getopt::Long::Configure('bundling'); Getopt::Long::Configure('bundling');
GetOptions GetOptions
( (
@ -123,13 +126,13 @@ my ($session_params) = Centreon::SNMP::Utils::check_snmp_options($ERRORS{'UNKNOW
###### ######
### Others ### Others
## ##
if ($opt_n && !$opt_i) { if (defined($opt_n) && !defined($opt_i)) {
print "Option -n (--name) need option -i (--interface)\n"; print "Option -n (--name) need option -i (--interface)\n";
exit $ERRORS{'UNKNOWN'}; exit $ERRORS{'UNKNOWN'};
} }
if (!$opt_i) { if (!defined($opt_i)) {
$opt_i = 2; $opt_i = 1;
} }
if (!defined($opt_a)) { if (!defined($opt_a)) {
@ -146,9 +149,9 @@ if ($opt_w && $opt_w =~ /[0-9]+/) {
} }
my $interface; my $interface;
if ($opt_i =~ /^([0-9]+)$/ && !$opt_n){ if ($opt_i =~ /^([0-9]+)$/ && !defined($opt_n)) {
$interface = $1; $interface = $1;
} elsif (!$opt_n) { } elsif (!defined($opt_n)) {
print "Unknown -i number expected... or it doesn't exist, try another interface - number\n"; print "Unknown -i number expected... or it doesn't exist, try another interface - number\n";
exit $ERRORS{'UNKNOWN'}; exit $ERRORS{'UNKNOWN'};
} }