Module: monitoring-plugins
Branch: check_snmp_fixes
Commit: 8d44bd8e6902dbb49e98c12bb6bf3d9e4c4b413f
Author: Andreas Baumann <[email protected]>
Date: Fri Jan 23 08:57:19 2026 +0100
URL:
https://www.monitoring-plugins.org/repositories/monitoring-plugins/commit/?id=8d44bd8e
added CHECK_EOF to work around warnings about EOF and -1 being the same
---
plugins/check_cluster.c | 2 +-
plugins/check_curl.c | 2 +-
plugins/check_dig.c | 2 +-
plugins/check_disk.c | 2 +-
plugins/check_fping.c | 2 +-
plugins/check_game.c | 2 +-
plugins/check_hpjd.c | 2 +-
plugins/check_ide_smart.c | 2 +-
plugins/check_ldap.c | 2 +-
plugins/check_load.c | 2 +-
plugins/check_mysql.c | 2 +-
plugins/check_nagios.c | 2 +-
plugins/check_ping.c | 2 +-
plugins/check_procs.c | 2 +-
plugins/check_radius.c | 2 +-
plugins/check_snmp.c | 4 ++--
plugins/check_tcp.c | 2 +-
plugins/check_time.c | 2 +-
plugins/common.h | 7 +++++++
19 files changed, 26 insertions(+), 19 deletions(-)
diff --git a/plugins/check_cluster.c b/plugins/check_cluster.c
index 1cbdcd60..92c3827a 100644
--- a/plugins/check_cluster.c
+++ b/plugins/check_cluster.c
@@ -175,7 +175,7 @@ check_cluster_config_wrapper process_arguments(int argc,
char **argv) {
while (true) {
int option_index = getopt_long(argc, argv, "hHsvVw:c:d:l:",
longopts, &option);
- if (option_index == -1 || option_index == EOF || option_index
== 1) {
+ if (CHECK_EOF(option_index) || option_index == 1) {
break;
}
diff --git a/plugins/check_curl.c b/plugins/check_curl.c
index 1dec8a2a..0bc5f9b0 100644
--- a/plugins/check_curl.c
+++ b/plugins/check_curl.c
@@ -946,7 +946,7 @@ check_curl_config_wrapper process_arguments(int argc, char
**argv) {
int option_index = getopt_long(
argc, argv,
"Vvh46t:c:w:A:k:H:P:j:T:I:a:b:d:e:p:s:R:r:u:f:C:J:K:DnlLS::m:M:NEB",
longopts, &option);
- if (option_index == -1 || option_index == EOF || option_index
== 1) {
+ if (CHECK_EOF(option_index) || option_index == 1) {
break;
}
diff --git a/plugins/check_dig.c b/plugins/check_dig.c
index 9ea19e6a..9ec8028a 100644
--- a/plugins/check_dig.c
+++ b/plugins/check_dig.c
@@ -284,7 +284,7 @@ check_dig_config_wrapper process_arguments(int argc, char
**argv) {
int option_index =
getopt_long(argc, argv, "hVvt:l:H:w:c:T:p:a:A:E:X:46",
longopts, &option);
- if (option_index == -1 || option_index == EOF) {
+ if (CHECK_EOF(option_index)) {
break;
}
diff --git a/plugins/check_disk.c b/plugins/check_disk.c
index e1a2baff..4ec1b211 100644
--- a/plugins/check_disk.c
+++ b/plugins/check_disk.c
@@ -448,7 +448,7 @@ check_disk_config_wrapper process_arguments(int argc, char
**argv) {
int option_index = getopt_long(
argc, argv,
"+?VqhvefCt:c:w:K:W:u:p:x:X:N:mklLPg:R:r:i:I:MEAn", longopts, &option);
- if (option_index == -1 || option_index == EOF) {
+ if (CHECK_EOF(option_index)) {
break;
}
diff --git a/plugins/check_fping.c b/plugins/check_fping.c
index 6160c2cb..86ef64a4 100644
--- a/plugins/check_fping.c
+++ b/plugins/check_fping.c
@@ -361,7 +361,7 @@ check_fping_config_wrapper process_arguments(int argc, char
**argv) {
int option_index =
getopt_long(argc, argv,
"+hVvaH:S:c:w:b:n:T:i:I:M:R:46", longopts, &option);
- if (option_index == -1 || option_index == EOF || option_index
== 1) {
+ if (CHECK_EOF(option_index) || option_index == 1) {
break;
}
diff --git a/plugins/check_game.c b/plugins/check_game.c
index 974a7253..48ec6883 100644
--- a/plugins/check_game.c
+++ b/plugins/check_game.c
@@ -186,7 +186,7 @@ check_game_config_wrapper process_arguments(int argc, char
**argv) {
while (true) {
int option_index = getopt_long(argc, argv, "hVvt:H:P:G:g:p:m:",
long_opts, &opt_index);
- if (option_index == -1 || option_index == EOF) {
+ if (CHECK_EOF(option_index)) {
break;
}
diff --git a/plugins/check_hpjd.c b/plugins/check_hpjd.c
index 9907abc5..883f1df0 100644
--- a/plugins/check_hpjd.c
+++ b/plugins/check_hpjd.c
@@ -299,7 +299,7 @@ check_hpjd_config_wrapper process_arguments(int argc, char
**argv) {
while (true) {
int option_index = getopt_long(argc, argv, "+hVH:C:p:D",
longopts, &option);
- if (option_index == -1 || option_index == EOF || option_index
== 1) {
+ if (CHECK_EOF(option_index) || option_index == 1) {
break;
}
diff --git a/plugins/check_ide_smart.c b/plugins/check_ide_smart.c
index c1325cf9..43731039 100644
--- a/plugins/check_ide_smart.c
+++ b/plugins/check_ide_smart.c
@@ -169,7 +169,7 @@ static check_ide_smart_config_wrapper process_arguments(int
argc, char **argv) {
int longindex = 0;
int option_index = getopt_long(argc, argv, "+d:iq10nhVv",
longopts, &longindex);
- if (option_index == -1 || option_index == EOF || option_index
== 1) {
+ if (CHECK_EOF(option_index) || option_index == 1) {
break;
}
diff --git a/plugins/check_ldap.c b/plugins/check_ldap.c
index 1b2e2826..300a0dc1 100644
--- a/plugins/check_ldap.c
+++ b/plugins/check_ldap.c
@@ -362,7 +362,7 @@ check_ldap_config_wrapper process_arguments(int argc, char
**argv) {
int option_index =
getopt_long(argc, argv,
"hvV234TS6t:c:w:H:b:p:a:D:P:C:W:", longopts, &option);
- if (option_index == -1 || option_index == EOF) {
+ if (CHECK_EOF(option_index)) {
break;
}
diff --git a/plugins/check_load.c b/plugins/check_load.c
index 644cd604..60fa646f 100644
--- a/plugins/check_load.c
+++ b/plugins/check_load.c
@@ -295,7 +295,7 @@ static check_load_config_wrapper process_arguments(int
argc, char **argv) {
int option = 0;
int option_index = getopt_long(argc, argv, "Vhrc:w:n:",
longopts, &option);
- if (option_index == -1 || option_index == EOF) {
+ if (CHECK_EOF(option_index)) {
break;
}
diff --git a/plugins/check_mysql.c b/plugins/check_mysql.c
index 26730d4c..20b4ac3f 100644
--- a/plugins/check_mysql.c
+++ b/plugins/check_mysql.c
@@ -504,7 +504,7 @@ check_mysql_config_wrapper process_arguments(int argc, char
**argv) {
int option_index =
getopt_long(argc, argv,
"hlvVnSP:p:u:d:H:s:c:w:a:k:C:D:L:f:g:", longopts, &option);
- if (option_index == -1 || option_index == EOF) {
+ if (CHECK_EOF(option_index)) {
break;
}
diff --git a/plugins/check_nagios.c b/plugins/check_nagios.c
index a46dc1ed..e2f230c9 100644
--- a/plugins/check_nagios.c
+++ b/plugins/check_nagios.c
@@ -227,7 +227,7 @@ check_nagios_config_wrapper process_arguments(int argc,
char **argv) {
while (true) {
int option_index = getopt_long(argc, argv, "+hVvF:C:e:t:",
longopts, &option);
- if (option_index == -1 || option_index == EOF || option_index
== 1) {
+ if (CHECK_EOF(option_index) || option_index == 1) {
break;
}
diff --git a/plugins/check_ping.c b/plugins/check_ping.c
index 61feb958..cb9e8f57 100644
--- a/plugins/check_ping.c
+++ b/plugins/check_ping.c
@@ -221,7 +221,7 @@ check_ping_config_wrapper process_arguments(int argc, char
**argv) {
while (true) {
int option_index = getopt_long(argc, argv, "VvhnL46t:c:w:H:p:",
longopts, &option);
- if (option_index == -1 || option_index == EOF) {
+ if (CHECK_EOF(option_index)) {
break;
}
diff --git a/plugins/check_procs.c b/plugins/check_procs.c
index ae6e9c23..01476504 100644
--- a/plugins/check_procs.c
+++ b/plugins/check_procs.c
@@ -432,7 +432,7 @@ check_procs_config_wrapper process_arguments(int argc, char
**argv) {
int option_index =
getopt_long(argc, argv,
"Vvhkt:c:w:p:s:u:C:a:z:r:m:P:T:X:", longopts, &option);
- if (option_index == -1 || option_index == EOF) {
+ if (CHECK_EOF(option_index)) {
break;
}
diff --git a/plugins/check_radius.c b/plugins/check_radius.c
index 93352bcc..6e062ff8 100644
--- a/plugins/check_radius.c
+++ b/plugins/check_radius.c
@@ -332,7 +332,7 @@ check_radius_config_wrapper process_arguments(int argc,
char **argv) {
int option = 0;
int option_index = getopt_long(argc, argv,
"+hVvH:P:F:u:p:n:N:t:r:e:", longopts, &option);
- if (option_index == -1 || option_index == EOF || option_index
== 1) {
+ if (CHECK_EOF(option_index) || option_index == 1) {
break;
}
diff --git a/plugins/check_snmp.c b/plugins/check_snmp.c
index f470d222..52090bf8 100644
--- a/plugins/check_snmp.c
+++ b/plugins/check_snmp.c
@@ -445,7 +445,7 @@ static process_arguments_wrapper process_arguments(int
argc, char **argv) {
argc, argv,
"nhvVO46t:c:w:H:C:o:e:E:d:D:s:t:R:r:l:u:p:m:P:N:L:U:a:x:A:X:M:f:z:", longopts,
&option);
- if (option_char == -1 || option_char == EOF) {
+ if (CHECK_EOF(option_char)) {
break;
}
@@ -515,7 +515,7 @@ static process_arguments_wrapper process_arguments(int
argc, char **argv) {
argc, argv,
"nhvVO46t:c:w:H:C:o:e:E:d:D:s:t:R:r:l:u:p:m:P:N:L:U:a:x:A:X:M:f:z:", longopts,
&option);
- if (option_char == -1 || option_char == EOF) {
+ if (CHECK_EOF(option_char)) {
break;
}
diff --git a/plugins/check_tcp.c b/plugins/check_tcp.c
index 09806373..ec334941 100644
--- a/plugins/check_tcp.c
+++ b/plugins/check_tcp.c
@@ -550,7 +550,7 @@ static check_tcp_config_wrapper process_arguments(int argc,
char **argv, check_t
int option_index =
getopt_long(argc, argv,
"+hVv46EAH:s:e:q:m:c:w:t:p:C:W:d:Sr:jD:M:", longopts, &option);
- if (option_index == -1 || option_index == EOF || option_index
== 1) {
+ if (CHECK_EOF(option_index) || option_index == 1) {
break;
}
diff --git a/plugins/check_time.c b/plugins/check_time.c
index 99708ad3..aec995d4 100644
--- a/plugins/check_time.c
+++ b/plugins/check_time.c
@@ -213,7 +213,7 @@ check_time_config_wrapper process_arguments(int argc, char
**argv) {
int option = 0;
option_char = getopt_long(argc, argv, "hVH:w:c:W:C:p:t:u",
longopts, &option);
- if (option_char == -1 || option_char == EOF) {
+ if (CHECK_EOF(option_char)) {
break;
}
diff --git a/plugins/common.h b/plugins/common.h
index b3053486..9d1434a3 100644
--- a/plugins/common.h
+++ b/plugins/common.h
@@ -205,4 +205,11 @@ enum {
# define __attribute__(x) /* do nothing */
#endif
+/* for checking the result of getopt_long */
+#if EOF == -1
+#define CHECK_EOF(c) ((c) == EOF)
+#else
+#define CHECK_EOF(c) ((c) == -1 || (c) == EOF)
+#endif
+
#endif /* _COMMON_H_ */
