Branch: refs/heads/master Home: https://github.com/dyninst/dyninst Commit: 2c5ebda5cee7ba83b3a29dc5b72a7b6d1e837b79 https://github.com/dyninst/dyninst/commit/2c5ebda5cee7ba83b3a29dc5b72a7b6d1e837b79 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021)
Changed paths: R symtabAPI/doc/API/Symtab/.DS_Store Log Message: ----------- delete unnecessary .DS_Store file (#1082) Commit: bde441435cb3f645baa6232ac7cab25ace0f27ff https://github.com/dyninst/dyninst/commit/bde441435cb3f645baa6232ac7cab25ace0f27ff Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/image.C M dyninstAPI/src/image.h M symtabAPI/h/Symbol.h M symtabAPI/src/Object.C M symtabAPI/src/Symbol.C Log Message: ----------- remove emptyString static members (#1082) Commit: c545f7b426a5091fe832ed568cc53fa3ebc38eec https://github.com/dyninst/dyninst/commit/c545f7b426a5091fe832ed568cc53fa3ebc38eec Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/image.C M dyninstAPI/src/image.h M dyninstAPI/src/mapped_object.C Log Message: ----------- remove dynamic_ and dynamic() from fileDescriptor (#1082) These are always 0 and unused. Also remove code that no longer function from mapped_object::createMappedObject that works around a bug when processing a 32-bit mutatee on a 64-bit system Commit: 4e8ce4927a05d9d0f8b39ae6443d0f9b08373914 https://github.com/dyninst/dyninst/commit/4e8ce4927a05d9d0f8b39ae6443d0f9b08373914 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/src/addrtranslate-sysv.C M stackwalk/src/linux-swk.C Log Message: ----------- compute num array elements instead of fixed values (#1082) Commit: 58bca57a8d5e2b71918bf52fea8f6346b7723ca5 https://github.com/dyninst/dyninst/commit/58bca57a8d5e2b71918bf52fea8f6346b7723ca5 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M proccontrol/src/loadLibrary/codegen.C M symtabAPI/src/Symtab-edit.C M symtabAPI/src/parseStab.C Log Message: ----------- add missing break statements (#1082) Commit: a2c8b533d3739b04cae47f409a9d5ad8143ef8e7 https://github.com/dyninst/dyninst/commit/a2c8b533d3739b04cae47f409a9d5ad8143ef8e7 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: A common/h/compiler_annotations.h Log Message: ----------- add header with compiler annotation macros (#1082) compiler_annotations.h contains macros that expand to annotations based on the compiler and language to add attributes for functions for statements. Currently defines three macros: - DYNINST_FALLTHROUGH - indicate that a switch statement is supposed to fall through to the next case - DYNINST_PRINTF_ANNOTATION - annotate a function as printf-like - DYNINST_SCANF_ANNOTATION - annotate a function as scanf-like - DYNINST_MALLOC_ANNOTATION - mark function as malloc-like Commit: 94978b0a8628bee0bfe8f0f4b6fcdc45bc47998d https://github.com/dyninst/dyninst/commit/94978b0a8628bee0bfe8f0f4b6fcdc45bc47998d Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/src/arch-x86.C M dwarf/src/dwarfExprParser.C M dyninstAPI/src/codegen.C M dyninstAPI/src/emit-x86.C M dyninstAPI/src/inst-x86.C M dyninstAPI/src/mapped_object.C M patchAPI/src/PatchMgr.C M symtabAPI/src/parseStab.C Log Message: ----------- eliminate switch case fall through warnings (#1082) Commit: 1d7f83a8a71a56d706a14ca447031fc13822eaff https://github.com/dyninst/dyninst/commit/1d7f83a8a71a56d706a14ca447031fc13822eaff Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI_RT/src/RTcommon.c M parseAPI/src/IA_x86.h M parseAPI/src/IndirectAnalyzer.C M parseAPI/src/SymbolicExpression.C M symtabAPI/src/Object-elf.C M symtabAPI/src/dwarfWalker.h Log Message: ----------- miscellaneous compiler warning cleanups (#1082) Commit: 519942a9d502481bfd2b7d1a91387289a03ed6a7 https://github.com/dyninst/dyninst/commit/519942a9d502481bfd2b7d1a91387289a03ed6a7 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M cmake/warnings.cmake Log Message: ----------- enable more warnings and test compiler support (#1082) There is now a list of potential warning flags to be enabled for C/C++ code. These potential warning flags are then tested for support in the selected C and C++ compilers and are enabled if supported. Commit: 3b03516c66b3beb317a05d55a7b3b0bf8cbaffc5 https://github.com/dyninst/dyninst/commit/3b03516c66b3beb317a05d55a7b3b0bf8cbaffc5 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/src/NodeIterator.h M common/src/addrtranslate-linux.C M common/src/addrtranslate-sysv.C M common/src/arch-x86.C M common/src/symbolDemangle.c M dwarf/h/dwarfFrameParser.h M dynC_API/src/C.y M dynC_API/src/dynC.tab.C M dyninstAPI/src/BPatch_flowGraph.C M dyninstAPI/src/BPatch_function.C M dyninstAPI/src/BPatch_image.C M dyninstAPI/src/BPatch_memoryAccessAdapter.C M dyninstAPI/src/BPatch_module.C M dyninstAPI/src/BPatch_object.C M dyninstAPI/src/BPatch_snippet.C M dyninstAPI/src/Parsing.C M dyninstAPI/src/Relocation/Springboard.C M dyninstAPI/src/Relocation/Transformers/Movement-adhoc.C M dyninstAPI/src/Relocation/Widgets/CFWidget-x86.C M dyninstAPI/src/StackMod/StackModChecker.C M dyninstAPI/src/addressSpace.C M dyninstAPI/src/ast.C M dyninstAPI/src/codeRange.C M dyninstAPI/src/dynProcess.C M dyninstAPI/src/emit-x86.C M dyninstAPI/src/hybridInstrumentation.C M dyninstAPI/src/hybridOverwrites.C M dyninstAPI/src/image.C M dyninstAPI/src/inst-power.C M dyninstAPI/src/mapped_object.C M dyninstAPI/src/parse-aarch64.C M dyninstAPI/src/parse-power.C M dyninstAPI/src/parse-x86.C M dyninstAPI/src/registerSpace.C M instructionAPI/src/InstructionDecoder-aarch64.C M instructionAPI/src/InstructionDecoder-aarch64.h M instructionAPI/src/InstructionDecoder-amdgpu-vega.C M instructionAPI/src/InstructionDecoder-amdgpu-vega.h M instructionAPI/src/InstructionDecoder-power.C M instructionAPI/src/InstructionDecoder-x86.C M instructionAPI/src/InstructionDecoder.C M instructionAPI/src/Ternary.C M instructionAPI/src/amdgpu_decoder_impl_vega.C M parseAPI/src/BoundFactCalculator.C M parseAPI/src/CodeObject.C M parseAPI/src/Function.C M parseAPI/src/IA_power.C M parseAPI/src/IndirectAnalyzer.C M parseAPI/src/LoopAnalyzer.C M parseAPI/src/Parser.C M parseAPI/src/ProbabilisticParser.C M parseAPI/src/SymbolicExpression.C M parseThat/src/parseThat.C M proccontrol/src/event.C M proccontrol/src/generator.C M proccontrol/src/handler.C M proccontrol/src/int_thread_db.C M proccontrol/src/linux.C M proccontrol/src/notify_pipe.C M proccontrol/src/process.C M proccontrol/src/processplat.C M proccontrol/src/procpool.C M proccontrol/src/procset.C M proccontrol/src/resp.h M proccontrol/src/x86_process.C M stackwalk/src/libstate.C M stackwalk/src/linuxbsd-swk.C M stackwalk/src/sw_pcontrol.C M stackwalk/src/walker.C M symlite/src/SymLite-elf.C M symtabAPI/src/AddrLookup.C M symtabAPI/src/LineInformation.C M symtabAPI/src/Module.C M symtabAPI/src/Object-elf.C M symtabAPI/src/Symtab.C M symtabAPI/src/Type.C M symtabAPI/src/dwarfWalker.C M symtabAPI/src/emitElf.C M symtabAPI/src/parseStab.C Log Message: ----------- fix shadow identifier warnings (#1082) Commit: 392109c1d1f84003d33557f763a94b8a2efdb5ea https://github.com/dyninst/dyninst/commit/392109c1d1f84003d33557f763a94b8a2efdb5ea Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/h/Annotatable.h M common/src/DOT.C M dynC_API/src/C.l M dynC_API/src/lex.dynC.C M dyninstAPI/src/BPatch_flowGraph.C M dyninstAPI/src/Parsing.C M dyninstAPI/src/Relocation/DynCFGMaker.C M dyninstAPI/src/addressSpace.C M dyninstAPI/src/ast.C M dyninstAPI/src/function.C M dyninstAPI/src/mapped_object.C M dyninstAPI/src/parse-cfg.h M dyninstAPI/src/registerSpace.C M parseAPI/h/CFG.h M proccontrol/src/event.C M proccontrol/src/handler.C M proccontrol/src/int_thread_db.C M proccontrol/src/process.C M proccontrol/src/procset.C M symtabAPI/src/dwarfWalker.C M symtabAPI/src/parseStab.C Log Message: ----------- make printf format and argument types match (#1082) Commit: f038ff498dc14e943366f7edb97d588a929bd05d https://github.com/dyninst/dyninst/commit/f038ff498dc14e943366f7edb97d588a929bd05d Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/src/linuxKludges.C M dyninstAPI/src/parse-power.C M symtabAPI/src/emitElf.C Log Message: ----------- remove non-C++ variable length arrays (#1082) Commit: 0c671933028fa943aec1bb0101d9bbabe890a21d https://github.com/dyninst/dyninst/commit/0c671933028fa943aec1bb0101d9bbabe890a21d Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/h/dyntypes.h Log Message: ----------- remove use of GNU binary operator ?: (#1082) since the condition only used a pure function, the compiler generates identical code and only evaluates the condition once (if -O1 or greater) Commit: 3489ba73c54957d81f565b80794245d684e89e57 https://github.com/dyninst/dyninst/commit/3489ba73c54957d81f565b80794245d684e89e57 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI_RT/src/RTstatic_ctors_dtors-ppc64.c M dyninstAPI_RT/src/RTstatic_ctors_dtors-x86.c Log Message: ----------- fix illegal function pointer to void* compare (#1082) Commit: a9c994ae4192c27fc4e5098123e1989a5032e1af https://github.com/dyninst/dyninst/commit/a9c994ae4192c27fc4e5098123e1989a5032e1af Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M symtabAPI/src/emitElfStatic-x86.C Log Message: ----------- fix overflow warning for 0x90 assigned to a char (#1082) Commit: 4794203914e2f43a0675177a8ff10db937377872 https://github.com/dyninst/dyninst/commit/4794203914e2f43a0675177a8ff10db937377872 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/h/Buffer.h M common/h/DynAST.h M common/h/IBSTree.h M common/src/Buffer.C M common/src/DOT.C M common/src/Graph.C M common/src/arch-x86.h M common/src/concurrent.C M common/src/singleton_object_pool.h M dataflowAPI/h/Absloc.h M dataflowAPI/h/AbslocInterface.h M dataflowAPI/h/SymEval.h M dataflowAPI/h/slicing.h M dataflowAPI/h/stackanalysis.h M dyninstAPI/h/BPatch_addressSpace.h M dyninstAPI/h/BPatch_basicBlock.h M dyninstAPI/h/BPatch_edge.h M dyninstAPI/h/BPatch_function.h M dyninstAPI/h/BPatch_object.h M dyninstAPI/src/BPatch_basicBlock.C M dyninstAPI/src/BPatch_frame.C M dyninstAPI/src/BPatch_function.C M dyninstAPI/src/BPatch_snippet.C M dyninstAPI/src/MemoryEmulator/memEmulatorStub.h M dyninstAPI/src/Parsing.h M dyninstAPI/src/Patching.h M dyninstAPI/src/Relocation/CFG/RelocBlock.h M dyninstAPI/src/Relocation/CFG/RelocEdge.h M dyninstAPI/src/Relocation/CFG/RelocGraph.h M dyninstAPI/src/Relocation/CFG/RelocTarget.h M dyninstAPI/src/Relocation/CodeBuffer.C M dyninstAPI/src/Relocation/CodeBuffer.h M dyninstAPI/src/Relocation/CodeMover.C M dyninstAPI/src/Relocation/CodeMover.h M dyninstAPI/src/Relocation/CodeTracker.C M dyninstAPI/src/Relocation/CodeTracker.h M dyninstAPI/src/Relocation/Relocation.h M dyninstAPI/src/Relocation/Springboard.h M dyninstAPI/src/Relocation/Transformers/Instrumenter.h M dyninstAPI/src/Relocation/Transformers/Modification.C M dyninstAPI/src/Relocation/Transformers/Modification.h M dyninstAPI/src/Relocation/Transformers/Movement-adhoc.h M dyninstAPI/src/Relocation/Transformers/Movement-analysis.h M dyninstAPI/src/Relocation/Transformers/Transformer.h M dyninstAPI/src/Relocation/Widgets/ASTWidget.h M dyninstAPI/src/Relocation/Widgets/CFWidget.h M dyninstAPI/src/Relocation/Widgets/CallbackWidget.h M dyninstAPI/src/Relocation/Widgets/InsnWidget.h M dyninstAPI/src/Relocation/Widgets/InstWidget.h M dyninstAPI/src/Relocation/Widgets/PCWidget.h M dyninstAPI/src/Relocation/Widgets/RelDataWidget.h M dyninstAPI/src/Relocation/Widgets/StackModWidget.h M dyninstAPI/src/Relocation/Widgets/Widget.h M dyninstAPI/src/addressSpace.C M dyninstAPI/src/addressSpace.h M dyninstAPI/src/ast.C M dyninstAPI/src/block.C M dyninstAPI/src/codegen.C M dyninstAPI/src/hybridCallbacks.C M dyninstAPI/src/instPoint.C M dyninstAPI/src/instPoint.h M dyninstAPI/src/pcEventMuxer.C M elf/src/Elf_X.C M instructionAPI/h/ArchSpecificFormatters.h M instructionAPI/h/BinaryFunction.h M instructionAPI/h/Dereference.h M instructionAPI/h/Expression.h M instructionAPI/h/Immediate.h M instructionAPI/h/Instruction.h M instructionAPI/h/InstructionAST.h M instructionAPI/h/InstructionDecoder.h M instructionAPI/h/Operand.h M instructionAPI/h/Operation_impl.h M instructionAPI/h/Register.h M instructionAPI/h/RegisterIDs.h M instructionAPI/h/Result.h M instructionAPI/h/Ternary.h M instructionAPI/src/BinaryFunction.C M instructionAPI/src/Expression.C M instructionAPI/src/Immediate.C M instructionAPI/src/Instruction.C M instructionAPI/src/InstructionAST.C M instructionAPI/src/InstructionDecoder-aarch64.C M instructionAPI/src/InstructionDecoder-amdgpu-vega.C M instructionAPI/src/InstructionDecoder-amdgpu-vega.h M instructionAPI/src/InstructionDecoder-power.C M instructionAPI/src/InstructionDecoder-x86.C M instructionAPI/src/InstructionDecoder-x86.h M instructionAPI/src/InstructionDecoder.C M instructionAPI/src/InstructionDecoderImpl.C M instructionAPI/src/Operand.C M instructionAPI/src/Operation.C M instructionAPI/src/Register.C M instructionAPI/src/Ternary.C M parseAPI/h/CFGModifier.h M parseAPI/h/LockFreeQueue.h M parseAPI/src/CFGModifier.C M parseAPI/src/ParseCallback.C M parseAPI/src/Parser-speculative.C M parseAPI/src/ProbabilisticParser.h M parseAPI/src/StackTamperVisitor.h M patchAPI/h/CFGMaker.h M patchAPI/h/PatchCFG.h M patchAPI/h/PatchCallback.h M patchAPI/h/PatchMgr.h M patchAPI/h/PatchModifier.h M patchAPI/src/ParseCallback.h M patchAPI/src/PatchFunction.C M patchAPI/src/PatchMgr.C M proccontrol/src/event.C M proccontrol/src/linux.C M proccontrol/src/loadLibrary/codegen.h M proccontrol/src/loadLibrary/injector.h M stackwalk/src/dbgstepper-impl.h M symlite/src/SymLite-elf.C M symtabAPI/src/Region.C M symtabAPI/src/Symtab.C M symtabAPI/src/dwarfWalker.h Log Message: ----------- remove excess semicolons as reported by -pedantic (#1082) Commit: 5c753b05f657d599ec5b965abddb4f60b9dff040 https://github.com/dyninst/dyninst/commit/5c753b05f657d599ec5b965abddb4f60b9dff040 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/src/debug_common.h M dyninstAPI/src/debug.h M parseAPI/src/debug_parse.h M patchAPI/h/PatchCommon.h M proccontrol/h/PCErrors.h Log Message: ----------- fix non-standard use of __VA_ARGS__ (#1082) Eliminate use of gnu extension of "## __VA_ARGS__" to allow and empty __VA_ARGS__. Some printf's had to broken into two statement. Someday when Dyninst supports C++20, __VA_OPT__(, __VA_ARGS__) could be used. Commit: bd92e51923887efe923fd0ec6d9797edcebc0de6 https://github.com/dyninst/dyninst/commit/bd92e51923887efe923fd0ec6d9797edcebc0de6 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI_RT/h/dyninstAPI_RT.h M dyninstAPI_RT/src/RTcommon.c M dyninstAPI_RT/src/RTcommon.h Log Message: ----------- fix discard qualifiers: make char* -> const char* (#1082) Commit: c9c5fbb7048824a3845457014d431b0a14e963e4 https://github.com/dyninst/dyninst/commit/c9c5fbb7048824a3845457014d431b0a14e963e4 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI_RT/h/dyninstAPI_RT.h Log Message: ----------- disable flexible array member warning in C++ (#1082) trap_mapping_header has a flexible array member which is not allowed in C++. This disables the warning for the struct and the initialization. This struct should be rewritten. Commit: 39da54ae8874bd6c2b9c9fff9c52613908767edf https://github.com/dyninst/dyninst/commit/39da54ae8874bd6c2b9c9fff9c52613908767edf Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI_RT/src/RTmemEmulator.c Log Message: ----------- fix illegal in C empty brace initialization (#1082) Commit: 8c971dbabde9b9f613058dc9c19d90f9557b58d4 https://github.com/dyninst/dyninst/commit/8c971dbabde9b9f613058dc9c19d90f9557b58d4 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/src/stats.C M common/src/stats.h M dyninstAPI/src/infHeap.C M dyninstAPI/src/infHeap.h M instructionAPI/h/InstructionDecoder.h M instructionAPI/src/InstructionDecoder.C M proccontrol/src/int_process.h M proccontrol/src/process.C M symtabAPI/h/Symtab.h M symtabAPI/src/Object.C M symtabAPI/src/Symtab.C Log Message: ----------- add missing copy assignment (#1082) Add missing copy assignment operator if a copy constructor is defined. Made the copy constructor, destructor, and copy assignment be the default compiler generated version if possible (or deleted them to use the implicitly generated versions if they were not exported). The inferiorMap compiler generated version was not correct as it created copies of pointers that left dangling pointers when the inferiorMap was deleted. Commit: 085fd584bf9c72e481d8195aa7df6a4db76063b4 https://github.com/dyninst/dyninst/commit/085fd584bf9c72e481d8195aa7df6a4db76063b4 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/h/Annotatable.h M common/src/IntervalTree.h M common/src/stats.h M dataflowAPI/h/stackanalysis.h M dataflowAPI/src/stackanalysis.C M dwarf/h/dwarfResult.h M dyninstAPI/src/Relocation/CodeBuffer.h M dyninstAPI/src/codeRange.h M dyninstAPI/src/codegen.C M dyninstAPI/src/image.h M dyninstAPI/src/patch.h M instructionAPI/h/ArchSpecificFormatters.h M parseAPI/h/CFG.h M parseAPI/h/InstructionAdapter.h M parseAPI/src/InstructionAdapter.C M proccontrol/h/PCProcess.h M proccontrol/h/ProcessSet.h M proccontrol/src/memcache.C M proccontrol/src/memcache.h M proccontrol/src/process.C M proccontrol/src/procset.C M stackwalk/h/frame.h M symtabAPI/h/Symbol.h M symtabAPI/h/Symtab.h M symtabAPI/h/Type.h M symtabAPI/src/Type.C Log Message: ----------- fix deprecated implicit copy constructor if dtor (#1082) Commit: f1082f07c5930d09777c915479920b40f3555bc4 https://github.com/dyninst/dyninst/commit/f1082f07c5930d09777c915479920b40f3555bc4 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI_RT/CMakeLists.txt Log Message: ----------- do not compile empty compilation units (#1082) 3 files in dyninstAPI_RT only contain source code when building the static library so only compile them for that library when the preprocessor variable DYNINST_RT_STATIC_LIB is set. Commit: 5c48995d25bc4fd85c4f485dad4b11c974e8955e https://github.com/dyninst/dyninst/commit/5c48995d25bc4fd85c4f485dad4b11c974e8955e Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M symtabAPI/src/dwarfWalker.C Log Message: ----------- remove non-C++ compound literal (#1082) Commit: c4e3dd2ed973df4f8ab04a2300ce83167db5e443 https://github.com/dyninst/dyninst/commit/c4e3dd2ed973df4f8ab04a2300ce83167db5e443 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/unix.C Log Message: ----------- remove default argument from lambda (#1082) not valid in C++11 (is in C++14 and later). Commit: b0136fb81b85941fc408eea5c7d2a66e8d87b6f6 https://github.com/dyninst/dyninst/commit/b0136fb81b85941fc408eea5c7d2a66e8d87b6f6 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M symtabAPI/src/Region.C M symtabAPI/src/Variable.C Log Message: ----------- explicit base class initialization in constructor (#1082) Commit: 57372ed0cd590171e894603796861aaddeb8938d https://github.com/dyninst/dyninst/commit/57372ed0cd590171e894603796861aaddeb8938d Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/src/arch-x86.C M dyninstAPI/src/ast.C M proccontrol/src/linux.C M stackwalk/src/x86-wanderer.C Log Message: ----------- eliminate switch case fall through warnings (#1082) Commit: 3fec620b44f14a1a9e42c271eb8754afbd702959 https://github.com/dyninst/dyninst/commit/3fec620b44f14a1a9e42c271eb8754afbd702959 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/inst-x86.C M parseAPI/src/CFGModifier.C M symtabAPI/src/parseStab.C Log Message: ----------- fix duplicate branch warnings (#1082) caused by: - commented out code or the possibiltity that branches should be treated differently (commented these out) - register number for i386 and x86_64 having same ids for corresponding 32 and 64-bit registers (DISABLED via a pragma for gcc >= 7, does not exist in earlier gcc versions) only ignore -Wduplicated-branches for gcc >= 7 Commit: ee9d1af52cd34ef4ed6e68aba65499cfd3c20641 https://github.com/dyninst/dyninst/commit/ee9d1af52cd34ef4ed6e68aba65499cfd3c20641 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/StackMod/StackModChecker.C Log Message: ----------- fix shadow variable warning, has other brokenness (#1082) The methods alignStackRanges and findContiguousRange function are fundamentally broken and need to be rewritten. Only the shadow variable warning was fixed. Commit: 7ae8991d2d43eae8a410de38fcd8f2564f23c48d https://github.com/dyninst/dyninst/commit/7ae8991d2d43eae8a410de38fcd8f2564f23c48d Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M cmake/Boost.cmake M cmake/ThreadingBuildingBlocks.cmake Log Message: ----------- make Boost and TBB include dirs be system includes (#1082) This eliminates warnings from gcc 11 and from -Wduplicated-branches Commit: 6c6533650779ca1b81a04416687f7204cdbf8a20 https://github.com/dyninst/dyninst/commit/6c6533650779ca1b81a04416687f7204cdbf8a20 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dataflowAPI/rose/semantics/BinarySymbolicExpr.C M dataflowAPI/rose/semantics/BinarySymbolicExpr.h M dyninstAPI/src/BPatch_function.C M dyninstAPI/src/StackMod/StackLocation.h M parseAPI/src/Parser.C M parseAPI/src/ParserDetails.h M proccontrol/h/Decoder.h M proccontrol/src/handler.C M proccontrol/src/int_handler.h M symtabAPI/src/LinkMap.h M symtabAPI/src/Object-elf.C M symtabAPI/src/Object-nt.C M symtabAPI/src/Symtab-lookup.C M symtabAPI/src/emitElf.h Log Message: ----------- make cmp function object operator() a const func (#1082) STL in C++17 requires this Commit: bfa3101fd22c7a0d422c3765166f4217f51cc644 https://github.com/dyninst/dyninst/commit/bfa3101fd22c7a0d422c3765166f4217f51cc644 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/src/singleton_object_pool.h Log Message: ----------- fix for C++20 removal of std::allocator methods (#1082) C++20 removes pointer, construct and destroy from std::allocator - Define pointer = T* in all language versions - For C++20 and later use std::construct_at and std::destroy_at functions instread of construct and destory mothods Commit: a1a2002d02ddaa492d0004b6a62c5cf44690148a https://github.com/dyninst/dyninst/commit/a1a2002d02ddaa492d0004b6a62c5cf44690148a Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/emit-x86.C Log Message: ----------- remove obvious null pointer dereference (#1082) Commit: 57427c8085f517bb65fa15fb94e4267817ce2e96 https://github.com/dyninst/dyninst/commit/57427c8085f517bb65fa15fb94e4267817ce2e96 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/registerSpace.C Log Message: ----------- remove assert(this) as 'this' should never be null (#1082) Commit: 035e83133f9e12a8a56f3d838d0dc7deccf81fe0 https://github.com/dyninst/dyninst/commit/035e83133f9e12a8a56f3d838d0dc7deccf81fe0 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/CMakeLists.txt M instructionAPI/CMakeLists.txt Log Message: ----------- fix var-tracking-assignments warnings (#1082) remove -fno-var-tracking-assignments from IntructionsDecoder-aarch64.C as the warning is never produced for this file (with gcc >6) add -fno-var-tracking-assigments for InstructionDecoder-power.C for gcc 6-11, and arch-x86.C for gcc 7, 8 and 9. Commit: 812a14abd272c84b6ab6c47361ededb004da1154 https://github.com/dyninst/dyninst/commit/812a14abd272c84b6ab6c47361ededb004da1154 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/h/Annotatable.h M common/src/debug_common.h M common/src/lprintf.h M dyninstAPI/src/Relocation/patchapi_debug.C M dyninstAPI/src/debug.C M dyninstAPI/src/debug.h M dyninstAPI_RT/src/RTcommon.h M parseAPI/src/debug_parse.h M parseThat/src/log.h M parseThat/src/utils.h M proccontrol/src/int_thread_db.C M stackwalk/h/swk_errors.h M symtabAPI/src/debug.h Log Message: ----------- add compiler annotation to printf-like functions (#1082) Commit: 0f1042855429db0151d0e842d8a6c431e772e6ec https://github.com/dyninst/dyninst/commit/0f1042855429db0151d0e842d8a6c431e772e6ec Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/src/addrtranslate-sysv.C M common/src/debug_common.h M dwarf/src/dwarfExprParser.C M dwarf/src/dwarfResult.C M dyninstAPI/src/BPatch_function.C M dyninstAPI/src/BPatch_image.C M dyninstAPI/src/BPatch_object.C M dyninstAPI/src/addressSpace.C M dyninstAPI/src/baseTramp.C M dyninstAPI/src/binaryEdit.C M dyninstAPI/src/dynProcess.C M dyninstAPI/src/emit-x86.C M dyninstAPI/src/function.C M dyninstAPI/src/hybridCallbacks.C M dyninstAPI/src/hybridInstrumentation.C M dyninstAPI/src/hybridOverwrites.C M dyninstAPI/src/instPoint.C M dyninstAPI/src/mapped_object.C M dyninstAPI/src/parse-cfg.C M dyninstAPI/src/pcEventMuxer.C M dyninstAPI/src/registerSpace.C M dyninstAPI/src/stackwalk-aarch64.C M dyninstAPI/src/stackwalk-x86.C M dyninstAPI/src/unix.C M dyninstAPI_RT/src/RTcommon.c M dyninstAPI_RT/src/RTposix.c M parseAPI/src/BoundFactData.C M parseAPI/src/CFGModifier.C M parseAPI/src/Function.C M parseAPI/src/IA_IAPI.C M parseAPI/src/IA_power.C M parseAPI/src/IA_x86.C M parseAPI/src/IndirectAnalyzer.C M parseAPI/src/JumpTableFormatPred.C M parseAPI/src/JumpTableIndexPred.C M parseAPI/src/ParseData.h M parseAPI/src/Parser.C M parseAPI/src/ParserDetails.C M parseAPI/src/SymbolicExpression.C M parseAPI/src/SymtabCodeSource.C M parseThat/src/dyninstCompat.v5.C M parseThat/src/dyninstCore.C M stackwalk/src/aarch64-swk.C M stackwalk/src/callchecker-IAPI.C M stackwalk/src/dbginfo-stepper.C M stackwalk/src/frame.C M stackwalk/src/framestepper.C M stackwalk/src/framestepper_pimple.h M stackwalk/src/linux-aarch64-swk.C M stackwalk/src/linux-swk.C M stackwalk/src/linux-x86-swk.C M stackwalk/src/linuxbsd-swk.C M stackwalk/src/linuxbsd-x86-swk.C M stackwalk/src/steppergroup.C M stackwalk/src/sw_pcontrol.C M stackwalk/src/symlookup.C M stackwalk/src/walker.C M stackwalk/src/x86-swk.C M symtabAPI/src/Object-elf.C M symtabAPI/src/Symtab.C M symtabAPI/src/dwarfWalker.C M symtabAPI/src/emitElf.C M symtabAPI/src/emitElfStatic-x86.C M symtabAPI/src/parseStab.C Log Message: ----------- make printf format and argument types match (#1082) Commit: 3e6dfbb1f37f04603403bbc8a70bdb88fdcf7a67 https://github.com/dyninst/dyninst/commit/3e6dfbb1f37f04603403bbc8a70bdb88fdcf7a67 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/h/Annotatable.h M common/src/Annotatable.C M common/src/MappedFile.C M common/src/addrtranslate-sysv.C M common/src/arch-x86.C M common/src/dyn_regs.C M common/src/linuxKludges.C M dwarf/src/dwarfExprParser.C M dyninstAPI/src/BPatch.C M dyninstAPI/src/BPatch_addressSpace.C M dyninstAPI/src/BPatch_binaryEdit.C M dyninstAPI/src/BPatch_flowGraph.C M dyninstAPI/src/BPatch_image.C M dyninstAPI/src/BPatch_process.C M dyninstAPI/src/BPatch_snippet.C M dyninstAPI/src/Relocation/Transformers/Movement-adhoc.C M dyninstAPI/src/StackMod/StackAccess.C M dyninstAPI/src/StackMod/StackModChecker.C M dyninstAPI/src/addressSpace.C M dyninstAPI/src/ast.C M dyninstAPI/src/baseTramp.C M dyninstAPI/src/binaryEdit.C M dyninstAPI/src/codegen-aarch64.C M dyninstAPI/src/codegen-power.C M dyninstAPI/src/codegen.C M dyninstAPI/src/debug.C M dyninstAPI/src/dynProcess.C M dyninstAPI/src/emit-x86.C M dyninstAPI/src/function.C M dyninstAPI/src/hybridCallbacks.C M dyninstAPI/src/hybridInstrumentation.C M dyninstAPI/src/hybridOverwrites.C M dyninstAPI/src/image.C M dyninstAPI/src/inst-aarch64.C M dyninstAPI/src/inst-x86.C M dyninstAPI/src/instPoint.C M dyninstAPI/src/parse-cfg.C M dyninstAPI/src/pcEventHandler.C M dyninstAPI/src/pcEventMuxer.C M dyninstAPI/src/registerSpace.C M dyninstAPI/src/stackwalk-x86.C M dyninstAPI/src/unix.C M dyninstAPI_RT/src/RTcommon.c M instructionAPI/h/Result.h M instructionAPI/src/InstructionDecoder-x86.C M parseAPI/src/BoundFactCalculator.C M parseAPI/src/BoundFactData.C M parseAPI/src/IA_x86.C M parseAPI/src/JumpTableIndexPred.C M parseAPI/src/SymtabCodeSource.C M parseThat/src/ipc.C M parseThat/src/utils.C M proccontrol/h/PCErrors.h M proccontrol/src/generator.C M proccontrol/src/int_thread_db.C M proccontrol/src/irpc.C M proccontrol/src/linux.C M proccontrol/src/mmapalloc.C M proccontrol/src/process.C M proccontrol/src/procset.C M proccontrol/src/resp.C M proccontrol/src/response.C M proccontrol/src/x86_process.C M stackwalk/src/aarch64-swk.C M stackwalk/src/analysis_stepper.C M stackwalk/src/callchecker-IAPI.C M stackwalk/src/dbginfo-stepper.C M stackwalk/src/frame.C M stackwalk/src/framestepper.C M stackwalk/src/framestepper_pimple.h M stackwalk/src/libstate.C M stackwalk/src/linux-aarch64-swk.C M stackwalk/src/linux-ppc-swk.C M stackwalk/src/linux-swk.C M stackwalk/src/linux-x86-swk.C M stackwalk/src/linuxbsd-swk.C M stackwalk/src/linuxbsd-x86-swk.C M stackwalk/src/ppc-swk.C M stackwalk/src/procstate.C M stackwalk/src/steppergroup.C M stackwalk/src/sw_pcontrol.C M stackwalk/src/symlookup.C M stackwalk/src/symtab-swk.C M stackwalk/src/walker.C M stackwalk/src/x86-swk.C M stackwalk/src/x86-wanderer.C M symtabAPI/src/Object-elf.C M symtabAPI/src/dwarfWalker.C M symtabAPI/src/emitElf.C Log Message: ----------- make printf format and argument signedness match (#1082) Commit: b94c3dcb5cabf8482df9d939c93d4ddff9cdb8dd https://github.com/dyninst/dyninst/commit/b94c3dcb5cabf8482df9d939c93d4ddff9cdb8dd Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dwarf/h/dwarfResult.h M dyninstAPI/h/StackMod.h M parseAPI/h/CFG.h M symtabAPI/h/Function.h M symtabAPI/h/Type.h M symtabAPI/src/dwarfWalker.h Log Message: ----------- make destructor virtual if a virtual method exist (#1082) Commit: 6684b593e4816f4457c2c4157a08aa8558ed4a7e https://github.com/dyninst/dyninst/commit/6684b593e4816f4457c2c4157a08aa8558ed4a7e Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/h/BPatch_binaryEdit.h Log Message: ----------- delete unnecessary ambiguous forward class decl (#1082) Commit: ce90af082c53796965fd170a5484330a8c6551ae https://github.com/dyninst/dyninst/commit/ce90af082c53796965fd170a5484330a8c6551ae Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M symtabAPI/src/Type.C Log Message: ----------- fix possible sprintf buffer overflow (#1082) - made buffer big enough hold maximum output to prevent overflow - changed to snprintf to prevent overflow - commented out code as buffer was only used in a commented out debug print Commit: 0750d46a36797af1f0e9e1ea5f38f6506dca5336 https://github.com/dyninst/dyninst/commit/0750d46a36797af1f0e9e1ea5f38f6506dca5336 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M instructionAPI/src/InstructionDecoder-aarch64.C Log Message: ----------- remove always true || sub-expression (#1082) Commit: 0b0d3d20b6f0499832e7d5d5b14e53072b9185de https://github.com/dyninst/dyninst/commit/0b0d3d20b6f0499832e7d5d5b14e53072b9185de Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M stackwalk/src/aarch64-swk.C Log Message: ----------- fix ambiguous type name warning (#1082) struct pair vs std::pair: made the struct name its typedef'd name Commit: e869f1203e6ee156a0125900e09ece435a1aa261 https://github.com/dyninst/dyninst/commit/e869f1203e6ee156a0125900e09ece435a1aa261 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/BPatch_object.C M dyninstAPI/src/codegen-aarch64.C M dyninstAPI/src/emit-aarch64.C M dyninstAPI/src/inst-aarch64.C M dyninstAPI/src/parse-aarch64.C M dyninstAPI_RT/src/RTlinux.c M dyninstAPI_RT/src/RTstatic_ctors_dtors-aarch64.c M stackwalk/src/aarch64-swk.C Log Message: ----------- fix unused vars/params/funcs on aarch64 (#1082) Commit: 4764c1813d3f4840e9252b7a020d30066c3da14a https://github.com/dyninst/dyninst/commit/4764c1813d3f4840e9252b7a020d30066c3da14a Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI_RT/src/RTstatic_ctors_dtors-aarch64.c Log Message: ----------- add missing #include <assert.h> (#1082) Commit: 18e2b4cff1c9d477732c56d2106fb11db0d61f61 https://github.com/dyninst/dyninst/commit/18e2b4cff1c9d477732c56d2106fb11db0d61f61 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI_RT/src/RTthread-aarch64.c Log Message: ----------- do not discard volatile type qualifier in cast (#1082) Commit: 093b544b4e2785f2417eb5dd6cacaf26b07f0d97 https://github.com/dyninst/dyninst/commit/093b544b4e2785f2417eb5dd6cacaf26b07f0d97 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI_RT/src/RTheap-linux.c Log Message: ----------- fgetc returns an int not a char (#1082) Commit: be7906d2e370af92eab5c14655b04df393a934f4 https://github.com/dyninst/dyninst/commit/be7906d2e370af92eab5c14655b04df393a934f4 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M stackwalk/src/aarch64-swk.C Log Message: ----------- replace if stmt with identical branches with then stmt (#1082) Commit: 416250dbc87e9270b82fbf046bed918977d91cc3 https://github.com/dyninst/dyninst/commit/416250dbc87e9270b82fbf046bed918977d91cc3 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/inst-aarch64.C M dyninstAPI/src/parse-power.C Log Message: ----------- fix variable signedness (#1082) Commit: 177779a6071cc5be6c3a8bf39757f06d5626402f https://github.com/dyninst/dyninst/commit/177779a6071cc5be6c3a8bf39757f06d5626402f Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/inst-aarch64.C Log Message: ----------- eliminate conversion of NULL to non-pointer type (#1082) Commit: 7af89d0a042727f876a6b93f62348cb3f5cf7f70 https://github.com/dyninst/dyninst/commit/7af89d0a042727f876a6b93f62348cb3f5cf7f70 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/inst-aarch64.C M dyninstAPI/src/inst-power.C Log Message: ----------- fix int to void* cast if sizeof(int)<sizeof(void*) (#1082) Commit: 5a5764cf80ba63ae40ae8be90ebc91843450c85a https://github.com/dyninst/dyninst/commit/5a5764cf80ba63ae40ae8be90ebc91843450c85a Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/codegen-aarch64.C Log Message: ----------- add missing default to switch statement (#1082) Commit: 2622b50b6571b966215e55f69968185e43a8f632 https://github.com/dyninst/dyninst/commit/2622b50b6571b966215e55f69968185e43a8f632 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/h/Annotatable.h M common/h/Edge.h M common/h/Node.h M dataflowAPI/h/Absloc.h M dataflowAPI/h/stackanalysis.h M instructionAPI/h/Operand.h M symtabAPI/src/dwarfWalker.h Log Message: ----------- make method noexcept, so noexcept expr can be true (#1082) Commit: 399dba27507ab2350a6ea23988e1e289a1fe124e https://github.com/dyninst/dyninst/commit/399dba27507ab2350a6ea23988e1e289a1fe124e Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/h/Annotatable.h M common/src/linuxHeaders.h M common/src/singleton_object_pool.h M dynC_API/src/C.l M dynC_API/src/lex.dynC.C M parseThat/src/parseThat.C M parseThat/src/strlist.h Log Message: ----------- annotate malloc-like functions (#1082) Commit: 4c570d0819a8c42d0cb3d433415798fd8e889413 https://github.com/dyninst/dyninst/commit/4c570d0819a8c42d0cb3d433415798fd8e889413 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M instructionAPI/h/Result.h M instructionAPI/src/InstructionDecoder-amdgpu-vega.C Log Message: ----------- make implicit double promotions explicit (#1082) Commit: 4c5582350cee11821953d27458670099049cd0c8 https://github.com/dyninst/dyninst/commit/4c5582350cee11821953d27458670099049cd0c8 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/src/arch-x86.h M dyninstAPI/src/emit-x86.C Log Message: ----------- eliminate logical op warning (#1082) gcc warns about clauses to a logical operation being identical. They are identical comparisons but the values are from different macros and/or enum values. Commit: 939b127da5d84ad8052a3acdd378a115fe78f348 https://github.com/dyninst/dyninst/commit/939b127da5d84ad8052a3acdd378a115fe78f348 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/h/Buffer.h M common/h/DynAST.h M common/h/Edge.h M common/h/Graph.h M common/h/IBSTree.h M common/h/MachSyscall.h M common/h/Node.h M common/h/concurrent.h M common/src/NodeIterator.h M common/src/addrRange.h M common/src/addrtranslate-sysv.C M common/src/addrtranslate-sysv.h M common/src/arch-aarch64.h M common/src/arch-power.h M common/src/singleton_object_pool.h M dataflowAPI/h/Absloc.h M dataflowAPI/h/AbslocInterface.h M dataflowAPI/h/SymEval.h M dataflowAPI/h/slicing.h M dataflowAPI/h/stackanalysis.h M dwarf/h/dwarfResult.h M dynC_API/h/snippetGen.h M dyninstAPI/h/BPatch_Set.h M dyninstAPI/h/BPatch_instruction.h M dyninstAPI/h/BPatch_object.h M dyninstAPI/h/BPatch_process.h M dyninstAPI/h/BPatch_snippet.h M dyninstAPI/h/BPatch_sourceObj.h M dyninstAPI/h/BPatch_type.h M dyninstAPI/src/BPatch_collections.h M dyninstAPI/src/BPatch_object.C M dyninstAPI/src/LinearVariable.h M dyninstAPI/src/MemoryEmulator/memEmulatorStub.h M dyninstAPI/src/Parsing.h M dyninstAPI/src/Relocation/CFG/RelocBlock.h M dyninstAPI/src/Relocation/CFG/RelocEdge.h M dyninstAPI/src/Relocation/CFG/RelocGraph.h M dyninstAPI/src/Relocation/CFG/RelocTarget.h M dyninstAPI/src/Relocation/CodeBuffer.h M dyninstAPI/src/Relocation/CodeTracker.h M dyninstAPI/src/Relocation/Springboard.h M dyninstAPI/src/Relocation/Transformers/Instrumenter.h M dyninstAPI/src/Relocation/Transformers/Modification.h M dyninstAPI/src/Relocation/Transformers/Movement-adhoc.h M dyninstAPI/src/Relocation/Transformers/Movement-analysis.C M dyninstAPI/src/Relocation/Transformers/Movement-analysis.h M dyninstAPI/src/Relocation/Transformers/Transformer.h M dyninstAPI/src/Relocation/Widgets/ASTWidget.h M dyninstAPI/src/Relocation/Widgets/CFWidget.h M dyninstAPI/src/Relocation/Widgets/CallbackWidget.h M dyninstAPI/src/Relocation/Widgets/InsnWidget.h M dyninstAPI/src/Relocation/Widgets/InstWidget.h M dyninstAPI/src/Relocation/Widgets/PCWidget.h M dyninstAPI/src/Relocation/Widgets/RelDataWidget.h M dyninstAPI/src/Relocation/Widgets/StackModWidget.h M dyninstAPI/src/Relocation/Widgets/Widget.h M dyninstAPI/src/StackMod/StackLocation.h M dyninstAPI/src/addressSpace.h M dyninstAPI/src/ast.h M dyninstAPI/src/binaryEdit.h M dyninstAPI/src/block.h M dyninstAPI/src/codeRange.h M dyninstAPI/src/codegen-power.C M dyninstAPI/src/dynProcess.h M dyninstAPI/src/emit-aarch64.h M dyninstAPI/src/emit-power.h M dyninstAPI/src/emit-x86.h M dyninstAPI/src/emitter.h M dyninstAPI/src/frame.h M dyninstAPI/src/function.h M dyninstAPI/src/hybridAnalysis.h M dyninstAPI/src/image.C M dyninstAPI/src/inst.h M dyninstAPI/src/mapped_object.h M dyninstAPI/src/parse-cfg.h M dyninstAPI/src/patch.h M dyninstAPI/src/registerSpace.h M dyninstAPI/src/syscallNotification.h M instructionAPI/h/Expression.h M instructionAPI/src/InstructionDecoder-amdgpu-vega.h M instructionAPI/src/InstructionDecoder-power.h M parseAPI/h/CFGFactory.h M parseAPI/h/CFGModifier.h M parseAPI/h/CodeObject.h M parseAPI/h/Location.h M parseAPI/h/LockFreeQueue.h M parseAPI/h/ParseCallback.h M parseAPI/src/IA_power.h M parseAPI/src/JumpTableIndexPred.h M parseAPI/src/ProbabilisticParser.h M parseAPI/src/StackTamperVisitor.h M patchAPI/h/PatchCFG.h M patchAPI/h/PatchCallback.h M patchAPI/h/PatchMgr.h M patchAPI/h/PatchModifier.h M patchAPI/h/Point.h M patchAPI/h/Snippet.h M patchAPI/src/ParseCallback.h M proccontrol/h/ProcessSet.h M proccontrol/src/int_process.h M stackwalk/src/dbgstepper-impl.h M symtabAPI/h/Aggregate.h M symtabAPI/h/StringTable.h M symtabAPI/h/Symtab.h M symtabAPI/h/Type.h M symtabAPI/h/Variable.h M symtabAPI/src/Object-elf.C M symtabAPI/src/Object-elf.h M symtabAPI/src/Object.h M symtabAPI/src/dwarfWalker.h Log Message: ----------- remove ';' after in-class method definitions (#1082) Commit: fd3567176e755d598273dcb31316606c5b924a40 https://github.com/dyninst/dyninst/commit/fd3567176e755d598273dcb31316606c5b924a40 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/h/BPatch_snippet.h Log Message: ----------- make constructor public so class is usable (#1082) Commit: 38afc8faf45d8c9f4edda589e21f10966c95810d https://github.com/dyninst/dyninst/commit/38afc8faf45d8c9f4edda589e21f10966c95810d Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M instructionAPI/src/InstructionDecoder-aarch64.C M instructionAPI/src/InstructionDecoder-amdgpu-vega.C Log Message: ----------- add missing initializer braces (#1082) Should be OK without extra brace, but early C++11 required it, and some versions of gcc complain with -Wmissing-braces Commit: ec3d7b89dc90f68dcc7ccbc307ddcf10d59e5064 https://github.com/dyninst/dyninst/commit/ec3d7b89dc90f68dcc7ccbc307ddcf10d59e5064 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M symtabAPI/src/emitElfStatic-ppc64.C Log Message: ----------- fix broken bool expression that was always true (#1082) Commit: 8d61e486f96526ef3fc1669f28c56650ab6a841e https://github.com/dyninst/dyninst/commit/8d61e486f96526ef3fc1669f28c56650ab6a841e Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/image.C M dyninstAPI/src/inst-power.C M dyninstAPI/src/stackwalk-ppc.C M stackwalk/src/ppc-swk.C M symtabAPI/src/emitElfStatic-ppc64.C Log Message: ----------- make printf format and argument types match (#1082) Commit: d8ae055477cf0354435d73d927e9396eb8a02706 https://github.com/dyninst/dyninst/commit/d8ae055477cf0354435d73d927e9396eb8a02706 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/codegen-power.C M dyninstAPI/src/image.C M dyninstAPI/src/inst-power.C Log Message: ----------- remove unused variables (#1082) Commit: 1383ae62a545219fe4eb8f56191fb76bde92c9de https://github.com/dyninst/dyninst/commit/1383ae62a545219fe4eb8f56191fb76bde92c9de Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M stackwalk/src/ppc-swk.C Log Message: ----------- use unused variable to correct code (#1082) Commit: cca9c166adf1c746b1af13323b836319ff41830e https://github.com/dyninst/dyninst/commit/cca9c166adf1c746b1af13323b836319ff41830e Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M symtabAPI/src/emitElfStatic-ppc64.C Log Message: ----------- fix potentially uninitialized variable warning (#1082) Commit: ad2b3e85144b94391424c5b7b887034e9918634e https://github.com/dyninst/dyninst/commit/ad2b3e85144b94391424c5b7b887034e9918634e Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI_RT/src/RTlinux.c Log Message: ----------- fix out of bounds array access (#1082) use ->uc_mcontext.regs->nip which follows the 32 array entries of the same type in ->uc_mcontext.regs->gpr[] instead of access gpr[32] which happens to work Commit: 7c71346f1d8fe0ff3660e2c556bd4b940023e1be https://github.com/dyninst/dyninst/commit/7c71346f1d8fe0ff3660e2c556bd4b940023e1be Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dyninstAPI/src/codegen-power.C Log Message: ----------- fix duplicate branch condition by removing branch (#1082) Commented out else if condition and then block. else if condition happens to be identical to the if condition due to constant identifiers having the same value. The tests should be made more specific to distinguish the two cases or the commented out else if should be removed. Commit: 75a88f08fddbfa8d8964825ad40435d76706ff57 https://github.com/dyninst/dyninst/commit/75a88f08fddbfa8d8964825ad40435d76706ff57 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M dynC_API/src/C.y M dynC_API/src/dynC.tab.C Log Message: ----------- fix buffer overflow (#1082) Commit: 3ccf5245f9c0e22bfdcda5344d10e17d8e3187ea https://github.com/dyninst/dyninst/commit/3ccf5245f9c0e22bfdcda5344d10e17d8e3187ea Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/src/arch-power.h Log Message: ----------- fix deprecated implicit assignment operator (#1082) remove (default equivalent) copy constructors, so all implicitly generated operators are defaulted. THey should all be default or explicitly defined Commit: 5d90c9d988f4443e29d9ea92109c8ce4b54043a2 https://github.com/dyninst/dyninst/commit/5d90c9d988f4443e29d9ea92109c8ce4b54043a2 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M common/CMakeLists.txt M instructionAPI/CMakeLists.txt Log Message: ----------- adjust large frame threshold for specific sources (#1082) Commit: 5a2b418a7e810ab930e2342a5e024c6c89ac77c3 https://github.com/dyninst/dyninst/commit/5a2b418a7e810ab930e2342a5e024c6c89ac77c3 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M instructionAPI/src/InstructionDecoder-aarch64.C Log Message: ----------- prevent maybe uninitialized warning (#1082) This is a false positive with -Og. Initialize the variable to 0 to prevent the warning. The value is only used if a boolean is true and if it is true then the value is set. Commit: 1d43ecc12c134d3ee6037f951e3f5a83b7f6ae6c https://github.com/dyninst/dyninst/commit/1d43ecc12c134d3ee6037f951e3f5a83b7f6ae6c Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M symtabAPI/h/Type.h Log Message: ----------- fix possible null 'this' pointer dereference (#1082) If a nullptr is passed to the second parameter of the Field class's constructor then there is a nullptr this pointer dereference for the boost::shared_ptr that is the second parameter as the reshare() method is called. The second parameter is a nullptr in BPatch_type::getComponents is the type is and enum type. This warning is only diagnosed with -Og (and maybe -O1). Commit: 699c285ce1b56dc0d763d6073e7c051442603b95 https://github.com/dyninst/dyninst/commit/699c285ce1b56dc0d763d6073e7c051442603b95 Author: James A. Kupsch <kup...@cs.wisc.edu> Date: 2021-08-04 (Wed, 04 Aug 2021) Changed paths: M instructionAPI/src/InstructionDecoder-aarch64.C Log Message: ----------- fix possibly uninitialized variables (#1082) make method always assign the parameters even in the event of errors, and check for errors and do not use them if an error occurred. This diagnostic only occurs with -O3. In practice this is a false positive as this should not occur with a valid arm instruction. Compare: https://github.com/dyninst/dyninst/compare/d36317eb5c40...699c285ce1b5 _______________________________________________ Dyninst-api mailing list Dyninst-api@cs.wisc.edu https://lists.cs.wisc.edu/mailman/listinfo/dyninst-api