Fixed memory on Wire instance

gbr1 · fpistm · commit e88e08f27083 · 2023-11-21T16:46:36.000+01:00
The constructor doesn't cleanup memory for a wire instance.
Issues could happen, such as wrong initialization during boot

Signed-off-by: Giovanni Bruno &lt;giovannididio.bruno@gmail.com&gt;
diff --git a/libraries/Wire/src/Wire.cpp b/libraries/Wire/src/Wire.cpp
@@ -35,12 +35,14 @@ static const uint8_t MASTER_ADDRESS = 0x01;
 
 TwoWire::TwoWire()
 {
+  memset((void *)&_i2c, 0, sizeof(_i2c));
   _i2c.sda = digitalPinToPinName(SDA);
   _i2c.scl = digitalPinToPinName(SCL);
 }
 
 TwoWire::TwoWire(uint32_t sda, uint32_t scl)
 {
+  memset((void *)&_i2c, 0, sizeof(_i2c));
   _i2c.sda = digitalPinToPinName(sda);
   _i2c.scl = digitalPinToPinName(scl);
 }

Original file line number	Diff line number	Diff line change
`@@ -35,12 +35,14 @@ static const uint8_t MASTER_ADDRESS = 0x01;`
`35`	`35`
`36`	`36`	`TwoWire::TwoWire()`
`37`	`37`	`{`
	`38`	`+ memset((void *)&_i2c, 0, sizeof(_i2c));`
`38`	`39`	`_i2c.sda = digitalPinToPinName(SDA);`
`39`	`40`	`_i2c.scl = digitalPinToPinName(SCL);`
`40`	`41`	`}`
`41`	`42`
`42`	`43`	`TwoWire::TwoWire(uint32_t sda, uint32_t scl)`
`43`	`44`	`{`
	`45`	`+ memset((void *)&_i2c, 0, sizeof(_i2c));`
`44`	`46`	`_i2c.sda = digitalPinToPinName(sda);`
`45`	`47`	`_i2c.scl = digitalPinToPinName(scl);`
`46`	`48`	`}`