Skip to content

Commit bf39067

Browse files
committed
avoid buffer overflow in debug
1 parent 29288a0 commit bf39067

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

Diff for: cores/esp32/esp32-hal-uart.c

+10-3
Original file line numberDiff line numberDiff line change
@@ -828,9 +828,16 @@ uartDetectBaudrate(uart_t *uart)
828828
}
829829

830830
log_v("Rounded\t%d\tto\t%d", baudrate, default_rates[i]);
831-
log_v("From Previous\t( %d )\t:\t( %d )", default_rates[i-1], baudrate - default_rates[i-1] ); //THIS WILL OVERFLOW, only for testing now
832-
log_v("From Current\t( %d )\t:\t( %d )", default_rates[i], default_rates[i] - baudrate); //THIS WILL OVERFLOW, only for testing now
833-
log_v("From Next\t( %d )\t:\t( %d )", default_rates[i+1], default_rates[i+1] - baudrate); //THIS WILL OVERFLOW, only for testing now
831+
832+
if(i>0){
833+
log_v("From Previous\t( %d )\t:\t( %d )", default_rates[i-1], baudrate - default_rates[i-1] );
834+
}
835+
836+
log_v("From Current\t( %d )\t:\t( %d )", default_rates[i], default_rates[i] - baudrate);
837+
838+
if(i < ( sizeof(default_rates) / sizeof(default_rates[0]) - 2 ) ){
839+
log_v("From Next\t( %d )\t:\t( %d )", default_rates[i+1], default_rates[i+1] - baudrate);
840+
}
834841

835842
return default_rates[i];
836843
#else

0 commit comments

Comments
 (0)