# HG changeset patch # User Bram Moolenaar # Date 1646946004 -3600 # Node ID 73f4d1e479f2746a478e69bd5acab5780365f0b3 # Parent 8b90d89c9bdd84a32ddd7daa39b4a1815840b395 patch 8.2.4541: Crash in debugger when a variable is not available Commit: https://github.com/vim/vim/commit/e406ff87c86de9da2d02d0e5ebbbf5c5eac051a6 Author: Bram Moolenaar Date: Thu Mar 10 20:47:43 2022 +0000 patch 8.2.4541: Crash in debugger when a variable is not available Problem: Crash in debugger when a variable is not available in the current block. Solution: Check for a NULL name. (closes #9926) diff --git a/src/testdir/test_debugger.vim b/src/testdir/test_debugger.vim --- a/src/testdir/test_debugger.vim +++ b/src/testdir/test_debugger.vim @@ -73,6 +73,13 @@ func Test_Debugger() endtry return var1 endfunc + def Vim9Func() + for cmd in ['confirm', 'xxxxxxx'] + for _ in [1, 2] + echo cmd + endfor + endfor + enddef END call writefile(lines, 'Xtest.vim') @@ -298,6 +305,14 @@ func Test_Debugger() \ 'line 5: catch']) call RunDbgCmd(buf, 'c') + " Test showing local variable in :def function + call RunDbgCmd(buf, ':breakadd func 2 Vim9Func') + call RunDbgCmd(buf, ':call Vim9Func()', ['line 2: for _ in [1, 2]']) + call RunDbgCmd(buf, 'next', ['line 2: for _ in [1, 2]']) + call RunDbgCmd(buf, 'echo cmd', ['confirm']) + call RunDbgCmd(buf, 'breakdel *') + call RunDbgCmd(buf, 'cont') + " Test for :quit call RunDbgCmd(buf, ':debug echo Foo()') call RunDbgCmd(buf, 'breakdel *') diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -751,6 +751,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 4541, +/**/ 4540, /**/ 4539, diff --git a/src/vim9execute.c b/src/vim9execute.c --- a/src/vim9execute.c +++ b/src/vim9execute.c @@ -1622,7 +1622,10 @@ lookup_debug_var(char_u *name) // Go through the local variable names, from last to first. for (idx = debug_var_count - 1; idx >= 0; --idx) { - if (STRCMP(((char_u **)dfunc->df_var_names.ga_data)[idx], name) == 0) + char_u *varname = ((char_u **)dfunc->df_var_names.ga_data)[idx]; + + // the variable name may be NULL when not available in this block + if (varname != NULL && STRCMP(varname, name) == 0) return STACK_TV_VAR(idx); }