* radsecproxy.c (extractattr): Bug! When s is changed, update len. Th…

…is might actually work now!
mariux64 · May 25, 2010 · 684ade5 · 684ade5
1 parent 2e9d382
commit 684ade5
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/radsecproxy.c b/radsecproxy.c
@@ -2335,19 +2335,20 @@ struct tlv *extractattr(char *nameval) {
     struct tlv *a;
 
     s = strchr(nameval, ':');
-    name = atoi(nameval);
     if (!s)
 	return NULL;
-    len = strlen(s + 1);
-    if (len > 253)
-	return NULL;
+    name = atoi(nameval);
 
+    /* Two ':' means that we have vendor:name:val.  */
     s2 = strchr(s + 1, ':');
-    if (s2) {		 /* Two ':' means we have vendor:name:val.  */
+    if (s2) {
 	vendor = name;
 	name = atoi(s + 1);
 	s = s2;
     }
+    len = strlen(s + 1);
+    if (len > 253)
+	return NULL;
 
     if (name < 1 || name > 255)
 	return NULL;