Skip to content

Dev#90

Merged
io-no merged 47 commits intomainfrom
dev
Aug 1, 2024
Merged

Dev#90
io-no merged 47 commits intomainfrom
dev

Conversation

@io-no
Copy link
Copy Markdown
Member

@io-no io-no commented Aug 1, 2024

This PR includes:

Hotfixes

  • A hotfix for the _background_invalid_call function, which is called when a user tries to call an unsupported libdebug function in a callback. It did not correctly handle a variadic number of (positional) arguments.
  • Better management of edge cases in the get_return_address function.
  • Solved a problem with the breakpoint disable option if called right after the installation of the breakpoint.

Performance Optimizations

  • A slight optimization in the performance and logic of breakpoints.
  • Memory access latency improvements with absolute addresses.

Additional Improvements

  • An info message is now displayed when symbols are downloaded from debuginfod.
  • Now the backtrace function has a new parameter to choose to return either a list of addresses or symbols. Additionally, a public pretty print for the backtrace has been added.
  • Rationalized the debugging levels.

Tests and Examples

  • Added use case examples to show how to use libdebug in different contexts.
  • Added benchmark tests on event handling, related results, and comparison with GDB Python API.

Documentation

  • Updated the documentation for the new logging system.

MrIndeciso and others added 30 commits July 13, 2024 23:15
Detects the bug fixed in the previous commit
Fix software breakpoint disabling
…t should contain symbols or addresses. Added also a pretty print of the backtrace. Solve #84
…mory operations

We can use the return value from the ptrace calls to judge if the absolute address was valid or not, thus skipping the /proc/pid/maps open and read
@io-no io-no requested review from Frank01001 and MrIndeciso August 1, 2024 12:37
@io-no io-no self-assigned this Aug 1, 2024
Copy link
Copy Markdown
Member

@Frank01001 Frank01001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The pull request has been reviewed and has been found compliant with specifications

@io-no io-no merged commit e9e7b7a into main Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants