On Wed, Oct 17, 2018 at 09:41:25AM -0400, Peng Hao wrote: > > From: Peng Hao <peng.hao2@xxxxxxxxxx> > > FILE pointer variable f is opened but never closed. > > Signed-off-by: Peng Hao <peng.hao2@xxxxxxxxxx> > --- > tools/testing/selftests/bpf/trace_helpers.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tools/testing/selftests/bpf/trace_helpers.c b/tools/testing/selftests/bpf/trace_helpers.c > index cabe2a3..a3d1dac 100644 > --- a/tools/testing/selftests/bpf/trace_helpers.c > +++ b/tools/testing/selftests/bpf/trace_helpers.c > @@ -41,6 +41,7 @@ int load_kallsyms(void) > syms[i].name = strdup(func); > i++; > } > + fclose(f); looks good. please resubmit cc-ing netdev, so it gets into patchwork and we can apply it properly. Please change subj to [PATCH bpf-next] and add a Fixes tag. Thanks!