Skip to content

Commit 3ca4139

Browse files
Merge pull request PowerShell#583 from tylerl0706/issue582-large-number-crash
Catch ArgumentOutOfRange exception
2 parents ff59c49 + 1c1f6c4 commit 3ca4139

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

src/PowerShellEditorServices/Workspace/ScriptRegion.cs

+12-1
Original file line numberDiff line numberDiff line change
@@ -83,10 +83,21 @@ public sealed class ScriptRegion : IScriptExtent
8383
/// </returns>
8484
public static ScriptRegion Create(IScriptExtent scriptExtent)
8585
{
86+
// IScriptExtent throws an ArgumentOutOfRange exception if Text is null
87+
string scriptExtentText;
88+
try
89+
{
90+
scriptExtentText = scriptExtent.Text;
91+
}
92+
catch (ArgumentOutOfRangeException e)
93+
{
94+
scriptExtentText = string.Empty;
95+
}
96+
8697
return new ScriptRegion
8798
{
8899
File = scriptExtent.File,
89-
Text = scriptExtent.Text,
100+
Text = scriptExtentText,
90101
StartLineNumber = scriptExtent.StartLineNumber,
91102
StartColumnNumber = scriptExtent.StartColumnNumber,
92103
StartOffset = scriptExtent.StartOffset,

0 commit comments

Comments
 (0)