mirror of
https://github.com/espressif/esp-idf.git
synced 2026-09-22 13:01:16 +03:00
Merge branch 'contrib/github_pr_18641_v6.0' into 'release/v6.0'
fix(protocomm): prevent out-of-bounds write in console line buffer (GitHub PR) (v6.0) See merge request espressif/esp-idf!49343
This commit is contained in:
@@ -94,7 +94,7 @@ static void protocomm_console_task(void *arg)
|
||||
}
|
||||
}
|
||||
if (event.type == UART_DATA) {
|
||||
while (uart_read_bytes(uart_num, (uint8_t *) &linebuf[i], 1, 0) && (i < LINE_BUF_SIZE)) {
|
||||
while ((i < LINE_BUF_SIZE - 1) && uart_read_bytes(uart_num, (uint8_t *) &linebuf[i], 1, 0)) {
|
||||
if (linebuf[i] == '\r') {
|
||||
uart_write_bytes(uart_num, "\r\n", 2);
|
||||
} else {
|
||||
@@ -106,7 +106,7 @@ static void protocomm_console_task(void *arg)
|
||||
if ((i > 0) && (linebuf[i-1] == '\r')) {
|
||||
break;
|
||||
}
|
||||
} while (i < LINE_BUF_SIZE);
|
||||
} while (i < LINE_BUF_SIZE - 1);
|
||||
if (stopped()) {
|
||||
break;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user