!390 fix CVE-2022-3134

From: @wangjiang37 
Reviewed-by: @markeryang, @xiezhipeng1 
Signed-off-by: @xiezhipeng1
This commit is contained in:
openeuler-ci-bot 2022-09-15 01:44:52 +00:00 committed by Gitee
commit 9710307115
No known key found for this signature in database
GPG Key ID: 173E9B9CA92EEF8F
3 changed files with 79 additions and 2 deletions

View File

@ -59,7 +59,7 @@ index 242aa3a..74ad3d1 100644
+ endfunc
+ set tagfunc=g:Tag0unc0
+ new
+ cal assert_fails('tag 0', 'E426:')
+ cal assert_fails('tag 0', 'E1299:')
+
+ delfunc g:Tag0unc0
+ set tagfunc=

View File

@ -0,0 +1,70 @@
From ccfde4d028e891a41e3548323c3d47b06fb0b83e Mon Sep 17 00:00:00 2001
From: Bram Moolenaar <Bram@vim.org>
Date: Mon, 5 Sep 2022 19:51:13 +0100
Subject: [PATCH] patch 9.0.0389: crash when 'tagfunc' closes the window
Problem: Crash when 'tagfunc' closes the window.
Solution: Bail out when the window was closed.
---
src/globals.h | 3 +++
src/tag.c | 10 ++++++++++
src/testdir/test_tagfunc.vim | 12 ++++++++++++
3 files changed, 25 insertions(+)
diff --git a/src/globals.h b/src/globals.h
index 4d40de4..f8eabcf 100644
--- a/src/globals.h
+++ b/src/globals.h
@@ -1761,3 +1761,6 @@ EXTERN char e_illegal_character_in_word[]
EXTERN char e_command_too_recursive[]
INIT(= N_("E169: Command too recursive"));
+
+EXTERN char e_window_unexpectedly_close_while_searching_for_tags[]
+ INIT(= N_("E1299: Window unexpectedly closed while searching for tags"));
diff --git a/src/tag.c b/src/tag.c
index aceb6e4..e49a4d4 100644
--- a/src/tag.c
+++ b/src/tag.c
@@ -570,6 +570,16 @@ do_tag(
max_num_matches = MAXCOL; // If less than max_num_matches
// found: all matches found.
+ // A tag function may do anything, which may cause various
+ // information to become invalid. At least check for the tagstack
+ // to still be the same.
+ if (tagstack != curwin->w_tagstack)
+ {
+ emsg(_(e_window_unexpectedly_close_while_searching_for_tags));
+ FreeWild(new_num_matches, new_matches);
+ break;
+ }
+
// If there already were some matches for the same name, move them
// to the start. Avoids that the order changes when using
// ":tnext" and jumping to another file.
diff --git a/src/testdir/test_tagfunc.vim b/src/testdir/test_tagfunc.vim
index 68f2a50..cd60afe 100644
--- a/src/testdir/test_tagfunc.vim
+++ b/src/testdir/test_tagfunc.vim
@@ -93,4 +93,16 @@ func Test_tagfunc_wipes_buffer()
set tagfunc=
endfunc
+func Test_tagfunc_closes_window()
+ split any
+ func MytagfuncClose(pat, flags, info)
+ close
+ return [{'name' : 'mytag', 'filename' : 'Xtest', 'cmd' : '1'}]
+ endfunc
+ set tagfunc=MytagfuncClose
+ call assert_fails('tag xyz', 'E1299:')
+
+ set tagfunc=
+endfunc
+
" vim: shiftwidth=2 sts=2 expandtab
--
2.33.0

View File

@ -11,7 +11,7 @@
Name: vim
Epoch: 2
Version: 8.2
Release: 55
Release: 56
Summary: Vim is a highly configurable text editor for efficiently creating and changing any kind of text.
License: Vim and MIT
URL: http://www.vim.org
@ -158,6 +158,7 @@ Patch6124: backport-CVE-2022-2980.patch
Patch6125: backport-patch-8.2.1677-memory-access-errors-when-calling-set.patch
Patch6126: backport-CVE-2022-3016.patch
Patch6127: backport-CVE-2022-3099.patch
Patch6128: backport-CVE-2022-3134.patch
Patch9000: bugfix-rm-modify-info-version.patch
Patch9001: remove-failed-tests-due-to-patch.patch
@ -560,6 +561,12 @@ LC_ALL=en_US.UTF-8 make -j1 test
%{_mandir}/man1/evim.*
%changelog
* Wed Sep 14 2022 wangjiang <wangjiang37@h-partners.com> - 2:8.2-56
- Type:CVE
- ID:CVE-2022-3134
- SUG:NA
- DESC:fix CVE-2022-3134
* Thu Sep 08 2022 renhongxun <renhongxun@h-partners.com> - 2:8.2-55
- Type:CVE
- ID:CVE-2022-3099