Fix bug in `expr_clone`

On allocation failure of nhls for EXPR_OP_CALL2,
EXPR_OP_UP, EXPR_OP_CALL1, the correct behaviour,
that is returning -1, was only effective if

  node->kind == EXPR_OP_CALL2 and node->u.call.own_rhs

This change implements the correct behaviour.

It also solves a warning when compiling with gcc4.9 that
broke integration when compiling with -Werror: the compiler
was hurt to use nrhs that he suspected (wrongly) to be
uninitialised.

Change-Id: I184fea5e121d38fa1df5d0a8680edf51e9fff1a8
Signed-off-by: José Bollo <jose.bollo@open.eurogiciel.org>

packaging: update to 0.7.91

Change-Id: I2e383896f948a19dcd5ba07b8e7f60b6de0c5c74
Signed-off-by: Chanho Park <chanho61.park@samsung.com>

do not error on warnings

resetting manifest requested domain to floor

 do not depend on dejagnu

Update to 0.7.2

Update to 0.7.2

do not run tests

clean up spec

demangle lib.cstdlib

add packaging

Guard a dwarf_prototypes.h include with a conditional

- Thanks to Kai Noda for reporting this.

Mention support of powerpc64le ELFv2 in README and NEWS

PowerPC: convert ELFv2 conditionals form preprocessor to plain conditions

- The reason being that this way, all of the code is exposed all the
  time, which should make future refactoring easier.  There's nothing
  that needs hiding, no system-specific constants or such.

Fix coding style in PowerPC's arch.h

Fix bugs in fetch backend of powerpc64le

- One bug was in a function that returned a small structure and whose
  first argument was a pointer that pointed to stack.  The old code
  assumed that meant that the structure was returned by implicit
  reference instead of in registers.

- Another was in passing large HFA's (larger than 8 elements) in
  arguments, and in returning HFA's.

Recognize powerpc64le in configure

Fix an error path problem introduced in last commit

In config files, allow whitespace between identifier and opening paren

minor whitespace-only style fix

fixed testsuite typos: ltarce -> ltrace

Add a missing license blurb

Normalize some include guards

Drop uses of #pragma once

PPC: fix PPC32 build

Compilation on PPC32 fails because STACK_FRAME_OVERHEAD is never defined
in arch.h.

Define it to 112 on that platform to restore the same behaviour as
before commit eea4ad2cce289753aaa35b4e0258a76d8f8f367c.

Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

Copy struct library.should_activate_latent on clone

- I don't think it makes actual difference, as the only consumer of this
  symbol is proc_add_library, and that's not called from proc_clone (the
  clone is done by directly iterating the library list).  But the code is
  more obviously right this way.

Fix cloning of libraries with Dwarf support

- struct library::dwfl_module was left unitialized after
  library_clone.  Garbage was kept in, which prevented proper
  initialization later, and resulted in segmentation violation.

I now always build the export list

Previously I only built the export list when tracing with -l. Since I was using
this export list to resolve aliased symbols in addition to setting breakpoints,
this aliased symbol resolution was only working with -l. I now always build the
export list to make aliased symbol resolution always work.

I now have a separate variable to control whether we should activate latent
symbols or not; previously the existence of the export list was used to make
this determination.

Furthermore populate_this_symtab() now takes an extra argument to indicate that
ONLY the export list should be filled in

un-consted some variables to build with older (<= gcc-4.6) compilers

Made activate_latent_in() iterations much more efficient

Previously activate_latent_in() iterations looked like

for(export names in lib1) // hash table iteration
{
  for(symbol names in lib2) // list iteration
  {
    if(names equal && libsym->latent)
    {
      proc_activate_latent_symbol(proc, libsym)
    }
  }
}

This is inefficient both due to the double iteration but also since iterating
over a hash table in slow (have to look through all cells, even empty ones).
This patch turns this logic into

for(symbol names in lib2) // list iteration
{
  if(name in lib1 export names && libsym->latent) // hash table lookup
  {
    proc_activate_latent_symbol(proc, libsym)
  }
}

So there's no more double iteration, and the hash iteration was turned into a
hash lookup. Much better.

removed needless #include

library_exported_names_each_alias(), library_exported_names_each() now restartable

These function now takes a *start_after, and return a pointer to the first
failing key, like the other ltrace iterators

added forgotten free(), forgotten return

clarified some int return code checking

whitespace 'if(' -> 'if ('

renamed _dtor_* and _clone_* functions to remove the leading _

DWARF prototypes are now generated with both the plain and the linkage names

If a die has a DW_AT_linkage_name, I now use it: this is required for C++ code,
in particular.

I use the plain name regardless, since sometimes the exported symbol corresponds
to the plain name, NOT the linkage name. For instance I see this on my
Debian/sid amd64 box. In its libc, the linkage name of __nanosleep is
__GI___nanosleep, but the export is __nanosleep

We now use known prototypes for all aliased symbols (same address)

Some libraries have multiple names for the same function. Prior to this patch,
it was possible to define a prototype for a symbol, and not have ltrace use it
because it saw a different symbol be called. libc is a common source of this.
For instance (on my amd64 Debian box) it defines the nanosleep symbol as both
'nanosleep' and '__GI___nanosleep', at the same address. If a calling library
calls '__GI___nanosleep', then an ltrace prototype for 'nanosleep' would not be
used, even though it should apply to this call

dict_hash_uint64: do not ignore upper 32 bits of the quantity

added hash and equality functions for uint64_t

The hash function is identical to the 32-bit signed int hash function. This
function is unideal for such extended use, but is sufficient for now

added NEWS, CREDITS info about the new DWARF prototyping code

Added manpage section giving examples of -x, -e and -l usage

Making sure to not double-examine the same DWARF CU

CREDITS update for ppc64el support

Signed-off-by: Thierry Fauck <thierry@linux.vnet.ibm.com>

In ppc-lwarx.exp, Dwarf backend may resolve atomic_add's prototype

- in which case the patterns are wrong.  Change the pattern to match
  both cases.

Support for powerpc64 arch ppc64el

Signed-off-by: Thierry Fauck <thierry@linux.vnet.ibm.com>
Add support for ppc64le proc and ELF ABIv2.
Provides support for irelative and wchar

I only build the dwarf_prototypes if we have libdw

removed needless cast

added forgotten error checking to some ltrace calls

fixed typo

I only explicitly look at sizeof(long) if it differs from sizeof(int)

If they're the same, checking for both in a switch() is a compile error

style whitespace changes

- * with name, not type
- if (blah), not if( blah )

whitespace style change

 - Tabs should be 8 characters.
 - Lines should be no longer than 80 characters
 - Pointer star belongs to the variable, not the type
 - Lines shouldn't be formatted into tables arbitrarily
 - if and while should get a space before the paren

a bit more debug printing

added a bit more diagnostic printing

fixed memory leaks

Two leaks plugged:

1. value used for array-length
2. prototype return type

fixed prototype memory leak

protolib_add_prototype() makes a copy of the prototype, so allocating a new
pointer there was an error

removed notes

removed, commented testing code

dwarf diagnostic output now controlled by -D

fixed broken diagnostic output

reworked error and memory handling

I now longer exit() on the slightest sign of trouble, nor do I leak all my heap
memory allocations

unknown pointers now return void*, not void

DWARF parser now looks for exported types

I now pass true to protolib_lookup_type(). Thus the user can now define a type
in .ltrace.conf, and it would be picked up

renamed type_hash -> type_dieoffset_hash

type_hash is no longer global

I now read the DWARF data in addition to the .conf files

Previously if a .conf file for a library was available, the corresponding DWARF
data would not be parsed. Now I use one to complement the other

If a functions is defined in a .conf, I do not parse the DWARF

Previously I would parse the DWARF data for such a function, but I wouldn't use
the result. Now I don't even parse

types defined in .conf files now take precedence over DWARF types

style whitespace

import_DWARF_prototypes() now takes JUST a lib argument

moved DWARF prototype reading to library_get_prototype()

updated error message to reflect the new dwfl use case

whitespace

Stylistic refactoring. No functional changes

I now import functions using their linkage name

This is required for C++ methods

enums are no longer assumed to have type 'int'

reworked attr_numeric() to report failures

This whole file is thus copyright me

added a disabled stub for complex float support

a bit more debug printing

style changes

- no spaces in ()
- hanging braces
- no camelCase
- collapsed long lines

debug-printing via complain()

added explicit support for some more types

- boolean constant
- long long double (as void)
- variable-length array (assuming len 0)
- union (as void)
- empty compile units

added some optional debug printing

tree dumping goes to stderr

initial prototypes-from-dwarf implementation

fixed typo: toplevel_format_lens() now uses ptr data when talking about pointers

Fix lookup of prototypes from non-leader threads

Add a couple TODO items

Improve wording on -e, -x, -l in ltrace.1

Add missing #include stdio.h

Include stdio.h in files that use fprintf()

Signed-off-by: dann frazier <dannf@debian.org>

add missing <stdint.h> include

aarch64's fetch.c uses the uintptr_t typedef defined by stdint.h without
including it. This doesn't currently cause a build failure because stdint.h
is indirectly included via proc.h.

Signed-off-by: dann frazier <dannf@debian.org>

Add --config to man, drop -X from options.c

Initialize struct process.e_machine and .e_class for secondary threads

Add some stuff to libc.so.conf

Update NEWS, README

Implement aarch64 support

- IFUNC support is not implemented, the rest works well.  The only
  other failure is in wide char functions, and that occurs on x86_64
  as well.

Set child stack alignment in trace-clone.c

- This is important on aarch64, which requires 16-byte aligned
  stack pointer.  This might be relevant on other arches as well,
  I suspect we just happened to get the 16-byte boundary in some
  cases.

Force use of SYS_open on aarch64 as well

- That system call is not implemented on aarch64, but we don't
  care, we are only calling it to see if the parameters get decoded
  properly.  So call using the "syscall" wrapper, and hard-code
  SYS_open value on aarch64, where glibc doesn't define it.

Rewrite system_calls.exp

- Port this to new style test case

- Be more clever about what syscalls were invoked from main,
  as opposed to the multitude of calls that the dynamic linker
  typically makes

- Allow some basic calls to come in XXXat variants (e.g. openat
  instead of open), such is the case on aarch64.