Re: [PATCH bpf v1 2/2] selftests/bpf: assert link info uprobe_multi count & path_size if unset

[Andrii Nakryiko]

On Wed, Oct 9, 2024 at 3:07 PM Tyrone Wu <wudevelops@xxxxxxxxx> wrote:
>
> Add assertions in `bpf_link_info.uprobe_multi` test to verify that
> `count` and `path_size` fields are correctly populated when the fields
> are unset.
>
> This tests a previous bug where the `path_size` field was not populated
> when `path` and `path_size` were unset.
>
> Signed-off-by: Tyrone Wu <wudevelops@xxxxxxxxx>
> ---
>  tools/testing/selftests/bpf/prog_tests/fill_link_info.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/tools/testing/selftests/bpf/prog_tests/fill_link_info.c b/tools/testing/selftests/bpf/prog_tests/fill_link_info.c
> index f3932941bbaa..a38cf2a999fe 100644
> --- a/tools/testing/selftests/bpf/prog_tests/fill_link_info.c
> +++ b/tools/testing/selftests/bpf/prog_tests/fill_link_info.c
> @@ -417,6 +417,13 @@ verify_umulti_link_info(int fd, bool retprobe, __u64 *offsets,
>         if (!ASSERT_NEQ(err, -1, "readlink"))
>                 return -1;
>
> +       memset(&info, 0, sizeof(info));
> +       err = bpf_link_get_info_by_fd(fd, &info, &len);

if (!ASSERT_OK(err, "link_get_info"))
    return -1;

?

Other than this, LGTM.

pw-bot: cr

> +
> +       ASSERT_EQ(info.uprobe_multi.count, 3, "info.uprobe_multi.count");
> +       ASSERT_EQ(info.uprobe_multi.path_size, strlen(path) + 1,
> +                 "info.uprobe_multi.path_size");
> +
>         for (bit = 0; bit < 8; bit++) {
>                 memset(&info, 0, sizeof(info));
>                 info.uprobe_multi.path = ptr_to_u64(path_buf);
> --
> 2.43.0
>