Enforce detection of libiberty (#1099)

This correctly requires that libiberty be present and removes building it from source. Both of these were bugs introduced by 08097f23.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

remove executable flag from .dyninst_heap section (#1096)

This prevents the library/object with this code from having a segment violates
the W^X security property for segments as the current setting resulted in
segment with both the WRITE and EXECUTE flags set.  The function
mark_heaps_exec() restores execute protection to just the heap arrays.

fix compiler warnings to work with clang (#1092)

- detect warning flag test failure on clang correctly by updating the failure
  regular expression to include clang's unknown warning flag message

- fix preprocessor macro tests for scoped attributes in C

update optimization (-Og) and debug flags (-g3) (#1084)

use the C11 standard for C code in Dyninst (#1086)

fix broken cast of a char literal to pointer (#1090)

The code should have been a string literal not a char literal.

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.

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).

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.

adjust large frame threshold for specific sources (#1082)

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

fix buffer overflow (#1082)

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.

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

fix potentially uninitialized variable warning (#1082)

use unused variable to correct code (#1082)

remove unused variables (#1082)

make printf format and argument types match (#1082)

fix broken bool expression that was always true (#1082)

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

make constructor public so class is usable (#1082)

remove ';' after in-class method definitions (#1082)

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.

make implicit double promotions explicit (#1082)

annotate malloc-like functions (#1082)

make method noexcept, so noexcept expr can be true (#1082)

add missing default to switch statement (#1082)

fix int to void* cast if sizeof(int)<sizeof(void*) (#1082)

eliminate conversion of NULL to non-pointer type (#1082)

fix variable signedness (#1082)

replace if stmt with identical branches with then stmt (#1082)

fgetc returns an int not a char (#1082)

do not discard volatile type qualifier in cast (#1082)

add missing #include <assert.h> (#1082)

fix unused vars/params/funcs on aarch64 (#1082)

fix ambiguous type name warning (#1082)

struct pair vs std::pair:  made the struct name its typedef'd name

remove always true || sub-expression (#1082)

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

delete unnecessary ambiguous forward class decl (#1082)

make destructor virtual if a virtual method exist (#1082)

make printf format and argument signedness match (#1082)

make printf format and argument types match (#1082)

add compiler annotation to printf-like functions (#1082)

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.

remove assert(this) as 'this' should never be null (#1082)

remove obvious null pointer dereference (#1082)

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

make cmp function object operator() a const func (#1082)

STL in C++17 requires this

make Boost and TBB include dirs be system includes (#1082)

This eliminates warnings from gcc 11 and from -Wduplicated-branches

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.

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

eliminate switch case fall through warnings (#1082)

explicit base class initialization in constructor (#1082)

remove default argument from lambda (#1082)

not valid in C++11 (is in C++14 and later).

remove non-C++ compound literal (#1082)

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.

fix deprecated implicit copy constructor if dtor (#1082)

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.

fix illegal in C empty brace initialization (#1082)

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.

fix discard qualifiers: make char* -> const char* (#1082)

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.

remove excess semicolons as reported by -pedantic (#1082)

fix overflow warning for 0x90 assigned to a char (#1082)

fix illegal function pointer to void* compare (#1082)

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)

remove non-C++ variable length arrays (#1082)

make printf format and argument types match (#1082)

fix shadow identifier warnings (#1082)

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.

miscellaneous compiler warning cleanups (#1082)

eliminate switch case fall through warnings (#1082)

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

add missing break statements (#1082)

compute num array elements instead of fixed values (#1082)

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

remove emptyString static members (#1082)

delete unnecessary .DS_Store file (#1082)

Make Dyninst buildable with Clang (#1021)

* Use 'typename' in using declarations

* Ignore .machine directives under clang

https://github.com/dyninst/dyninst/pull/631#issuecomment-537659696

* Fix #497

Add x86 xsavec instruction (#1074)

Cleanup orphaned code (#1064)

* Remove 'dummy.C' from dyninstAPI/CMakeLists.txt

* Remove unused scripts/buildstamp

* Remove unused files from parseAPI/CMakeLists.txt

* Remove instructionAPI/src/MainDocPage.C

The usage of this file was removed by 5c88375a7 in 2016.

* Remove stackwalk/src/firstparty.C

This file was abandoned by 44507e6b0 in 2013.

* Remove BPatch_dependenceGraph{Edge,Node}.{C,h}

These files were abandoned by bbc9ff655 in 2009.

* Remove dyninstAPI/src/dummy.C

This file was abandoned by 44507e6b0 in 2013.

* Remove dyninstAPI/src/Relocation/Widgets/CFPatch.C

This has been abandoned since at least 91246b9ca in 2011.

* Remove dyninstAPI/src/InstructionCache.C

This file was never compiled. It seems to be an outdated copy of dataflowAPI/src/InstructionCache.C.

* Remove dyninstAPI/src/BPatch_templates.C

This was orphaned by 44507e6b0 in 2013.

* Remove symtabAPI/src/staticRewrite.C

This was orphaned by 44507e6b0 in 2013.

* Remove dyninstAPI_RT/src/RTheap-svr4.c

This was orphaned by 44507e6b0 in 2013.

* Remove dyninstAPI_RT/src/RTmutatedBinary{_ELF}.c

These were orphaned sometime before 44507e6b0 in 2013. The git history doesn't show when. I'm assuming it was lost in the transition from CVS.

* Remove dyninstAPI_RT/src/RTthread-power.c

This was a holdover from the rs6000 AIX support.

* Remove common/src/tunableConst.h

This has never been part of the build system.

* Remove common/src/List.{h,C}

These were orphaned by 44507e6b0 in 2013.

* Remove common/src/auxvtypes.{h,C}

These should have been removed by cccad8ee.

* Remove common/src/int64iostream.{h,C}

These were orphaned by 44507e6b0 in 2013.

* Remove unused 'timing.h' interface

* Remove unused dyninstAPI/src/clock.h

* Remove unused common/src/mmtimer.c

* Remove common/src/debugOstream.{h,C}

This was abandoned by by 65bd8055 in 2008

* Remove unused common/src/Time.{h,C}

* Remove unused common/src/fraction.{h,C}

* Remove symtabAPI/src/Symtab-deprecated.C

There are no symbols defined by this file.

* Remove common/src/Ident.{h,C}

This was orphaned by 44507e6b0 in 2013.

* Remove parseThat/src/dyninstCompat.v4.C

This should have been removed as part of 63d791419.

* Remove common/src/addrtranslate-static.C

This was orphaned by 44507e6b0 in 2013.

* Remove dyninstAPI/src/asmExterns.h

This was abandoned by 5019a9c5a in 2007.

* Remove dyninstAPI/src/BPatch_typePrivate.h

This was abandoned by b3105c49c in 2008.

* Remove dyninstAPI/src/codebuffer.h

This was added by 936238d in 2010, but never added to the build system.

* Remove parseAPI/h/GraphAdapter.h

This was abandoned by 44507e6b0 in 2013.

* Remove common/src/cstring.h

This was abandoned by f4613dcfe in 2008.

* Remove common/src/keylist.h

This was abandoned by f4613dcfe in 2008.

* Remove common/src/klist.h

This was abandoned by f4613dcfe in 2008.

* Remove common/src/Line.h

This was abandoned by 4b1956db in 2013.

* Remove stackwalk/h/local_var.h

This was abandoned by 44507e6b0 in 2013.

* Remove common/src/LockTraits.h

Was added by a38bacc5 in 2017, but never added to the build system.

* Remove common/src/matherr.h

This was added by 4b1956db in 2013, but was never part of the build system.

* Remove dyninstAPI/src/ntDL.h

This was abandoned by 5c88375a in 2016.

* Remove dyninstAPI/src/NTTypes.h

This was abandoned by 5c88375a78 in 2016.

* Remove dataflowAPI/src/Operations.h

This was abandoned by 5c88375a in 2016.

* Remove common/h/stl_utils.h

This should have been removed by cccad8ee.

* Remove common/src/string-regex.{h,C}

This was abandoned by f1b11229 in 2008.

* Remove stackwalk/{src,h}/sw_c.{h,C}

This was abanded by 44507e6 in 2013.

* Remove elf/src/SystemTap.{h,C}

This was abandoned by 44507e6 in 2013.

* Remove common/src/triple.h

This was abandoned by 5c88375a7 in 2016.

* Remove dataflowAPI/rose/x86InstructionSemantics-notemplate.h

This was added by 8d6626c in 2010, but was never part of the build system.

* Remove unused man files from common/docs

* Remove declaration of 'friend operator<<' from BPatch_basicBlockLoop.h

Its implementation was removed by f3b69f6c8 in 2005.

* Remove declaration of 'friend operator<<' from BPatch_flowGraph.h

Its implementation was removed by f3b69f6c8 in 2005.

* Remove declaration of 'friend operator<<' from BPatch_frame.h

It introduced by 954046f99 in 2005, but never implemented.

* Remove declaration of 'friend operator<<' from CFG.h/Loop

It introduced by 6b2b85ee5 in 2014, but never implemented.

* Remove 'operator<<' from BPatch_sourceBlock

This was abandoned by 0d9f1455cc in 2003.

Convert TRAMP_*_OFFSET macros to functions (#1073)

The macro for the TRAMP_*_OFFSET_32 cases expanded into `({assert(0); 0;})` which is not legal C++. It was allowed by gcc by extension. Converting them to functions preserves the behavior without code changes. They are made `inline` to prevent ODR violations.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Add x86_64 xrstor instruction (#1070)

Fix insertion operators in BPatch and Symtab (#1069)

* SymtabAPI::Variable

* Provide an exported operator<< for BPatch_basicBlock

* Provide an exported operator<< for SymtabAPI::Symbol

* Fix operator<< in Symtab/Aggregate and Variable

The existing version never worked because they didn't have exported
symbols and, if they had, led to ambiguous overloads because both
operators were in the global namespace and Variable is a child of
Aggregate. Making operator<< a member function for both allows ADL to
disambiguate.

Remove Module::getAllVariables (#1066)

It was declared by c848409ec9 in 2009, but never defined.

Remove valueAdded subdirectory completely (#1065)

The directory was not removed fully by #1063.

Remove valueAdded subdirectory (#1063)

The function wrapper example has been moved to the examples repository (https://github.com/dyninst/examples/pull/21). The shared memory example is no longer relevant.

Add DWARF4 base type entry encodings to symtabAPI::typeScalar (#1059)

* Add error message when unable to find size of a baseType

* Inline the default constructor

* Use delegating construtor

* Rename isSigned_ to is_signed

* Inline constructors

* Add new encoding properties

Add xsave instruction (#1055)

Prep for release 11.0.1 (#1053)

* Update CHANGELOG

* Bump version to 11.0.1

Find function by address, not by entry (#1047)

This was broken by #837

Fix implicit operand decoding of x86 instructions on non-x86 architectures (#1044)

fix Position Independent Executable (PIE) handling (#1042)

* fix Position Independent Executable (PIE) handling

All functions that determine if an object is a shared library and/or an
executable are now based on a single set of functions in Object-elf.
This make all the functions in the various Dyninst libraries and classes
consistent and also makes Dyninst correctly handle PIE executables.
These functions in Object-elf are based on functions found in elfutils's
elfclassify program (with small improvements to better determine if a
shared library is also an executable).   The new functions in Object-elf
include:

    * isOnlyExecutable()
    * isExecutable()
    * isSharedLibrary()
    * isOnlySharedLibrary()
    * isLoadable()
    * isDebugOnly()
    * isLinuxKernelModule()

Symtab exposes some of these functions through its public API.  isExec()
now correctly handles PIE executables, and two new functions were added
to determine if the underlying file is a shared library, an executable,
or both.  These function are:

    * isExec()            (isExecutable() and not isSharedLibrary())
    * isExecutable()
    * isSharedLibrary()

* remove unnecessary assert

Fixing calling `dwarf_getabbrevcode` and refactoring debug logging in common (#1037)

* 1. Fix segfault in dwarf_getabbrevcode. Do not call dwarf_getabbrevcode when the input is DWARF_END_ABBREV
2. Refactor common deubg logging to avoid evaluating logging arguments when logging is not enabled

* Remove call to dwarf_getabbrevcode().

Co-authored-by: Sasha Nicolas <sasha@cs.wisc.edu>

Search for 'version.h' when determining version for TBB (#1041)

2020.1.1 renamed 'tbb_stddef.h' to 'version.h'

Fix data races and asserts discovered on Power  (#1038)

* Use thread-safe source and target edge copy methods to iterate edges during finliazation

* Use std::map::find rather than [] to avoid corruption of a power decoding table

* More fixes for parsing Power binaries:
1. convert asserts in slicing to debug logging
2. copy block edge list during finalization when determining function's block list

* Convert asserts to debug logging when setting a function's return status

These asserts indeed represent errors where parsing has gone wrong in some fashion,
but these errors are not necessarily fatal overall.

Remove the use of C++17's structured binding (#1036)

We only require C++11 at this time.

Skip parsing of blocks whose code buffer is null (#1033)

Remove debug printing that causes large output (#1029)

* remove debug cout

* remove unused backup code

* add common_parsing_printf as the way of outputing parsing related information in common

* change the usage of cout/cerr to common_parsing_printf

* remove the reuse of environmental variable as suggested

Fix catch block parsing (#1030)

The start address of a catch block is specified with a 2, 4, or 8 byte value.
SymtabAPI uses a 8-byte integer and properly clip the value when reading it.
However, there is an add to the read value, which will cause the final result
to wrap around under its specified size (such as 4-byte), but not under 8-byte.