Skip to content

Commit 945121d

Browse files
authored
Merge pull request github#15922 from tamasvajk/buildless/namespace-extraction
C#: Handle namespace resolution error more gracefully
2 parents 2280469 + b5f349b commit 945121d

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

csharp/extractor/Semmle.Extraction.CSharp/Entities/NamespaceDeclaration.cs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,12 @@ public override void WriteId(EscapingTextWriter trapFile)
2727

2828
public override void Populate(TextWriter trapFile)
2929
{
30-
var @namespace = (INamespaceSymbol)Context.GetModel(node).GetSymbolInfo(node.Name).Symbol!;
30+
var @namespace = (INamespaceSymbol?)Context.GetModel(node).GetSymbolInfo(node.Name).Symbol;
31+
if (@namespace is null)
32+
{
33+
throw new InternalError(node, "Namespace symbol not found");
34+
}
35+
3136
var ns = Namespace.Create(Context, @namespace);
3237
trapFile.namespace_declarations(this, ns);
3338
trapFile.namespace_declaration_location(this, Context.CreateLocation(node.Name.GetLocation()));

0 commit comments

Comments
 (0)