De-reference ptr after checking it.
authorAlan T. DeKok <aland@freeradius.org>
Mon, 1 Dec 2008 09:33:41 +0000 (10:33 +0100)
committerAlan T. DeKok <aland@freeradius.org>
Mon, 1 Dec 2008 16:47:24 +0000 (17:47 +0100)
This fixes Coverity #79

src/main/evaluate.c

index 041aa8c..88aac8b 100644 (file)
@@ -526,7 +526,7 @@ int radius_evaluate_condition(REQUEST *request, int modreturn, int depth,
        int result = TRUE;
        int invert = FALSE;
        int evaluate_next_condition = evaluate_it;
-       const char *p = *ptr;
+       const char *p;
        const char *q, *start;
        FR_TOKEN token, lt, rt;
        char left[1024], right[1024], comp[4];
@@ -539,6 +539,7 @@ int radius_evaluate_condition(REQUEST *request, int modreturn, int depth,
                return FALSE;
        }
 
+       p =  *ptr;
        while (*p) {
                while ((*p == ' ') || (*p == '\t')) p++;