Code

Fix bogus 'eq ( || )' constructions in check_ifoperstatus.pl.
authorGavin Carr <gonzai@users.sourceforge.net>
Fri, 16 Mar 2007 12:11:46 +0000 (12:11 +0000)
committerGavin Carr <gonzai@users.sourceforge.net>
Fri, 16 Mar 2007 12:11:46 +0000 (12:11 +0000)
git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/trunk@1642 f882894a-f735-0410-b71e-b25c423dba1c

plugins-scripts/check_ifoperstatus.pl

index 59356bd7a21e65b4f6b34cce9184b455c6560784..62891ff76ea80a1421904ccb9b119dbaea038a06 100644 (file)
@@ -384,15 +384,15 @@ sub process_arguments() {
                if (defined $seclevel  && defined $secname) {
                
                        # Must define a security level even though defualt is noAuthNoPriv
-                       unless ($seclevel eq ('noAuthNoPriv' || 'authNoPriv' || 'authPriv' ) ) {
+                       unless ( grep /^$seclevel$/, qw(noAuthNoPriv authNoPriv authPriv) ) {
                                usage();
                                exit $ERRORS{"UNKNOWN"};
                        }
                        
                        # Authentication wanted
-                       if ($seclevel eq ('authNoPriv' || 'authPriv') ) {
+                       if ( $seclevel eq 'authNoPriv' || $seclevel eq 'authPriv' ) {
                
-                               unless ($authproto eq ('MD5' || 'SHA1') ) {
+                               unless ( $authproto eq 'MD5' || $authproto eq 'SHA1' ) {
                                        usage();
                                        exit $ERRORS{"UNKNOWN"};
                                }