fix for APs > maxResults creating invalid json resulting in empty AP list

andrbmgi · andrbmgi · commit 0b66f101b05f · 2017-02-18T07:59:28.000+01:00
diff --git a/esp8266_deauther/APScan.cpp b/esp8266_deauther/APScan.cpp
@@ -70,7 +70,7 @@ String APScan::getResults(){
     json += "\"vendor\": \""+getAPVendor(i)+"\",";
     json += "\"selected\": "+getAPSelected(i);
     json += "}";
-    if(i!=results-1) json += ",";
+    if((i!=results-1) && (i!=maxResults-1)) json += ",";
   }
   json += "] }";
   return json;

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ String APScan::getResults(){`
`70`	`70`	`json += "\"vendor\": \""+getAPVendor(i)+"\",";`
`71`	`71`	`json += "\"selected\": "+getAPSelected(i);`
`72`	`72`	`json += "}";`
`73`		`- if(i!=results-1) json += ",";`
	`73`	`+ if((i!=results-1) && (i!=maxResults-1)) json += ",";`
`74`	`74`	`}`
`75`	`75`	`json += "] }";`
`76`	`76`	`return json;`