Test GPIO number in attachInterrupt()

SuGlider · SuGlider · commit c8b1c1573169 · 2023-09-04T11:03:12.000-03:00
diff --git a/cores/esp32/esp32-hal-gpio.c b/cores/esp32/esp32-hal-gpio.c
@@ -167,6 +167,9 @@ extern void __attachInterruptFunctionalArg(uint8_t pin, voidFuncPtrArg userFunc,
 {
     static bool interrupt_initialized = false;
 
+    // makes sure that pin -1 (255) will never work -- this follows Arduino standard
+    if (pin >= NUM_DIGITAL_PINS) return;
+
     if(!interrupt_initialized) {
     	esp_err_t err = gpio_install_isr_service((int)ARDUINO_ISR_FLAG);
     	interrupt_initialized = (err == ESP_OK) || (err == ESP_ERR_INVALID_STATE);

Original file line number	Diff line number	Diff line change
`@@ -167,6 +167,9 @@ extern void __attachInterruptFunctionalArg(uint8_t pin, voidFuncPtrArg userFunc,`
`167`	`167`	`{`
`168`	`168`	`static bool interrupt_initialized = false;`
`169`	`169`
	`170`	`+ // makes sure that pin -1 (255) will never work -- this follows Arduino standard`
	`171`	`+ if (pin >= NUM_DIGITAL_PINS) return;`
	`172`	`+`
`170`	`173`	`if(!interrupt_initialized) {`
`171`	`174`	`esp_err_t err = gpio_install_isr_service((int)ARDUINO_ISR_FLAG);`
`172`	`175`	`interrupt_initialized = (err == ESP_OK) \|\| (err == ESP_ERR_INVALID_STATE);`