[vim/vim] Integer overflow with large 'sidescrolloff' (PR #20026)

zeertzjq

unread,

Apr 20, 2026, 10:27:19 AM (yesterday) Apr 20

to vim/vim, Subscribed

Problem: Integer overflow with large 'sidescrolloff'.
Solution: Check for overflow.

You can view, comment on, or merge this pull request online at:

https://github.com/vim/vim/pull/20026

Commit Summary

8d1a7da Integer overflow with large 'sidescrolloff'

File Changes

(2 files)

M src/normal.c (4)
M src/testdir/test_normal.vim (15)

Patch Links:

—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.

zeertzjq

unread,

Apr 20, 2026, 10:30:02 AM (yesterday) Apr 20

to vim/vim, Push

@zeertzjq pushed 1 commit.

a0eb565 Integer overflow with large 'sidescrolloff'

—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.

zeertzjq

unread,

Apr 20, 2026, 11:25:45 AM (yesterday) Apr 20

to vim/vim, Push

@zeertzjq pushed 1 commit.

d6faa64 Integer overflow with "ze" and large 'sidescrolloff'

—
View it on GitHub or unsubscribe.

You are receiving this because you are subscribed to this thread.

zeertzjq

unread,

Apr 20, 2026, 11:29:21 AM (yesterday) Apr 20

to vim/vim, Subscribed

@zeertzjq commented on this pull request.

In src/move.c:

> @@ -1329,9 +1328,9 @@ curs_columns(
 		if (diff < p_ss)
 		    diff = p_ss;
 		if (off_left < 0)
-		    new_leftcol = curwin->w_leftcol - diff;
+		    new_leftcol = curwin->w_leftcol - (int)diff;

Only reaching here when diff < width1 / 2

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

zeertzjq

unread,

Apr 20, 2026, 6:43:33 PM (21 hours ago) Apr 20

to vim/vim, Push

@zeertzjq pushed 1 commit.

e721a1f Integer overflow with "ze" and large 'sidescrolloff'

—
View it on GitHub or unsubscribe.

You are receiving this because you are subscribed to this thread.

zeertzjq

unread,

Apr 20, 2026, 7:05:20 PM (21 hours ago) Apr 20

to vim/vim, Push

@zeertzjq pushed 1 commit.

29f0dba Integer overflow with "ze" and large 'sidescrolloff'

—
View it on GitHub or unsubscribe.

You are receiving this because you are subscribed to this thread.

zeertzjq

unread,

Apr 20, 2026, 7:12:24 PM (21 hours ago) Apr 20

to vim/vim, Push

@zeertzjq pushed 1 commit.

50a5abf Integer overflow with "ze" and large 'sidescrolloff'

—
View it on GitHub or unsubscribe.

You are receiving this because you are subscribed to this thread.

zeertzjq

unread,

Apr 20, 2026, 8:26:00 PM (19 hours ago) Apr 20

to vim/vim, Push

@zeertzjq pushed 1 commit.

e9e619b Integer overflow with "ze" and large 'sidescrolloff'

—
View it on GitHub or unsubscribe.

You are receiving this because you are subscribed to this thread.

Christian Brabandt

unread,

3:41 PM (10 minutes ago) 3:41 PM

to vim/vim, Subscribed

chrisbra left a comment (vim/vim#20026)

thanks

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Christian Brabandt

unread,

3:50 PM (1 minute ago) 3:50 PM

to vim/vim, Subscribed

Closed #20026 via 33f3965.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Reply all

Reply to author

Forward