Merge pull request #23412 from sharwell/InternalsVisibleToCompletionP…

…rovider_NRE_Fix Fix for #22690 NRE in GetAttributeSyntaxNodeOfToken
dotnet · Nov 29, 2017 · 5429b35 · 5429b35
2 parents 925ac9b + 95e9efd
commit 5429b35
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...atures/Core/Portable/Completion/Providers/AbstractInternalsVisibleToCompletionProvider.cs b/...atures/Core/Portable/Completion/Providers/AbstractInternalsVisibleToCompletionProvider.cs
@@ -75,7 +75,7 @@ private static SyntaxNode GetAttributeSyntaxNodeOfToken(ISyntaxFactsService synt
             //[Attribute(""|
             //[Attribute("Text"|)
             var node = token.Parent;
-            if (syntaxFactsService.IsStringLiteralExpression(node))
+            if (node != null && syntaxFactsService.IsStringLiteralExpression(node))
             {
                 // Edge case: ElementAccessExpressionSyntax is present if the following statement is another attribute:
                 //   [assembly: System.Runtime.CompilerServices.InternalsVisibleTo("|
@@ -250,4 +250,4 @@ private static string GetPublicKeyAsHexString(ImmutableArray<byte> publicKey)
             return pooledStrBuilder.ToStringAndFree();
         }
     }
-}
+}