From: Bram Moolenaar <Bram@vim.org>
Date: Mon, 1 May 2017 12:14:04 +0000 (+0200)
Subject: patch 8.0.0595: Coverity warning for not checking return value
X-Git-Tag: v8.0.0595
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=beb9cb19c660484488a71a25eda46ab0fa579278;p=vim

patch 8.0.0595: Coverity warning for not checking return value

Problem:    Coverity warning for not checking return value of dict_add().
Solution:   Check the return value for FAIL.
---

diff --git a/src/quickfix.c b/src/quickfix.c
index 0c9770205..7b07f9358 100644
--- a/src/quickfix.c
+++ b/src/quickfix.c
@@ -4722,7 +4722,8 @@ get_errorlist_properties(win_T *wp, dict_T *what, dict_T *retdict)
 	    if (di != NULL)
 	    {
 		copy_tv(qi->qf_lists[qf_idx].qf_ctx, &di->di_tv);
-		dict_add(retdict, di);
+		if (dict_add(retdict, di) == FAIL)
+		    dictitem_free(di);
 	    }
 	}
 	else
diff --git a/src/version.c b/src/version.c
index 6e2620d25..8895b73b0 100644
--- a/src/version.c
+++ b/src/version.c
@@ -764,6 +764,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    595,
 /**/
     594,
 /**/