diff options
author | Bram Moolenaar <Bram@vim.org> | 2023-01-18 12:45:30 +0000 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2023-01-18 12:45:30 +0000 |
commit | 64f1c4152e939ca41d57d5b4376be522736ba22a (patch) | |
tree | a6743c1f9060d572f7320c8a363f923701bbc300 | |
parent | 0ef9a5c09482649cf0cc6768ed6fc640b4ed2a0a (diff) | |
download | vim-git-64f1c4152e939ca41d57d5b4376be522736ba22a.tar.gz |
patch 9.0.1216: Coverity warns for ignoring return valuev9.0.1216
Problem: Coverity warns for ignoring return value.
Solution: Break out of loop if function fails.
-rw-r--r-- | src/mbyte.c | 13 | ||||
-rw-r--r-- | src/version.c | 2 |
2 files changed, 10 insertions, 5 deletions
diff --git a/src/mbyte.c b/src/mbyte.c index 4f6c8502a..6d7137ed5 100644 --- a/src/mbyte.c +++ b/src/mbyte.c @@ -5756,11 +5756,14 @@ f_getcellwidths(typval_T *argvars UNUSED, typval_T *rettv) list_T *entry = list_alloc(); if (entry == NULL) break; - list_append_number(entry, (varnumber_T)cw_table[i].first); - list_append_number(entry, (varnumber_T)cw_table[i].last); - list_append_number(entry, (varnumber_T)cw_table[i].width); - - list_append_list(rettv->vval.v_list, entry); + if (list_append_number(entry, (varnumber_T)cw_table[i].first) == FAIL + || list_append_number(entry, (varnumber_T)cw_table[i].last) == FAIL + || list_append_number(entry, (varnumber_T)cw_table[i].width) == FAIL + || list_append_list(rettv->vval.v_list, entry) == FAIL) + { + list_free(entry); + break; + } } } diff --git a/src/version.c b/src/version.c index 5565ccbe9..d63abc349 100644 --- a/src/version.c +++ b/src/version.c @@ -696,6 +696,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1216, +/**/ 1215, /**/ 1214, |