summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBram Moolenaar <Bram@vim.org>2023-05-09 17:09:30 +0100
committerBram Moolenaar <Bram@vim.org>2023-05-09 17:09:30 +0100
commitd1ae8366aff286d41e7f5bc513cc0a1af5130aad (patch)
treec0b474e5d0767d6d4229090577c3a8cc92ecf0e1
parentaa6ba308a1498dc8da04d1d30ec0470018bf782a (diff)
downloadvim-git-d1ae8366aff286d41e7f5bc513cc0a1af5130aad.tar.gz
patch 9.0.1531: crash when register contents ends up being invalidv9.0.1531
Problem: Crash when register contents ends up being invalid. Solution: Check "y_array" is not NULL.
-rw-r--r--src/register.c2
-rw-r--r--src/testdir/test_registers.vim17
-rw-r--r--src/version.c2
3 files changed, 20 insertions, 1 deletions
diff --git a/src/register.c b/src/register.c
index f3df79cfd..e481d843c 100644
--- a/src/register.c
+++ b/src/register.c
@@ -301,7 +301,7 @@ get_register(
if (copy)
{
// If we run out of memory some or all of the lines are empty.
- if (reg->y_size == 0)
+ if (reg->y_size == 0 || y_current->y_array == NULL)
reg->y_array = NULL;
else
reg->y_array = ALLOC_MULT(char_u *, reg->y_size);
diff --git a/src/testdir/test_registers.vim b/src/testdir/test_registers.vim
index e96693247..33ea0f4bd 100644
--- a/src/testdir/test_registers.vim
+++ b/src/testdir/test_registers.vim
@@ -835,6 +835,23 @@ func Test_end_reg_executing()
bwipe!
endfunc
+" This was causing a crash because y_append was ending up being NULL
+func Test_zero_y_append()
+ " Run in a separate Vim instance because changing 'encoding' may cause
+ " trouble for later tests.
+ let lines =<< trim END
+ d
+ silent ?n
+ next <sfile>
+ so
+ sil! norm 0V€PSP
+ set enc=latin1
+  
+ END
+ call writefile(lines, 'XTest_zero_y_append', 'D')
+ call RunVim([], [], '-u NONE -i NONE -e -s -S XTest_zero_y_append -c qa\!')
+endfunc
+
" Make sure that y_append is correctly reset
" and the previous register is working as expected
func Test_register_y_append_reset()
diff --git a/src/version.c b/src/version.c
index 310803cb8..7ee9f575f 100644
--- a/src/version.c
+++ b/src/version.c
@@ -696,6 +696,8 @@ static char *(features[]) =
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
+ 1531,
+/**/
1530,
/**/
1529,