commit 2354b66ac52a8a9165e8bd0d642e19e757ba12c3
author Bram Moolenaar <Bram@vim.org> Sun Apr 23 21:42:25 2023 +0100
committer Bram Moolenaar <Bram@vim.org> Sun Apr 23 21:42:25 2023 +0100
tree 1233f7470292cf4b72583d7ad9fbe0595a290f93
parent aae583441bcdbb0bfef3b8a1d193e04ae09ca95d

patch 9.0.1482: crash when textprop has a very large "padding" value

Problem:    Crash when textprop has a very large "padding" value. (Yegappan
            Lakshmanan)
Solution:   Avoid the "after" count to go negative.

src/drawline.c

diff --git a/src/drawline.c b/src/drawline.c
index c7510a6..f3d70ed 100644
--- a/src/drawline.c
+++ b/src/drawline.c
@@ -665,6 +665,12 @@
 	{
 	    before = 0;
 	    after = wp->w_width - cells - win_col_off(wp) - padding;
+	    if (after < 0)
+	    {
+		// text "above" has too much padding to fit
+		padding += after;
+		after = 0;
+	    }
 	}
 	else
 	{

src/testdir/dumps/Test_prop_above_padding_1.dump

diff --git a/src/testdir/dumps/Test_prop_above_padding_1.dump b/src/testdir/dumps/Test_prop_above_padding_1.dump
new file mode 100644
index 0000000..1940542
--- /dev/null
+++ b/src/testdir/dumps/Test_prop_above_padding_1.dump
@@ -0,0 +1,8 @@
+> +0&#ffffff0@74
+|s|k|y| |i|s| |b|l|u|e| @63
+@74|…
+|o|c|e|a|n| |i|s| |b|l|u|e| @61
+|~+0#4040ff13&| @73
+|~| @73
+|~| @73
+| +0#0000000&@56|1|,|0|-|1| @8|A|l@1| 

src/testdir/test_textprop.vim

diff --git a/src/testdir/test_textprop.vim b/src/testdir/test_textprop.vim
index 9cd09bd..550a0e5 100644
--- a/src/testdir/test_textprop.vim
+++ b/src/testdir/test_textprop.vim
@@ -3199,6 +3199,26 @@
   call StopVimInTerminal(buf)
 endfunc
 
+func Test_prop_with_text_above_padding()
+  CheckRunVimInTerminal
+
+  let lines =<< trim END
+      vim9script
+
+      setlocal tabstop=8 noexpandtab
+      setline(1, ['', 'sky is blue', 'ocean is blue'])
+      prop_type_add('DiagVirtualText', {highlight: 'Search', override: true})
+      prop_add(3, 0, {text: "┌─ start", text_align: "above",
+               type: 'DiagVirtualText',
+               text_padding_left: 200})
+  END
+  call writefile(lines, 'XscriptAbovePadding', 'D')
+  let buf = RunVimInTerminal('-S XscriptAbovePadding', #{rows: 8})
+  call VerifyScreenDump(buf, 'Test_prop_above_padding_1', {})
+
+  call StopVimInTerminal(buf)
+endfunc
+
 func Test_prop_above_with_indent()
   new
   call setline(1, ['first line', '    second line', '    line below'])

src/version.c

diff --git a/src/version.c b/src/version.c
index a87121f..aa480c1 100644
--- a/src/version.c
+++ b/src/version.c
@@ -696,6 +696,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    1482,
+/**/
     1481,
 /**/
     1480,