Fix ilasm crash on empty local variable name (#46080)

Fixes #45908
dotnet · Dec 15, 2020 · 7ae298e · 7ae298e
1 parent 87ff54b
commit 7ae298e
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/src/coreclr/ilasm/portable_pdb.cpp b/src/coreclr/ilasm/portable_pdb.cpp
@@ -336,13 +336,16 @@ BOOL PortablePdbWriter::_DefineLocalScope(mdMethodDef methodDefToken, Scope* cur
 
     while (pLocalVar != NULL)
     {
-        mdLocalVariable locVarToken = mdLocalScopeNil;
-        USHORT attribute = 0;                                       // TODO: not supported for now
-        USHORT index = pLocalVar->dwAttr & 0xffff; // slot
-        if (FAILED(m_pdbEmitter->DefineLocalVariable(attribute, index, (char*)pLocalVar->szName, &locVarToken))) goto exit;
+        if (pLocalVar->szName != NULL)
+        {
+            mdLocalVariable locVarToken = mdLocalScopeNil;
+            USHORT attribute = 0;                                       // TODO: not supported for now
+            USHORT index = pLocalVar->dwAttr & 0xffff; // slot
+            if (FAILED(m_pdbEmitter->DefineLocalVariable(attribute, index, (char*)pLocalVar->szName, &locVarToken))) goto exit;
 
-        if (firstLocVarToken == mdLocalScopeNil)
-            firstLocVarToken = locVarToken;
+            if (firstLocVarToken == mdLocalScopeNil)
+                firstLocVarToken = locVarToken;
+        }
 
         pLocalVar = pLocalVar->pNext;
     }