patch 8.1.1344: Coverity complains about possibly using a NULL pointer
Problem: Coverity complains about possibly using a NULL pointer and copying
a string into a fixed size buffer.
Solution: Check for NULL, even though it should not happen. Use
vim_strncpy() instead of strcpy().
diff --git a/src/change.c b/src/change.c
index f1c3cc4..33db0c1 100644
--- a/src/change.c
+++ b/src/change.c
@@ -193,12 +193,16 @@
// can be merged.
di = dict_find(li->li_tv.vval.v_dict,
(char_u *)"end", -1);
- nr = tv_get_number(&di->di_tv);
- if (lnume > nr)
- di->di_tv.vval.v_number = lnume;
+ if (di != NULL)
+ {
+ nr = tv_get_number(&di->di_tv);
+ if (lnume > nr)
+ di->di_tv.vval.v_number = lnume;
+ }
di = dict_find(li->li_tv.vval.v_dict,
(char_u *)"added", -1);
- di->di_tv.vval.v_number += xtra;
+ if (di != NULL)
+ di->di_tv.vval.v_number += xtra;
return TRUE;
}
}