diff options
| author | Pali Rohár <pali@kernel.org> | 2022-02-18 12:24:13 +0100 |
|---|---|---|
| committer | Stefan Roese <sr@denx.de> | 2022-03-04 08:38:05 +0100 |
| commit | 7938b3be7cedcfe54e891c86e4297b0dccde0f9f (patch) | |
| tree | 1781128ca976857a8cee4298d964074cf8ca0513 /tools/kwboot.c | |
| parent | d274f922467ea20fdb517d68737a8af431a03fce (diff) | |
| download | u-boot-socfpga-7938b3be7cedcfe54e891c86e4297b0dccde0f9f.tar.gz | |
tools: kwboot: Fix quitting terminal
Sometimes kwboot after quitting terminal prints error message:
terminal: Bad address
This is caused by trying to call write() syscall with count of (size_t)-1
bytes.
When quit sequence is split into more read() calls then number of input
bytes (nin) at the end of cycle can underflow and be negative. Fix it.
Fixes: de7514046ea5 ("tools: kwboot: Fix detection of quit esc sequence")
Signed-off-by: Pali Rohár <pali@kernel.org>
Reviewed-by: Stefan Roese <sr@denx.de>
Diffstat (limited to 'tools/kwboot.c')
| -rw-r--r-- | tools/kwboot.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/kwboot.c b/tools/kwboot.c index 68c0ef1f1b..2d2d545d82 100644 --- a/tools/kwboot.c +++ b/tools/kwboot.c @@ -1197,7 +1197,7 @@ kwboot_term_pipe(int in, int out, const char *quit, int *s) if (buf[i] == quit[*s]) { (*s)++; if (!quit[*s]) { - nin = i - *s; + nin = (i > *s) ? (i - *s) : 0; break; } } else { @@ -1208,7 +1208,7 @@ kwboot_term_pipe(int in, int out, const char *quit, int *s) } if (i == nin) - nin -= *s; + nin -= (nin > *s) ? *s : nin; } if (kwboot_write(out, buf, nin) < 0) |