[llvm] r277681 - pdbdump: Fix crash bug.
Rui Ueyama via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 3 16:43:23 PDT 2016
Author: ruiu
Date: Wed Aug 3 18:43:23 2016
New Revision: 277681
URL: http://llvm.org/viewvc/llvm-project?rev=277681&view=rev
Log:
pdbdump: Fix crash bug.
pdbdump calls DbiStreamBuilder::commit through PDBFileBuilder::commit
without calling DbiStreamBuilder::finalize. Because `finalize` initializes
`Header` member, `Header` remained nullptr which caused a crash bug.
Differential Revision: https://reviews.llvm.org/D23143
Added:
llvm/trunk/test/DebugInfo/PDB/pdbdump-readwrite.test
Modified:
llvm/trunk/include/llvm/DebugInfo/PDB/Raw/DbiStreamBuilder.h
llvm/trunk/lib/DebugInfo/PDB/Raw/DbiStreamBuilder.cpp
Modified: llvm/trunk/include/llvm/DebugInfo/PDB/Raw/DbiStreamBuilder.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/DebugInfo/PDB/Raw/DbiStreamBuilder.h?rev=277681&r1=277680&r2=277681&view=diff
==============================================================================
--- llvm/trunk/include/llvm/DebugInfo/PDB/Raw/DbiStreamBuilder.h (original)
+++ llvm/trunk/include/llvm/DebugInfo/PDB/Raw/DbiStreamBuilder.h Wed Aug 3 18:43:23 2016
@@ -49,7 +49,7 @@ public:
Expected<std::unique_ptr<DbiStream>> build(PDBFile &File,
const msf::WritableStream &Buffer);
Error commit(const msf::MSFLayout &Layout,
- const msf::WritableStream &Buffer) const;
+ const msf::WritableStream &Buffer);
private:
Error finalize();
Modified: llvm/trunk/lib/DebugInfo/PDB/Raw/DbiStreamBuilder.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/DebugInfo/PDB/Raw/DbiStreamBuilder.cpp?rev=277681&r1=277680&r2=277681&view=diff
==============================================================================
--- llvm/trunk/lib/DebugInfo/PDB/Raw/DbiStreamBuilder.cpp (original)
+++ llvm/trunk/lib/DebugInfo/PDB/Raw/DbiStreamBuilder.cpp Wed Aug 3 18:43:23 2016
@@ -249,7 +249,10 @@ DbiStreamBuilder::build(PDBFile &File, c
}
Error DbiStreamBuilder::commit(const msf::MSFLayout &Layout,
- const msf::WritableStream &Buffer) const {
+ const msf::WritableStream &Buffer) {
+ if (auto EC = finalize())
+ return EC;
+
auto InfoS =
WritableMappedBlockStream::createIndexedStream(Layout, Buffer, StreamDBI);
Added: llvm/trunk/test/DebugInfo/PDB/pdbdump-readwrite.test
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/PDB/pdbdump-readwrite.test?rev=277681&view=auto
==============================================================================
--- llvm/trunk/test/DebugInfo/PDB/pdbdump-readwrite.test (added)
+++ llvm/trunk/test/DebugInfo/PDB/pdbdump-readwrite.test Wed Aug 3 18:43:23 2016
@@ -0,0 +1,4 @@
+; RUN: llvm-pdbdump pdb2yaml -dbi-module-info -dbi-module-source-info \
+; RUN: -dbi-stream -pdb-stream -stream-directory -stream-metadata \
+; RUN: %p/Inputs/empty.pdb > %t.1
+; RUN: llvm-pdbdump yaml2pdb -pdb=%t.2 %t.1
More information about the llvm-commits
mailing list