Merge pull request #22 from rdmrocha/master

abachman · web-flow · commit eb4d3cc6ebd1 · 2018-02-22T15:21:41.000-05:00
fixing memory leak on csv parser
diff --git a/src/AdafruitIO_Data.cpp b/src/AdafruitIO_Data.cpp
@@ -650,6 +650,13 @@ bool AdafruitIO_Data::_parseCSV()
       field_count--;
     }
 
+    // cleanup to avoid leaks
+    int i = 0;
+    while (fields[i] != NULL){
+      free(fields[i++]);
+    }
+    free(fields);
+
     return field_count == 0;
   }
   else

Original file line number	Diff line number	Diff line change
`@@ -650,6 +650,13 @@ bool AdafruitIO_Data::_parseCSV()`
`650`	`650`	`field_count--;`
`651`	`651`	`}`
`652`	`652`
	`653`	`+ // cleanup to avoid leaks`
	`654`	`+ int i = 0;`
	`655`	`+ while (fields[i] != NULL){`
	`656`	`+ free(fields[i++]);`
	`657`	`+ }`
	`658`	`+ free(fields);`
	`659`	`+`
`653`	`660`	`return field_count == 0;`
`654`	`661`	`}`
`655`	`662`	`else`