Fixes a crash when in/out varying variable had no semantic name and an error was…

Fixes a crash when in/out varying variable had no semantic name and an error was reported in ioremapper phase when the variable was rejected

Fixes a crash when in/out varying variable had no semantic name and an error was…
9f8cb850 · t.jung · 62faea79 · 9f8cb850
Commit 9f8cb850 authored Jul 24, 2018 by t.jung
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 2 deletions

iomapper.cpp glslang/MachineIndependent/iomapper.cpp +8 -2

No files found.
--- a/glslang/MachineIndependent/iomapper.cpp
+++ b/glslang/MachineIndependent/iomapper.cpp
@@ -331,8 +331,14 @@ struct TResolverInOutAdaptor
                                                      ent.symbol->getType(),
                                                      ent.live);
        } else {
-            TString errorMsg = "Invalid shader In/Out variable semantic: ";
+            TString errorMsg;
-            errorMsg += ent.symbol->getType().getQualifier().semanticName;
+            if (ent.symbol->getType().getQualifier().semanticName != nullptr) {
+                errorMsg = "Invalid shader In/Out variable semantic: ";
+                errorMsg += ent.symbol->getType().getQualifier().semanticName;
+            } else {
+                errorMsg = "Invalid shader In/Out variable: ";
+                errorMsg += ent.symbol->getName();
+            }
            infoSink.info.message(EPrefixInternalError, errorMsg.c_str());
            error = true;
        }