Skip to content

Commit c3961fb

Browse files
author
Quoc Truong
committed
Add null checks
1 parent 68a59ef commit c3961fb

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

Engine/Helper.cs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -615,6 +615,12 @@ public bool HasSpecialVars(string varName)
615615
public Dictionary<string, List<RuleSuppression>> GetRuleSuppression(Ast ast)
616616
{
617617
List<RuleSuppression> ruleSuppressionList = new List<RuleSuppression>();
618+
Dictionary<string, List<RuleSuppression>> results = new Dictionary<string, List<RuleSuppression>>(StringComparer.OrdinalIgnoreCase);
619+
620+
if (ast == null)
621+
{
622+
return results;
623+
}
618624

619625
ScriptBlockAst sbAst = ast as ScriptBlockAst;
620626

@@ -640,7 +646,6 @@ public Dictionary<string, List<RuleSuppression>> GetRuleSuppression(Ast ast)
640646
ruleSuppressionList.AddRange(GetSuppressionsClass(typeAst));
641647
}
642648

643-
Dictionary<string, List<RuleSuppression>> results = new Dictionary<string, List<RuleSuppression>>(StringComparer.OrdinalIgnoreCase);
644649
ruleSuppressionList.Sort((item, item2) => item.StartOffset.CompareTo(item2.StartOffset));
645650

646651
foreach (RuleSuppression ruleSuppression in ruleSuppressionList)
@@ -718,7 +723,8 @@ public List<DiagnosticRecord> SuppressRule(string ruleName, Dictionary<string, L
718723
{
719724
List<DiagnosticRecord> results = new List<DiagnosticRecord>();
720725

721-
if (!ruleSuppressionsDict.ContainsKey(ruleName) || diagnostics.Count == 0)
726+
if (ruleSuppressionsDict == null || !ruleSuppressionsDict.ContainsKey(ruleName)
727+
|| diagnostics == null || diagnostics.Count == 0)
722728
{
723729
return diagnostics;
724730
}

0 commit comments

Comments
 (0)