Merge pull request #6 from stickbreaker/stickbreaker-patch-1

stickbreaker · web-flow · commit d26310bdbaba · 2017-11-25T10:41:45.000-07:00
Memory Leak
diff --git a/cores/esp32/esp32-hal-i2c.c b/cores/esp32/esp32-hal-i2c.c
@@ -1130,7 +1130,11 @@ if(i2c->intr_handle){
 //  log_e("released ISR=%d",error);
   i2c->intr_handle=NULL;
   }
-return I2C_ERROR_OK;
+if(i2c->i2c_event){
+  xEventGroupDelete(i2c->i2c_event);
+  i2c->i2c_event = NULL;
+  }
+return i2cFreeQueue(i2c); //release dynamic memory
 }
 
 /* todo

Original file line number	Diff line number	Diff line change
`@@ -1130,7 +1130,11 @@ if(i2c->intr_handle){`
`1130`	`1130`	`// log_e("released ISR=%d",error);`
`1131`	`1131`	`i2c->intr_handle=NULL;`
`1132`	`1132`	`}`
`1133`		`-return I2C_ERROR_OK;`
	`1133`	`+if(i2c->i2c_event){`
	`1134`	`+ xEventGroupDelete(i2c->i2c_event);`
	`1135`	`+ i2c->i2c_event = NULL;`
	`1136`	`+ }`
	`1137`	`+return i2cFreeQueue(i2c); //release dynamic memory`
`1134`	`1138`	`}`
`1135`	`1139`
`1136`	`1140`	`/* todo`