From 6c2986484733b53056fe9977831b7dba821a2923 Mon Sep 17 00:00:00 2001 From: Daniel Imms <2193314+Tyriar@users.noreply.github.com> Date: Wed, 2 Nov 2022 06:45:25 -0700 Subject: [PATCH] Add 1024 character limit to web links addon Fixes #4250 --- addons/xterm-addon-web-links/src/WebLinkProvider.ts | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/addons/xterm-addon-web-links/src/WebLinkProvider.ts b/addons/xterm-addon-web-links/src/WebLinkProvider.ts index 2f2ccddf..fafbb614 100644 --- a/addons/xterm-addon-web-links/src/WebLinkProvider.ts +++ b/addons/xterm-addon-web-links/src/WebLinkProvider.ts @@ -47,6 +47,12 @@ export class LinkComputer { const [line, startLineIndex] = LinkComputer._translateBufferLineToStringWithWrap(y - 1, false, terminal); + // Don't try if the wrapped line if excessively large as the regex matching will block the main + // thread. + if (line.length > 1024) { + return []; + } + let match; let stringIndex = -1; const result: ILink[] = [];