Clean up library linkage

- Add -Wl,--no-undefined to gnu link flags, so it's a link-time error if
  any library doesn't declare everything it uses.

- Add a 'target_link_private_libraries' function to set LINK_PRIVATE, so
  dependencies aren't implicitly propagated.  Each library must thus
  explicitly declare its link needs.  This only works for CMake 2.8.7+,
  but falls back gracefully otherwise.

- All libraries are now whittled down to link only against their direct
  dependencies to satisfy used symbols.  Check this with "ldd -r -u".
  The only extras now are libelf in libdynDwarf, because libdwarf
  doesn't always link libelf itself, and libm is always added by g++.

Fully separate symtab and symlite

The cmake LIGHTWEIGHT_SYMTAB option is documented to toggle between
symtab and symlite for ParseAPI, ProcControl, and Stackwalker.  However,
these libraries didn't fully commit one way or the other, so we were
left in limbo with both loaded most of the time.  This patch fixes them
to be fully separated into just symtab or just symlite.

- All CMakeLists now respect LIGHTWEIGHT_SYMTAB when setting target
  libraries, so they'll fail to link if something is wrong.

- ParseAPI now builds with symlite, but note that DyninstAPI won't work
  yet in this mode, because it assumes full SymtabCodeSource.

- ProcControl's getElfReader() now works with either symtab or symlite.

- Stackwalker's DebugStepperImpl::GetReg is refactored a bit to work
  without symtab, and the latent cap_stackwalker_use_symtab is now
  converted to WITH_SYMTAB_API and enabled.

cmake: Look for plain libiberty after libiberty_pic

On Debian <= wheezy, the PIC library is named libiberty_pic.a.
On Debian >= jessie and all RHEL and Fedora, libiberty.a is PIC.

Merge remote-tracking branch 'origin/master' into v8.2

Conflicts:
cmake/optimization.cmake
dyninstAPI/src/BPatch_process.C

Update handleAddSub to properly handle signed ints

decodeImmediate: for 64-bit operands, all immediates are sign-extended

Tailcall fix: a jump to the current function's entry point is not a tail call.

Updated RelocBlock::determineSpringboards to filter tailcalls in addition to regular calls

Cleanup unused functions in the SpringboardBuilder class

Update springboard generation to take function and priority
information into account when checking for springboard conflicts.

We weren't ensuring that our section's alignment was at least equal to
    its data's alignment. Fixed. (Part 2, from Bill.)

Make type checking less aggressive--no type info != void

Windows fixes & debug log cleanup

Null check symtabreader's symtab before returning it, as we can't use a symtabreader if it failed to open a symtab

Fix some warning spam

If we won't be able to generate a stack unprotect, fall through to the library injection mechanism that might still work

One more debug statement

More debugging output changes

Debug tracing for the loadlibrary infmalloc

Debug output, don't push this

Test whether boost 1.40 on Ubuntu 10 works cleanly now

We weren't ensuring that our section's alignment was at least equal to its data's alignment. Fixed.

Types changed in the linkMap and ppc32 didn't build cleanly; fixed.

Fixes for BGQ functionality to kick threads out of locks during emergency shutdowns

Fix BGQ compilation errors with new Boost based mutex/condvar system

Fix problem with storing pretty names for inlined subroutines

Default to Boost_NO_BOOST_CMAKE because of boost bugs

Force boost >= 1.41.0

Add link dependency on Elf_X since we actually use it

Don't add link directories; trust boost_libraries to get things right

Remove last occurence of shared_dynamic_cast, which is deprecated in favor of dynamic_pointer_cast

Add boost system dependency, as thread doesn't take care of it automatically

Thread library doesn't always work, but libraries should now

Allow us to link against boost shared libraries

Merge branch 'master' of git.dyninst.org:/pub/dyninst

Pass any custom lib and include install locations to RT library

dyninstAPI: Squash library callbacks during exec bootstrap

When a process execs and the underlying PCProcess changes, there's
nothing that BPatch or the user can reasonably do with library
callbacks on the old process, so squash them.  Once the PCProcess gets
updated in BPatch::registerExecExit, callbacks will resume as normal.

Make sure SIGUSR2 is cleared from masked signals before using it in PC.

Build fixes for Linux

Clean up some missed exports & uninitialized variables.

Handle Windows' need to process .reloc and adjust the load address appropriately to handle ASLR-enabled binaries.

Massive simplifications to RTlib initialization process: run it once and only once via the library's on-load code, and adjust anything we need to by setting variables later.

Update to use boost mutexes and condition variables

Bump versions everywhere to 8.2.0

This is not to imply that 8.2.0 is finished or frozen yet, but it's still
useful to indicate that development is beyond the 8.1 branch.

Also declare DYNINST_8_1_2 and SW_VERSION_8_1_2 for consistency with the
v8.1.2 tag, which was not directly merged into the master branch.

cmake: set flags for RELWITHDEBINFO

This should match the usual cmake target RelWithDebInfo, sans "UG".

Merge branch 'master' of git.dyninst.org:/pub/dyninst

Fix Dyninst/ProcControlAPI hang when running iRPCs in multi-process environments.

dyninstAPI: Limit cerr messages to debug mode

A few places in AddressSpace were writing directly to cerr, when they
ought to save messages for debug-enabled cerr only.

dyninstAPI: Check for termination after removing tracedSyscalls

It's possible to get markExited() while handling events for the
tracedSyscalls_->remove* calls, which makes the following pcProc_->detach()
a NULL dereference.  Simply calling isTerminated() protects this case.

dyninstAPI: Continue after loadLibrary

If BPatch_process::loadLibrary() had to stop the process at its start,
then it should also continue execution before returning.  This was changed
in commit 3faa4175c8100, a large commit with no explanation for this
particular change, possibly an accident.

Use const string& parameters for stats

The stats functions don't need mutable parameters, so a const& suffices
just fine.  Also convert all the fixed string names of counters and
timers from string literals to std::string in dyninstAPI and parseAPI.

Even when stats were disabled, there was a fair amount of time spent
just preparing the string object for stats calls.  The optimization of
this patch shows 5-10% reduction in parse time on large binaries.

Increase Instruction::raw_insn_T.small_insn size

By making this union .small_insn a uintptr_t, so it's the same size as
.large_insn, we can avoid some memory allocation without taking any more
memory in Instruction itself.  (PPC instructions are always 32-bit, so
there it's left as unsigned int.)

With this optimization on x86_64, the number of large_insn allocations
required is greatly reduced.  Many instructions are more than 4 bytes,
but it's less common to have more than 8 bytes.

common: Skip get_immediate_operand without any imm

For x86 image::findMain, we blindly look for the first call from the
entry address, then grab the immediate operand from the preceding
instruction.  Sometimes that instruction may not even have any immediate
operands, and the heuristic fails with a message on stderr like:

  arch-x86.C[5602]:  invalid immediate size 0 in insn

This now occurs with some ET_DYN binaries that are allowed by commit
44205dbd2d90, when those are really DSOs and not PIE.  However, this
could conceivably also happen for any executable that doesn't fit the
heuristic pattern.

This patch quietly returns 0 from get_immediate_operand when there are
no operands, and image::findMain will treat this as an invalid address.

Merge branch 'jistone/coverity'

Merge branch 'jistone/platform'

Conflicts:
stackwalk/src/aix-swk.C
symtabAPI/doc/3-Examples.tex

Merge branch 'master' of ssh://follis/p/paradyn/development/bill/head/dyninst

Merge branch 'ssj_boost_mutex' of ssh://git.dyninst.org/pub/dyninst

Conflicts:
common/src/dthread-unix.C

More windows fixes

More windows fixes

Merge branch 'master' of git.dyninst.org:/pub/dyninst

Remove prints that were triggering when disassembling libraries under findMain

Allow Linux to build while implementing memory usage interface

Merge branch 'master' of git.dyninst.org:/pub/dyninst

Fix issue with dwarf range offsets being used with incorrect base addresses.

Moved r_debug assert to the Linux runDYNINSTBaseInit()

Install public dataflowAPI headers

Fix accumulate() in alias.isAlias case to record transfer func
whose target is the current target (rather than alias.target).

Handle cyclic dependencies in fixpoint algorithm in stackanalysis

Merge branch 'master' of git.dyninst.org:/pub/dyninst

Conflicts:
CMakeLists.txt

Updated parseAPI documentation PDF (missed in previous commit)

Fixed SymtabAPI documentation example 3 to use current API

Export r_debug from the RTlib on Linux and
verify that it's properly present before trying to use.

dyninstAPI: Fix a debug vsnprintf length

The vsnprintf call is writing to an offset in the fixed errbuf, so it
needs to subtract that offset from the length told to vsnprintf.

Reported by coverity as OVERRUN.

dyninstAPI: Pad the fixed x86 instructions

The arrays used to generate illegal & trap instructions can be static
const, but still the analyzer doesn't see that the instruction decoder
won't actually read past their ends.  It's not a big deal to just pad
both illegalRep[] and trapRep[] to 8 bytes to be sure.

Reported by coverity as OVERRUN.

symtabAPI: Initialize locals in read_except_table_gcc2

These can't actually be unset unless the address size was something other
than 4 or 8, but the static analyzer doesn't know that invariant.

Reported by coverity as UNINIT.

symtabAPI: Error out on unknown createRelocationSections

It shouldn't happen, but to the static analyzer it looks like an odd
relocation combination in createRelocationSections will lead to using
unset local variables.  Add a quick check to make sure we got something.

Reported by coverity as UNINIT.

stackwalk: Fix result logic in getInitialFrameImpl

The former "result = !result || getRegValue(...)" ends up toggling any
failed states.  A "result = result && ..." instead maintains failure.

Reported by coverity as UNINIT.

instructionAPI: Don't try to delete in Instruction copy ctor

The Instruction copy constructor was trying to check the previous m_size
and delete previous m_RawInsn before copying, but there isn't any previous
state because this is a new object.

Reported by coverity as UNINIT.

dyninstAPI: Check the bounds of boundsArray

When merging contiguous ranges of boundsArray, the code failed to make
sure that the array has anything in it yet.

Reported by coverity as UNINIT.

dyninstAPI: Make sure dominator nullNode always looks initialized

There's a mutual dependency between dominatorCFG and dominatorBB in how
nullNode is initialized.  To the analyzer, it looks like the uninitialized
pointer is used in BB, even though CFG straightens it out immediately.
Letting nullNode start NULL clears up the confusion.

Reported by coverity as UNINIT.

common: Initialize SymReader structs to zero

Reported by coverity as UNINIT at various use sites.

symtabAPI: Fix coverity UNINIT_CTOR errors

patchAPI: Fix coverity UNINIT_CTOR errors

dyninstAPI: Fix coverity UNINIT_CTOR errors

dyninstAPI: Remove mapped_object::getModulePart()

The last in-tree caller was removed in commit f2869360c0f0, over 16
years ago, when it was shared_object.  Since mapped_object is now an
internal class, it should be perfectly safe to remove this.

This function was flagged by cppcheck for mismatched allocation, using
delete on strdup memory.

Clean up bad merge

CMake 2.6 compatibility fixes.

More messages during build

Warning cleanup

Merge branch 'findMain'

Remove remnant .cvsignore files

isTailcall fix: fallthroughs are not tailcalls

Remove HP/UX remnant

Remove remnants of IA64

common: Note FreeBSD in Types.h

Purge AIX support

Remove remnants of Sparc and Solaris

Remove remnants of Alpha and OSF

Remove remnants of MIPS, including IRIX and WinCE

dyninstAPI: Set image::findMain's findings as LOCAL/INTERNAL

When we look for new symbols in image::findMain, it's not really the
intent to make these global/exported symbols.  It doesn't matter so much
for live processes, but these shouldn't be exported in rewritten
binaries.  This patch sets them as LOCAL and INTERNAL instead.

Note, this is only for Linux and FreeBSD, but Windows may want it too.

Merge branch 'master' of git.dyninst.org:/pub/dyninst

Conflicts:
CMakeLists.txt