1. 10 Jan, 2014 2 commits
  2. 22 Oct, 2013 3 commits
  3. 11 Oct, 2013 2 commits
  4. 19 Mar, 2013 1 commit
    • Petr Machata's avatar
      The same work needs to be done on attach as when _start is hit · df2c88ce
      Petr Machata authored
      - Extract the logic to a separate function, called from open_pid and
        entry_breakpoint_on_hit.
      
      - Since there is no other way to get on dyn_addr in open_pid anyway,
        simply look for the main library, and read it there.  That means we
        don't need to track that information at entry_breakpoint anymore,
        and we can get rid of struct entry_breakpoint.
      df2c88ce
  5. 11 Mar, 2013 2 commits
    • Petr Machata's avatar
      Fix tracing Thumb mode on ARM · f9d93c50
      Petr Machata authored
      - Drop what amounts to global variable thumb_mode from struct process
      - Thumb mode is encoded to address.  That means that the code in
        callstack_push_symfunc and insert_breakpoint has to assume that
        what comes from get_return_address is mangled, and needs to go
        through creating the breakpoint and then looking at its address.
        That's awkward, and we should come up with a better API.
      f9d93c50
    • Petr Machata's avatar
      Drop useless argument from breakpoint_clone · e5035529
      Petr Machata authored
      e5035529
  6. 08 Mar, 2013 3 commits
    • Petr Machata's avatar
      Change DICT_FIND to DICT_FIND_{REF,VAL} · 98ff309c
      Petr Machata authored
      - In many places, DICT_FIND was awkward to use, as it required the use
        of pointers even in cases that the dictionary already stored
        pointers.  For those cases, add DICT_FIND_VAL.
      
      - Rename the original DICT_FIND to DICT_FIND_REF to make the
        disctinction clear.  That is useful if the dictionary holds actual
        structures.
      98ff309c
    • Petr Machata's avatar
      Fixes for dict code · 2718e3fd
      Petr Machata authored
      2718e3fd
    • Petr Machata's avatar
      Rewrite dict module to be more in line with vect · d7e4ca82
      Petr Machata authored
      It's now a bit more strongly typed, can grow and shrink on demand, and has
      a more complete interface.
      It uses open addressing scheme to store hashes.
      d7e4ca82
  7. 14 Feb, 2013 1 commit
  8. 17 Dec, 2012 1 commit
    • Petr Machata's avatar
      struct Process becomes struct process · 929bd57c
      Petr Machata authored
      This is for consistency with other structures, and ultimately with
      Linux coding style.  The typedef ("Process") was dropped as well for
      this reason.
      
      This opportunity was used to fix coding style around the impacted
      lines.
      929bd57c
  9. 30 Oct, 2012 1 commit
  10. 27 Oct, 2012 1 commit
    • Petr Machata's avatar
      Add proc_find_symbol, use it to find corresponding symbols for clones · 165b566a
      Petr Machata authored
      The code in breakpoint_clone was buggy--it looked through old_proc's
      symbols.  It was also needlessly verbose, proc_find_symbol uses library
      key to find the right library.  Instead use the new interface and call
      it on new_proc to find the new symbol.
      
      We also didn't relink symbols in stack elements of a cloned process in
      process_clone.  Use proc_find_symbol for this as well.
      165b566a
  11. 25 Oct, 2012 1 commit
  12. 03 Sep, 2012 1 commit
    • Sedat Dilek's avatar
      breakpoints.c: handle_event.c: Fix compilation on MIPS · 5165cf30
      Sedat Dilek authored
      From my conversation with Petr Machata:
      
      >>> [ breakpoints.c ]
      
      > >>> MIPS arch has no own "breakpoints.c".
      > >>> IIRC sth, was wrong with "list_of_symbols" in
      > >>> enable_all_breakpoints()... "Process" has no member "list_of_symbols",
      > >>> so cut off the mips-ifdef part.
      > >>> Just testing compiles or not.
      > >>
      > >> Cutting this actually seems reasonable.  There was a similar ifdef for
      > >> PPC, and that is now gone as well (and PPC works fine).  I believe
      > >> ltrace now handles delaying breakpoint enablement.
      > >>
      > >>> [ handle_event.c ]
      
      >>> Same as for breakpoints.c cut off the mips-ifdef, seen
      >>> "list_of_symbols" errors in handle_breakpoint().
      >>
      >> Hmm, that seems like an implementation of the delayed start.  I think
      >> this can be removed as well, with the same rationale as above.
      
      These two should probably be in a patch by themselves.
      Signed-off-by: default avatarSedat Dilek <[email protected]>
      5165cf30
  13. 29 Aug, 2012 3 commits
  14. 15 May, 2012 1 commit
  15. 30 Apr, 2012 1 commit
  16. 25 Apr, 2012 1 commit
    • Petr Machata's avatar
      Replace some uses of error with fprintf · cc0e1e4b
      Petr Machata authored
      error is not standard so it has no business being used in generic code.
      The linux-gnu back end is useful for android, and that doesn't have that
      interface either.
      cc0e1e4b
  17. 24 Apr, 2012 1 commit
  18. 20 Apr, 2012 1 commit
  19. 19 Apr, 2012 2 commits
  20. 18 Apr, 2012 11 commits