wyj/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2024-09-17 20:58:20 -04:00
Code Issues Packages Projects Releases Wiki Activity

fix(coverity/509227): tui stop_driver_ti() avoid underflow

Browse Source 
Problem:
write() can return -1 but is cast to unsigned type causing coverity
to detect possible overflowed integer

Solution:
Perform check to ensure all negative values are captured rather than
just -1 before casting to unsigned type
This commit is contained in:
Devon Gardner 2024-09-11 00:38:17 -04:00
parent 5dad85b977
commit 9054993842
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/nvim/tui/termkey/driver-ti.c
View File

@ -449,10 +449,11 @@ int stop_driver_ti(TermKey *tk, void *info)
// Can't call putp or tputs because they suck and don't give us fd control
len = strlen(stop_string);
while (len) {
size_t written = (size_t)write(tk->fd, stop_string, (unsigned)len);
if (written == (size_t)-1) {
ssize_t result = write(tk->fd, stop_string, (unsigned)len);
if (result < 0) {
return 0;
}
size_t written = (size_t)result;
stop_string += written;
len -= written;
}