Fix for MD5 leak bug, issue #7195 (#7197)

lrodorigo · luiss-mind · d-a-v · web-flow · commit 41d271d9721f · 2020-04-09T10:36:57.000-07:00
Co-authored-by: Luiss &lt;luiss@mind.cc&gt;
Co-authored-by: david gauchard &lt;gauchard@laas.fr&gt;
Co-authored-by: Develo &lt;deveyes@gmail.com&gt;
Co-authored-by: Earle F. Philhower, III &lt;earlephilhower@yahoo.com&gt;
diff --git a/cores/esp8266/MD5Builder.cpp b/cores/esp8266/MD5Builder.cpp
@@ -55,6 +55,7 @@ bool MD5Builder::addStream(Stream & stream, const size_t maxLen){
         // read data and check if we got something
         int numBytesRead = stream.readBytes(buf, readBytes);
         if(numBytesRead< 1) {
+            free(buf); // release the buffer
             return false;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -55,6 +55,7 @@ bool MD5Builder::addStream(Stream & stream, const size_t maxLen){`
`55`	`55`	`// read data and check if we got something`
`56`	`56`	`int numBytesRead = stream.readBytes(buf, readBytes);`
`57`	`57`	`if(numBytesRead< 1) {`
	`58`	`+ free(buf); // release the buffer`
`58`	`59`	`return false;`
`59`	`60`	`}`
`60`	`61`