Fix HLSLParser: Always check preprocessor directive stack to be empty before using the top element

kblaschke · kblaschke · commit 3b1ead2af220 · 2026-04-24T10:35:20.000+02:00
diff --git a/vendor/hlslparser/src/HLSLParser.cpp b/vendor/hlslparser/src/HLSLParser.cpp
@@ -3600,7 +3600,7 @@ bool HLSLParser::ApplyPreprocessor(const char* fileName, const char* buffer, siz
     isCodeActive.push(true);
     m_tokenizer = HLSLTokenizer(fileName, buffer, length);
     sourcePreprocessed.clear();
-    while (m_tokenizer.GetToken() != HLSLToken_EndOfStream)
+    while (m_tokenizer.GetToken() != HLSLToken_EndOfStream && !isCodeActive.empty())
     {
         bool addOriginalSource = true;
 

Original file line number	Diff line number	Diff line change
`@@ -3600,7 +3600,7 @@ bool HLSLParser::ApplyPreprocessor(const char* fileName, const char* buffer, siz`
`3600`	`3600`	`isCodeActive.push(true);`
`3601`	`3601`	`m_tokenizer = HLSLTokenizer(fileName, buffer, length);`
`3602`	`3602`	`sourcePreprocessed.clear();`
`3603`		`- while (m_tokenizer.GetToken() != HLSLToken_EndOfStream)`
	`3603`	`+ while (m_tokenizer.GetToken() != HLSLToken_EndOfStream && !isCodeActive.empty())`
`3604`	`3604`	`{`
`3605`	`3605`	`bool addOriginalSource = true;`
`3606`	`3606`