Fixes for -Wsign-compare

1 files changed, 7 insertions, 8 deletions

diff --git a/plugins/check_by_ssh.c b/plugins/check_by_ssh.c
index 1f5f72d..2a23b39 100644
--- a/plugins/check_by_ssh.c
+++ b/plugins/check_by_ssh.c
@@ -68,7 +68,6 @@ main (int argc, char **argv)
         char *status_text;
         int cresult;
         int result = STATE_UNKNOWN;
-        int i;
         time_t local_time;
         FILE *fp = NULL;
         output chld_out, chld_err;
@@ -96,7 +95,7 @@ main (int argc, char **argv)
         /* run the command */
         if (verbose) {
                 printf ("Command: %s\n", commargv[0]);
-                for (i=1; i<commargc; i++)
+                for (int i = 1; i < commargc; i++)
                         printf ("Argument %i: %s\n", i, commargv[i]);
         }
 
@@ -110,9 +109,9 @@ main (int argc, char **argv)
         }
 
         if (verbose) {
-                for(i = 0; i < chld_out.lines; i++)
+                for(size_t i = 0; i < chld_out.lines; i++)
                         printf("stdout: %s\n", chld_out.line[i]);
-                for(i = 0; i < chld_err.lines; i++)
+                for(size_t i = 0; i < chld_err.lines; i++)
                         printf("stderr: %s\n", chld_err.line[i]);
         }
 
@@ -122,7 +121,7 @@ main (int argc, char **argv)
                 skip_stderr = chld_err.lines;
 
         /* UNKNOWN or worse if (non-skipped) output found on stderr */
-        if(chld_err.lines > skip_stderr) {
+        if(chld_err.lines > (size_t)skip_stderr) {
                 printf (_("Remote command execution failed: %s\n"),
                         chld_err.line[skip_stderr]);
                 if ( warn_on_stderr ) 
@@ -134,8 +133,8 @@ main (int argc, char **argv)
         /* this is simple if we're not supposed to be passive.
          * Wrap up quickly and keep the tricks below */
         if(!passive) {
-                if (chld_out.lines > skip_stdout)
-                        for (i = skip_stdout; i < chld_out.lines; i++)
+                if (chld_out.lines > (size_t)skip_stdout)
+                        for (size_t i = skip_stdout; i < chld_out.lines; i++)
                                 puts (chld_out.line[i]);
                 else
                         printf (_("%s - check_by_ssh: Remote command '%s' returned status %d\n"),
@@ -156,7 +155,7 @@ main (int argc, char **argv)
 
         local_time = time (NULL);
         commands = 0;
-        for(i = skip_stdout; i < chld_out.lines; i++) {
+        for(size_t i = skip_stdout; i < chld_out.lines; i++) {
                 status_text = chld_out.line[i++];
                 if (i == chld_out.lines || strstr (chld_out.line[i], "STATUS CODE: ") == NULL)
                         die (STATE_UNKNOWN, _("%s: Error parsing output\n"), progname);