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

General improvements #27

Merged
merged 6 commits into from
Nov 13, 2023
Merged

General improvements #27

merged 6 commits into from
Nov 13, 2023

Commits on Nov 13, 2023

  1. generator: tc: add missing post-unload callback 

    BPF_NETFILTER program type support introduced 2-step BPF program
loading, but the post-unload callback for TC wasn't available.

Signed-off-by: Quentin Deslandes <[email protected]>
    @qdeslandes
    qdeslandes committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    38e978b View commit details
    Browse the repository at this point in the history

  2. generator: tc: removed erroneous BPF_EXIT_INSN() 

    BPF_EXIT_INSN() is added to the codegen by the rules, or the program
itself. The flavour doesn't need to add a BPF_EXIT_INSN().

Signed-off-by: Quentin Deslandes <[email protected]>
    @qdeslandes
    qdeslandes committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    04fb939 View commit details
    Browse the repository at this point in the history

  3. build: add asan and ubsan to bpfilter.pc, and change install dir 

    Install bpfilter.pc into CMake's libdir instead of datadir.

Add ubsan and asan to bpfilter.pc in debug mode.

Signed-off-by: Quentin Deslandes <[email protected]>
    @qdeslandes
    qdeslandes committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    fcf1ff5 View commit details
    Browse the repository at this point in the history

  4. lib: forward-declard ipt-related structures in bpfilter.h 

    ipt_getinfo and ipt_get_entries are both used (as a pointer) but
undefined in bpfilter.h.

Forward-declare both structure to prevent compilation warning.

Signed-off-by: Quentin Deslandes <[email protected]>
    @qdeslandes
    qdeslandes committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    87ed424 View commit details
    Browse the repository at this point in the history

  5. generator: log the program's flavor during generation 

    Signed-off-by: Quentin Deslandes <[email protected]>
    @qdeslandes
    qdeslandes committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    140e66c View commit details
    Browse the repository at this point in the history

  6. shared: only check the content size if the response is successful 

    Do not read bf_response.data_len in bf_recv_response() if
bf_response.status is not successful.

Signed-off-by: Quentin Deslandes <[email protected]>
    @qdeslandes
    qdeslandes committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    24e3d9c View commit details
    Browse the repository at this point in the history