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

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

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.

Clean up bad merge

CMake 2.6 compatibility fixes.

More messages during build

Warning cleanup

Merge branch 'findMain'

isTailcall fix: fallthroughs are not tailcalls

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

dyninstAPI: Expand powerpc support in image::findMain

First, this function needs to respect the ELF entry address, rather than
assuming that entry is at the start of the .text section.  This was
already handled in x86, so we just need the same in powerpc.

Second, we need to add ppc64 support to the section which currently
handles ppc32.  This requires dereferencing .opd indirection, and ppc64
also uses a TOC-based load which the current dataflowAPI code can't
handle.  For this case, just use a simple InstructionAPI eval with the
known TOC r2 register.

dyninstAPI: Let image::findMain inspect PIE too

Position-independent executables (PIE) are generally indistinguishable
from any other shared object, but we still need to find main() in these
when they're stripped.  One clue is the presence of INTERP, which PIEs
will always have.  (Some DSOs also have INTERP, but this is rare.)

Fix AddressSpace initialization order

Fix another unused parameter

Better warning fix for unused parameters

Commit f284f869b0b4 turned unused parameters into unused "dummy"
variables, which gcc 4.8 still warns about.  Using the parameter in a
void expression is a more reliable workaround.

Fix to handleCondDirExits

Removed nonexistent API call in ParseAPI::Block documentation

Finish the plumbing to ensure that added libraries including the RTlib get DT_NEEDED = SONAME.

Warning fix

Build fix

fileName() returns SONAME if present now, for proper DT_NEEDED updates when rewriting

Re-transpose arguments, because CMake 2.6 is dumb

Proper fix for building RTlib under CMake 2.6

Two more CMake 2.6 fixes: proper order of arguments, and only use LINK_PRIVATE where it's supported.

Fixed a CMake 2.6 bug where it wouldn't recognize libdwarf_imp as a target (and thus wouldn't add a libelf_imp dependency) if we weren't building our own libdwarf. Also cleaned up indentation (as in, it exists now in these CMake files).

Increase BG/Q attach timeout to five minutes.

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

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.

Change __FILE__ to FILE__ prints in stackwalker to shorten logs

Fix PC/BGQ crashes on attach timeout

Implement PC memory usage interface on BGQ

Fix issue with optimization options not being used in RT library build

WIP for testing: properly track springboard locations persistently and per-process, rather than only persistent tracking of traps, globally

Clean up proccontrol debugging prints.  We now print base filenames rather than full paths in printfs

Clean RT library cmake cache before configuring to work around cmake bug where re-running cmake forgets the CMAKE_INSTALL_PREFIX

Fix procset iterator: we can't necessarily initialize the container from NULL. Had better be able to default-construct it though.

symtabAPI: Trim AObject and Object constructors.

Neither of these classes should ever be copied, so declare the standard
copy constructors as private, without implementation; operator= as well.
Remove a few other unimplemented constructors too.

Note: The updates to Object-nt were made by inspection only and should
be checked.  Object-coff and Object-xcoff were already using obsolete
AObject constructors; if dead they should be considered for removal.

Initialize thrd in a new FrameNode ctor

Reported by Coverity as UNINIT_CTOR.

Initialize and free PSetFeatures::memset

Reported by Coverity as UNINIT_CTOR

stackwalk: convert some stderr chatter to sw_printf

symtabAPI: Fix Coverity UNINIT_CTOR errors

symlite: Fix Coverity UNINIT_CTOR errors

stackwalk: Fix Coverity UNINIT_CTOR errors

proccontrol: Fix Coverity UNINIT_CTOR errors

parseAPI: Fix Coverity UNINIT_CTOR errors

instructionAPI: Fix Coverity UNINIT_CTOR errors

elf: Fix Coverity UNINIT_CTOR errors

dataflowAPI: Fix Coverity UNINIT_CTOR errors

common: Fix Coverity UNINIT_CTOR errors

Use 'delete' for objects created with 'new'

These two instances were incorrectly using free().

Reported by clang-analyzer as Memory Error : Bad deallocator.

Fix accidental switch fallthrough

These were reported by Coverity as MISSING_BREAK, and it seems true -
they should not be falling through to the next case.  Add breaks.

Annotate intentional switch fallthrough

These were reported by Coverity as MISSING_BREAK, but they're
intentionally falling through to the next case.  Coverity will infer
this intent if there's a comment in the last line instead of a break.

Reimplement Linux P_copy with fstream

Coverity reported RESOURCE_LEAK for the fds on error paths, and I see it
also doesn't check write() return value.  It's much easier to just let
fstream do the buffer management and IO.

Note: P_copy doesn't appear to actually be used anywhere, and the former
destination mode 0 was also suspicious.  Maybe it should be removed.

Reimplement getLinuxMaps in real C++

Coverity reported a few OVERFLOW weaknesses in the previous buffer-happy
code.  The whole function is much simpler using fstream to read maps.

emitElf: Fix possible sign-extension errors

When you have an expression type "<uint64_t> = <uint16_t> * <uint16_t>",
it sadly takes a detour into signed values.  The operands will be
promoted to int, then multiplied, then sign-extended to finally fill
uint64_t.  If the multiplication result is greater than 0x7fffffff,
overflowing int, then the top 32 bits of the uint64_t will all be 1s.
(That's if the compiler doesn't blow up the int overflow as undefined.)

The easiest solution is to forcefully promote the operands to the target
size and sign before multiplying.

Reported by Coverity as SIGN_EXTENSION.

stackwalk: Don't leak CallChecker instances

Both AnalysisStepperImpl and WandererHelper create CallCheckers in their
constructors, so they should delete in their deconstructors.

Reported by Coverity as CTOR_DTOR_LEAK.

proccontrol: Flatten up_notify as a member of int_notify

In the int_notify ctor, up_notify is newed, but this member is never
deleted.  Since this object never changes, it doesn't need to be a
pointer at all, so make it a direct member.

However, int_notify is a singleton that's never deleted, so the overall
leakiness isn't really changed here.

Reported by Coverity as CTOR_DTOR_LEAK.

symlite: Use pread rather than lseek save/restore

In SymLite::getInterpreterName, it was doing a SEEK_CUR, SEEK_SET,
read(), then SEEK_SET back to the original offset.  This can all be
replaced by using pread() to read from any offset without affecting the
actual file offset.

The former lseeks triggered Coverity CHECKED_RETURN & NEGATIVE_RETURNS.

common: Fix a sizeof error in calloc/realloc

An allocation resulting in (anno_list_t*) should not use the pointer
size, but rather sizeof(anno_list_t).  It just happened to be harmless
here because anno_list_t is itself typedef void*, so the same size.

Reported by Coverity as SIZEOF_MISMATCH.

patchAPI: Remove 'during' before destroying it

In PatchParseCallback::remove_edge_cb, if it has pe->points_.during,
then this is both destroyed and removed from PatchFunctions.  However,
PatchCallback::destroy will delete the point if not batching, and
PatchFunction::remove dereferences it, so destroy must come after.

Reported by Coverity as USE_AFTER_FREE.

proccontrol: Avoid dereferencing a deleted 'this'

In linux_process::preTerminate, a debug message asking, "Is this irony?"
has its own irony by calling getPid() when 'this' was just deleted.

Reported by Coverity as USE_AFTER_FREE.

symlite: Don't use a delete pointer

In SymElfFactory::closeSymbolReader, if 'ser' isn't in open_symelfs,
then it is deleted, so the function should return without using 'ser'
any more.

Reported by Coverity as USE_AFTER_FREE.

proccontrol: Fix a copy-paste typo in ppc_process::readInsnForSS

After iterating over all of map mm_for_ss, this function will create a
new mem_response for the request.  If that reports isReady() right away,
then the memcpy incorrectly copies from the previous map iterator, which
is now exhausted (==end()).  This appears to be a copy-paste typo of the
memcpy from within the for loop, and this memcpy should use new_resp.

Reported by Coverity as INVALIDATE_ITERATOR.

elf: Fix iteration in the destructor

In Elf_X::~Elf_X, the loop over elf_x_by_fd returns after it finds an
erases its element.  The loop over elf_x_by_ptr should do the same.
It's actually an error to continue the for loop with ++iter, because
map::erase() invalidates its iterator.

Reported by Coverity as INVALIDATE_ITERATOR.

Assert that dwarf_loclist_from_expr_a returns cnt==1

Coverity reports ARRAY_VS_SINGLETON in DwarfWalker::decodeExpression,
because the address of a singleton, &descs, is passed with cnt to
decodeLocationListForStaticOffsetOrAddress, which treats it as an array.
Without knowledge of cnt, the analyzer thinks this could be source of
memory corruption.

In fact, dwarf_loclist_from_expr_a always sets listlen (cnt) to 1 on
success.  If we assert this, then the analyzer can tell that the
singleton "array" bound is fine.

dataflowAPI: Correct an assertion test

Based on surrounding context, it seems clear that this assert meant to
test equality, not the result of assignment.

Reported by Coverity as ASSERT_SIDE_EFFECT.

We were double-adding field sizes to structs when they came from DWARF. This fixes that; we still need to add field sizes in cases where users create their own data types.

Indirect tail calls shouldn't count as unresolved control flow for instrumentation purposes.