On Tue, Jul 28, 2020 at 10:47 PM Song Liu <s...@kernel.org> wrote:
>
> On Tue, Jul 28, 2020 at 9:54 PM Andrii Nakryiko <andr...@fb.com> wrote:
> >
> > Check that link is NULL or proper pointer before invoking
> > bpf_link__destroy().
> > Not doing this causes crash in test_progs, when cg_storage_multi selftest
> > fails.
> >
> > Cc: YiFei Zhu <zhuyi...@google.com>
> > Fixes: 3573f384014f ("selftests/bpf: Test CGROUP_STORAGE behavior on shared
> > egress + ingress")
> > Signed-off-by: Andrii Nakryiko <andr...@fb.com>
>
> Acked-by: Song Liu <songliubrav...@fb.com>
>
> btw: maybe we can move the IS_ERR() check to bpf_link__destroy()?
Yeah, given how common this mistake seems to be, that wouldn't be a bad idea.
>
> > ---
> > .../bpf/prog_tests/cg_storage_multi.c | 42 ++++++++++++-------
> > 1 file changed, 28 insertions(+), 14 deletions(-)
> >
[...]
