search

lifting-bits / sleigh

Unofficial CMake build for Ghidra's C++ SLEIGH code
Apache License 2.0
129 stars 22 forks source link

Update Ghidra HEAD to commit 5ab72bf4f + UBSAN issue #272

Open auto-updater[bot] opened 2 months ago

auto-updater[bot] commented 2 months ago

Remove UBSAN fix in funcdata_varnode.cc

Tests now pass. The UBSAN changes will be revisited when future testing with sanitizers is investigated and integrated into CI.

Update Ghidra HEAD to commit 5ab72bf4f. Changed files:

M   Ghidra/Features/Decompiler/src/decompile/cpp/Makefile
M   Ghidra/Features/Decompiler/src/decompile/cpp/architecture.cc
A   Ghidra/Features/Decompiler/src/decompile/cpp/constseq.cc
A   Ghidra/Features/Decompiler/src/decompile/cpp/constseq.hh
M   Ghidra/Features/Decompiler/src/decompile/cpp/coreaction.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/coreaction.hh
M   Ghidra/Features/Decompiler/src/decompile/cpp/database.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/database.hh
M   Ghidra/Features/Decompiler/src/decompile/cpp/flow.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/funcdata.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/funcdata.hh
M   Ghidra/Features/Decompiler/src/decompile/cpp/funcdata_block.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/funcdata_varnode.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/jumptable.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/merge.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/op.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/op.hh
M   Ghidra/Features/Decompiler/src/decompile/cpp/printc.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/ruleaction.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/sleighbase.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/stringmanage.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/stringmanage.hh
M   Ghidra/Features/Decompiler/src/decompile/cpp/subflow.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/type.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/type.hh
M   Ghidra/Features/Decompiler/src/decompile/cpp/typeop.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/userop.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/userop.hh
M   Ghidra/Features/Decompiler/src/decompile/cpp/variable.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/variable.hh
M   Ghidra/Features/Decompiler/src/decompile/cpp/varmap.cc
M   Ghidra/Features/Decompiler/src/decompile/cpp/varmap.hh
A   Ghidra/Features/Decompiler/src/decompile/datatests/stackstring.xml
ekilmer commented 2 months ago

Update: After doing some testing, the failure in the tests is caused by a change somewhere in the UBSAN fixes patch file https://github.com/lifting-bits/sleigh/blob/2d6564857152983a6759162a7cb2e15e21b12f64/src/patches/HEAD/0001-Fix-UBSAN-errors-in-decompiler.patch

ekilmer commented 1 month ago

Hmmmm. Windows Debug builds of HEAD seem to time out. Need to test this further.