On Tue, 2024-03-05 at 10:51 +0100, Daniel Borkmann wrote:
[...]
> > diff --git a/tools/testing/selftests/bpf/prog_tests/struct_ops_autocreate.c b/tools/testing/selftests/bpf/prog_tests/struct_ops_autocreate.c
> > new file mode 100644
> > index 000000000000..883f938d518c
> > --- /dev/null
> > +++ b/tools/testing/selftests/bpf/prog_tests/struct_ops_autocreate.c
> > @@ -0,0 +1,76 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +
> > +#include <test_progs.h>
> > +#include "struct_ops_autocreate.skel.h"
> > +
> > +static void cant_load_full_object(void)
> > +{
> > + struct struct_ops_autocreate *skel;
> > + char *log;
> > + int err;
> > +
> > + skel = struct_ops_autocreate__open();
> > + if (!ASSERT_OK_PTR(skel, "struct_ops_autocreate__open"))
> > + return;
> > +
> > + if (start_libbpf_log_capture())
> > + goto cleanup;
>
> nit: This cleanup path triggers freeing of uninitialized log pointer.
Thank you for noticing, I will double check all tests and post v4
after collecting some additional feedback.
[...]
