diff options
| author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2017-02-24 10:29:05 +0100 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2017-02-28 09:05:47 +0100 |
| commit | c80f6aa9161e10259127b3b9abe7717c702daf39 (patch) | |
| tree | fa83592d69bfdcf41bf15cc932636e2f4f19ed3d /cpukit/libcsupport | |
| parent | termios: Change receive callback invocation (diff) | |
| download | rtems-c80f6aa9161e10259127b3b9abe7717c702daf39.tar.bz2 | |
termios: Fix infinite loop in receive path
In canonical mode, the raw input buffer or the canonical buffer may
overflow without an end of line. Avoid an infinite loop in this case.
Close #2915.
Diffstat (limited to 'cpukit/libcsupport')
| -rw-r--r-- | cpukit/libcsupport/src/termios.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/cpukit/libcsupport/src/termios.c b/cpukit/libcsupport/src/termios.c index 62964e4747..f5bf493966 100644 --- a/cpukit/libcsupport/src/termios.c +++ b/cpukit/libcsupport/src/termios.c @@ -1504,13 +1504,17 @@ fillBufferQueue (struct rtems_termios_tty *tty) /* * Wait for characters */ - if ( wait ) { - rtems_status_code sc; + if (wait) { + if (tty->ccount < CBUFSIZE - 1) { + rtems_status_code sc; - sc = rtems_semaphore_obtain( - tty->rawInBuf.Semaphore, tty->rawInBufSemaphoreOptions, timeout); - if (sc != RTEMS_SUCCESSFUL) + sc = rtems_semaphore_obtain( + tty->rawInBuf.Semaphore, tty->rawInBufSemaphoreOptions, timeout); + if (sc != RTEMS_SUCCESSFUL) + break; + } else { break; + } } } } |