Skip to content

Commit 8d4ed11

Browse files
committed
patch 8.2.0510: Coverity complains about using uninitialized variable
Problem: Coverity complains about using uninitialized variable. Solution: Assign a value to "scol". Move code inside NULL check.
1 parent cde0ff3 commit 8d4ed11

3 files changed

Lines changed: 14 additions & 10 deletions

File tree

src/beval.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,8 @@ find_word_under_cursor(
110110
lbuf = vim_strnsave(lbuf, len);
111111
}
112112
}
113+
else
114+
scol = col;
113115

114116
if (winp != NULL)
115117
*winp = wp;

src/popupwin.c

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -798,16 +798,16 @@ apply_general_options(win_T *wp, dict_T *dict)
798798
if (*str != NUL)
799799
wp->w_border_char[i] = mb_ptr2char(str);
800800
}
801-
}
802-
if (list->lv_len == 1)
803-
for (i = 1; i < 8; ++i)
804-
wp->w_border_char[i] = wp->w_border_char[0];
805-
if (list->lv_len == 2)
806-
{
807-
for (i = 4; i < 8; ++i)
808-
wp->w_border_char[i] = wp->w_border_char[1];
809-
for (i = 1; i < 4; ++i)
810-
wp->w_border_char[i] = wp->w_border_char[0];
801+
if (list->lv_len == 1)
802+
for (i = 1; i < 8; ++i)
803+
wp->w_border_char[i] = wp->w_border_char[0];
804+
if (list->lv_len == 2)
805+
{
806+
for (i = 4; i < 8; ++i)
807+
wp->w_border_char[i] = wp->w_border_char[1];
808+
for (i = 1; i < 4; ++i)
809+
wp->w_border_char[i] = wp->w_border_char[0];
810+
}
811811
}
812812
}
813813
}

src/version.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -738,6 +738,8 @@ static char *(features[]) =
738738

739739
static int included_patches[] =
740740
{ /* Add new patch number below this line */
741+
/**/
742+
510,
741743
/**/
742744
509,
743745
/**/

0 commit comments

Comments
 (0)