Commit: patch 9.0.1911: Vim9: segfault with null object and instanceof()

4 views
Skip to first unread message

Christian Brabandt

unread,
Sep 19, 2023, 2:30:14 PM9/19/23
to vim...@googlegroups.com
patch 9.0.1911: Vim9: segfault with null object and instanceof()

Commit: https://github.com/vim/vim/commit/3da696db6ac6103eb8f07cf7be23fa0e12c2419d
Author: Ernie Rael <err...@raelity.com>
Date: Tue Sep 19 20:14:18 2023 +0200

patch 9.0.1911: Vim9: segfault with null object and instanceof()

Problem: Vim9: segfault with null object and instanceof()
Solution: return early

closes: #13121

Signed-off-by: Christian Brabandt <c...@256bit.org>
Co-authored-by: Ernie Rael <err...@raelity.com>

diff --git a/src/testdir/test_vim9_class.vim b/src/testdir/test_vim9_class.vim
index fa621b25a..1021114a9 100644
--- a/src/testdir/test_vim9_class.vim
+++ b/src/testdir/test_vim9_class.vim
@@ -3032,6 +3032,10 @@ def Test_instanceof()
assert_true(instanceof(a3, [Base1, Base2, Intf1]))
enddef
Foo()
+
+ var o_null: Base1
+ assert_false(instanceof(o_null, Base1))
+
END
v9.CheckSourceSuccess(lines)
enddef
diff --git a/src/version.c b/src/version.c
index c2bef2ff5..79b1e11c7 100644
--- a/src/version.c
+++ b/src/version.c
@@ -699,6 +699,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
+/**/
+ 1911,
/**/
1910,
/**/
diff --git a/src/vim9class.c b/src/vim9class.c
index da862f266..f122f90c1 100644
--- a/src/vim9class.c
+++ b/src/vim9class.c
@@ -2865,6 +2865,9 @@ f_instanceof(typval_T *argvars, typval_T *rettv)
|| check_for_class_or_list_arg(argvars, 1) == FAIL)
return;

+ if (object_tv->vval.v_object == NULL)
+ return;
+
if (classinfo_tv->v_type == VAR_LIST)
{
FOR_ALL_LIST_ITEMS(classinfo_tv->vval.v_list, li)
Reply all
Reply to author
Forward
0 new messages