processes plugin: check boundary first

[src/processes.c:1042]: (style) Array index 'name_start_pos' is used before limits check.

diff --git a/src/processes.c b/src/processes.c
index 5a795b5aefd915a5e4c1faeb072c63ebc3e2dfa0..03b44b58d740b0359a8e9dfb876cd6095117dcf1 100644 (file)
--- a/src/processes.c
+++ b/src/processes.c
@@ -1038,13 +1038,11 @@ static int ps_read_process (long pid, procstat_t *ps, char *state)
         * strchr(3) and strrchr(3) to avoid pointer arithmetic which would
         * otherwise be required to determine name_len. */
        name_start_pos = 0;
-       while ((buffer[name_start_pos] != '(')
-                       && (name_start_pos < buffer_len))
+       while (name_start_pos < buffer_len && buffer[name_start_pos] != '(')
                name_start_pos++;
 
        name_end_pos = buffer_len;
-       while ((buffer[name_end_pos] != ')')
-                       && (name_end_pos > 0))
+       while (name_end_pos > 0 && buffer[name_end_pos] != ')')
                name_end_pos--;
 
        /* Either '(' or ')' is not found or they are in the wrong order.