Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(preload): experimental LD_PRELOAD env hook #28

Closed
wants to merge 12 commits into from
Closed

feat(preload): experimental LD_PRELOAD env hook #28

wants to merge 12 commits into from

Commits on Oct 18, 2024

  1. feat(preload): initial version of LD_PRELOAD env hook

    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    6045281 View commit details
    Browse the repository at this point in the history

  2. chore(preload): restructure sources, improve Makefile

    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    13cb2ed View commit details
    Browse the repository at this point in the history

  3. feat(preload): override GNU secure_getenv as well

    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    5ec2a6b View commit details
    Browse the repository at this point in the history

  4. test(preload): improve smoke test script

    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    7e83ac7 View commit details
    Browse the repository at this point in the history

  5. fix(preload): only malloc once per environment variable name 

    (instead of once per getenv call)
    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    dc11c89 View commit details
    Browse the repository at this point in the history

  6. feat(preload): support x86_64 as well as arm64

    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    350eb3c View commit details
    Browse the repository at this point in the history

  7. test(preload): test against glibc and musl 

    Also, convert all shell scripts to POSIX-compliant sh scripts and apply
most shellcheck suggestions.
    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    1ffd3f3 View commit details
    Browse the repository at this point in the history

  8. perf(preload): only look up original function once

    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    13b2fee View commit details
    Browse the repository at this point in the history

  9. refactor(preload): init map in constructor instead of lazy init

    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    17ab3c4 View commit details
    Browse the repository at this point in the history

  10. test(preload): build once, then test against both libc flavors 

    The previous build & test setup was invalid: So far we have built
_different_ libdash0envhook.so binaries for the LD_PRELOAD hook per
libc flavor. With this, the binary works in the target system.

But actually this approach is not not possible in the real world. We
can set one binary as LD_PRELOAD when instrumenting a container and we
do not know which libc flavor it is ahead of time. Thus we ultimately
need to build a libdash0envhook.so binary that works independently of
which libc flavor the target system uses.

This commit only fixes the test setup. The binary does not yet support
both libc flavors. This will be handled in a follow up commit.

[skip ci]
    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    46256ab View commit details
    Browse the repository at this point in the history

  11. test(preload): run tests for both CPU architectures 

    [skip ci]
    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    7190db6 View commit details
    Browse the repository at this point in the history

  12. chore(preload): move dash0-instrumentation -> instrumentation

    @basti1302
    basti1302 committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    38aa8fc View commit details
    Browse the repository at this point in the history