Memory leak fix in SPIRVDecoder::getSourceContinuedInstructions (intel#2259)

bwlodarcz · sys-ce-bb · commit 4e7b77780b8c · 2023-12-21T06:44:37.000-08:00
SPIRVEntry created in function getEntry isn't deleted when it is ExtOp and isn't SourceContinued. Original commit: KhronosGroup/SPIRV-LLVM-Translator@b891a51
diff --git a/llvm-spirv/lib/SPIRV/libSPIRV/SPIRVStream.cpp b/llvm-spirv/lib/SPIRV/libSPIRV/SPIRVStream.cpp
@@ -357,6 +357,7 @@ std::vector<SPIRVEntry *> SPIRVDecoder::getSourceContinuedInstructions() {
     SPIRVExtInst *Inst = static_cast<SPIRVExtInst *>(Entry);
     if (Inst->getExtOp() != SPIRVDebug::Instruction::SourceContinued) {
       IS.seekg(Pos); // restore position
+      delete Entry;
       return ContinuedInst;
     }
     M.add(Entry);

Original file line number	Diff line number	Diff line change
`@@ -357,6 +357,7 @@ std::vector<SPIRVEntry *> SPIRVDecoder::getSourceContinuedInstructions() {`
`357`	`357`	`SPIRVExtInst Inst = static_cast<SPIRVExtInst >(Entry);`
`358`	`358`	`if (Inst->getExtOp() != SPIRVDebug::Instruction::SourceContinued) {`
`359`	`359`	`IS.seekg(Pos); // restore position`
	`360`	`+ delete Entry;`
`360`	`361`	`return ContinuedInst;`
`361`	`362`	`}`
`362`	`363`	`M.add(Entry);`