Re: [PATCH 56/63] perf tests: Fix out of bounds memory access

From: Greg Kroah-Hartman
Date: Mon Dec 16 2019 - 11:20:32 EST


On Mon, Dec 16, 2019 at 09:37:02PM +0530, Naresh Kamboju wrote:
> This patch merged into stable-rc tree and perf build failed on OE for
> Linaro builds for 5.3, 4.19, 4.14 and 4.9.
> Please find build error logs here,
>
> tests/backward-ring-buffer.c: In function 'test__backward_ring_buffer':
> tests/backward-ring-buffer.c:147:2: warning: implicit declaration of
> function 'evlist__close'; did you mean 'perf_evlist__close'?
> [-Wimplicit-function-declaration]
> evlist__close(evlist);
> ^~~~~~~~~~~~~
> perf_evlist__close
> tests/backward-ring-buffer.c:147:2: warning: nested extern declaration
> of 'evlist__close' [-Wnested-externs]
> tests/backward-ring-buffer.c:149:8: warning: implicit declaration of
> function 'evlist__open'; did you mean 'perf_evlist__open'?
> [-Wimplicit-function-declaration]
> err = evlist__open(evlist);
> ^~~~~~~~~~~~
> perf_evlist__open
> tests/backward-ring-buffer.c:149:8: warning: nested extern declaration
> of 'evlist__open' [-Wnested-externs]
> perf/1.0-r9/recipe-sysroot/usr/lib/python2.7/config/libpython2.7.a(posixmodule.o):
> In function `posix_tmpnam':
> /usr/src/debug/python/2.7.15-r1/Python-2.7.15/Modules/posixmodule.c:7648:
> warning: the use of `tmpnam_r' is dangerous, better use `mkstemp'
> perf/1.0-r9/recipe-sysroot/usr/lib/python2.7/config/libpython2.7.a(posixmodule.o):
> In function `posix_tempnam':
> /usr/src/debug/python/2.7.15-r1/Python-2.7.15/Modules/posixmodule.c:7595:
> warning: the use of `tempnam' is dangerous, better use `mkstemp'
> perf/1.0-r9/perf-1.0/perf-in.o: In function `test__backward_ring_buffer':
> perf/1.0-r9/perf-1.0/tools/perf/tests/backward-ring-buffer.c:147:
> undefined reference to `evlist__close'
> perf/1.0-r9/perf-1.0/tools/perf/tests/backward-ring-buffer.c:149:
> undefined reference to `evlist__open'
>
> Full log can be found at,
> https://ci.linaro.org/view/lkft/job/openembedded-lkft-linux-stable-rc-5.3/DISTRO=lkft,MACHINE=hikey,label=docker-lkft/72/consoleText
> https://ci.linaro.org/view/lkft/job/openembedded-lkft-linux-stable-rc-4.19/DISTRO=lkft,MACHINE=intel-corei7-64,label=docker-lkft/378/consoleText
> https://ci.linaro.org/view/lkft/job/openembedded-lkft-linux-stable-rc-4.14/DISTRO=lkft,MACHINE=intel-corei7-64,label=docker-lkft/675/consoleText
> https://ci.linaro.org/view/lkft/job/openembedded-lkft-linux-stable-rc-4.9/DISTRO=lkft,MACHINE=intel-corei7-64,label=docker-lkft/753/consoleText
>

Good catch, thanks, I'll go drop it from all of these queues.

greg k-h