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 /src/mbyte.c | |
parent | 0ef9a5c09482649cf0cc6768ed6fc640b4ed2a0a (diff) |
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.
Diffstat (limited to 'src/mbyte.c')
-rw-r--r-- | src/mbyte.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/src/mbyte.c b/src/mbyte.c index 4f6c8502af..6d7137ed55 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; + } } } |