I2CSlave: restore critical section with temp buffer

pennam · pennam · commit 6ca44223cf2d · 2025-04-29T16:31:08.000+02:00
diff --git a/libraries/Wire/Wire.cpp b/libraries/Wire/Wire.cpp
@@ -158,8 +158,12 @@ void arduino::MbedI2C::receiveThd() {
 					onRequestCb();
 				}
 				if (usedTxBuffer != 0) {
-					slave->write((const char *) txBuffer, usedTxBuffer);
+					core_util_critical_section_enter();
+					uint8_t tempTxBuffer[usedTxBuffer];
+					memcpy(tempTxBuffer, txBuffer, sizeof(tempTxBuffer));
 					usedTxBuffer = 0;
+					core_util_critical_section_exit();
+					slave->write((const char *) tempTxBuffer, sizeof(tempTxBuffer));
 				}
 				//slave->stop();
 				break;

Original file line number	Diff line number	Diff line change
`@@ -158,8 +158,12 @@ void arduino::MbedI2C::receiveThd() {`
`158`	`158`	`onRequestCb();`
`159`	`159`	`}`
`160`	`160`	`if (usedTxBuffer != 0) {`
`161`		`- slave->write((const char *) txBuffer, usedTxBuffer);`
	`161`	`+ core_util_critical_section_enter();`
	`162`	`+ uint8_t tempTxBuffer[usedTxBuffer];`
	`163`	`+ memcpy(tempTxBuffer, txBuffer, sizeof(tempTxBuffer));`
`162`	`164`	`usedTxBuffer = 0;`
	`165`	`+ core_util_critical_section_exit();`
	`166`	`+ slave->write((const char *) tempTxBuffer, sizeof(tempTxBuffer));`
`163`	`167`	`}`
`164`	`168`	`//slave->stop();`
`165`	`169`	`break;`