]> granicus.if.org Git - vim/commitdiff
patch 9.0.1216: Coverity warns for ignoring return value v9.0.1216
authorBram Moolenaar <Bram@vim.org>
Wed, 18 Jan 2023 12:45:30 +0000 (12:45 +0000)
committerBram Moolenaar <Bram@vim.org>
Wed, 18 Jan 2023 12:45:30 +0000 (12:45 +0000)
Problem:    Coverity warns for ignoring return value.
Solution:   Break out of loop if function fails.

src/mbyte.c
src/version.c

index 4f6c8502afd8d8230eaa5efc79a6c4604c6564eb..6d7137ed5586ce37ca9c6f8bf3f0f03d690c11ef 100644 (file)
@@ -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;
+       }
     }
 }
 
index 5565ccbe974c3cf4320474ded7d205978fb5a7e0..d63abc34902bad6f524a698773e6c9bb369cb7d1 100644 (file)
@@ -695,6 +695,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1216,
 /**/
     1215,
 /**/