Properly release resources before Reset

stickbreaker · web-flow · commit 4bd1ba0a8dcd · 2017-11-25T10:31:54.000-07:00
Had not included cleanup for dynamic memory/ Interrupt in Reset() operation. Would cause a memory leak, and Possibly a total failure if Reset() was call. I have not verified how FreeRTOS would handle trying to assign a ISR to an interrupt that is already assigned an ISR?
diff --git a/libraries/Wire/src/Wire.cpp b/libraries/Wire/src/Wire.cpp
@@ -484,6 +484,8 @@ void TwoWire::flush(void)
 
 void TwoWire::reset(void)
 {
+    i2cFreeQueue(i2c); // release malloc memory
+    i2cReleaseISR(i2c); // remove ISR from Interrupt chain
     i2cReset( i2c );
     i2c = NULL;
     begin( sda, scl );

Original file line number	Diff line number	Diff line change
`@@ -484,6 +484,8 @@ void TwoWire::flush(void)`
`484`	`484`
`485`	`485`	`void TwoWire::reset(void)`
`486`	`486`	`{`
	`487`	`+ i2cFreeQueue(i2c); // release malloc memory`
	`488`	`+ i2cReleaseISR(i2c); // remove ISR from Interrupt chain`
`487`	`489`	`i2cReset( i2c );`
`488`	`490`	`i2c = NULL;`
`489`	`491`	`begin( sda, scl );`