Skip to content

Commit

Permalink
patch 8.0.0595: Coverity warning for not checking return value
Browse files Browse the repository at this point in the history
Problem:    Coverity warning for not checking return value of dict_add().
Solution:   Check the return value for FAIL.
  • Loading branch information
brammool committed May 1, 2017
1 parent 66c0e70 commit beb9cb1
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/quickfix.c
Expand Up @@ -4722,7 +4722,8 @@ get_errorlist_properties(win_T *wp, dict_T *what, dict_T *retdict)
if (di != NULL) if (di != NULL)
{ {
copy_tv(qi->qf_lists[qf_idx].qf_ctx, &di->di_tv); 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 else
Expand Down
2 changes: 2 additions & 0 deletions src/version.c
Expand Up @@ -764,6 +764,8 @@ static char *(features[]) =


static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
595,
/**/ /**/
594, 594,
/**/ /**/
Expand Down

0 comments on commit beb9cb1

Please sign in to comment.