# HG changeset patch # User Bram Moolenaar # Date 1625679005 -7200 # Node ID 9a2e7184b604c16b6c14c80810c0168fb7da320d # Parent 86c2d209f07bd795235c89cf4024556b70115452 patch 8.2.3115: Coverity complains about free_wininfo() use Commit: https://github.com/vim/vim/commit/b5098060f4acae4dac3203130278c948d670a3d5 Author: Bram Moolenaar Date: Wed Jul 7 19:26:19 2021 +0200 patch 8.2.3115: Coverity complains about free_wininfo() use Problem: Coverity complains about free_wininfo() use. Solution: Add a condition that "wip2" is not equal to "wip". (Neovim https://github.com/vim/vim/issues/14996) diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -756,6 +756,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 3115, +/**/ 3114, /**/ 3113, diff --git a/src/window.c b/src/window.c --- a/src/window.c +++ b/src/window.c @@ -5057,8 +5057,9 @@ win_free( // If there already is an entry with "wi_win" set to NULL it // must be removed, it would never be used. + // Skip "wip" itself, otherwise Coverity complains. for (wip2 = buf->b_wininfo; wip2 != NULL; wip2 = wip2->wi_next) - if (wip2->wi_win == NULL) + if (wip2 != wip && wip2->wi_win == NULL) { if (wip2->wi_next != NULL) wip2->wi_next->wi_prev = wip2->wi_prev;