mbox series

[0/3] perf tools: Address fixes

Message ID 20230316194156.8320-1-adrian.hunter@intel.com
Headers
Series perf tools: Address fixes |

Message

Adrian Hunter March 16, 2023, 7:41 p.m. UTC 
  Hi

Here are 3 small fixes resulting from a report from a
kernel test robot:

https://lore.kernel.org/oe-lkp/202303061424.6ad43294-yujie.liu@intel.com

Issues were revealed due to the use of build
option:

	EXTRA_CFLAGS="-fsanitize=undefined -fsanitize=address"


Adrian Hunter (3):
      perf symbols: Fix use-after-free in get_plt_got_name()
      perf symbols: Fix unaligned access in get_x86_64_plt_disp()
      perf tools: Avoid warning in do_realloc_array_as_needed()

 tools/perf/util/symbol-elf.c | 10 ++++++++--
 tools/perf/util/util.c       |  3 ++-
 2 files changed, 10 insertions(+), 3 deletions(-)


Regards
Adrian

Comments

Ian Rogers March 16, 2023, 9:21 p.m. UTC | #1 
On Thu, Mar 16, 2023 at 12:42 PM Adrian Hunter <adrian.hunter@intel.com> wrote:
>
> Hi
>
> Here are 3 small fixes resulting from a report from a
> kernel test robot:
>
> https://lore.kernel.org/oe-lkp/202303061424.6ad43294-yujie.liu@intel.com
>
> Issues were revealed due to the use of build
> option:
>
>         EXTRA_CFLAGS="-fsanitize=undefined -fsanitize=address"
>
>
> Adrian Hunter (3):
>       perf symbols: Fix use-after-free in get_plt_got_name()
>       perf symbols: Fix unaligned access in get_x86_64_plt_disp()
>       perf tools: Avoid warning in do_realloc_array_as_needed()

All 3:
Acked-by: Ian Rogers <irogers@google.com>

Thanks,
Ian

>  tools/perf/util/symbol-elf.c | 10 ++++++++--
>  tools/perf/util/util.c       |  3 ++-
>  2 files changed, 10 insertions(+), 3 deletions(-)
>
>
> Regards
> Adrian
Adrian Hunter March 29, 2023, 5:07 a.m. UTC | #2 
On 16/03/23 21:41, Adrian Hunter wrote:
> Hi
> 
> Here are 3 small fixes resulting from a report from a
> kernel test robot:
> 
> https://lore.kernel.org/oe-lkp/202303061424.6ad43294-yujie.liu@intel.com
> 
> Issues were revealed due to the use of build
> option:
> 
> 	EXTRA_CFLAGS="-fsanitize=undefined -fsanitize=address"
> 
> 
> Adrian Hunter (3):
>       perf symbols: Fix use-after-free in get_plt_got_name()
>       perf symbols: Fix unaligned access in get_x86_64_plt_disp()
>       perf tools: Avoid warning in do_realloc_array_as_needed()
> 
>  tools/perf/util/symbol-elf.c | 10 ++++++++--
>  tools/perf/util/util.c       |  3 ++-
>  2 files changed, 10 insertions(+), 3 deletions(-)

These seem to be still outstanding.
Arnaldo Carvalho de Melo March 29, 2023, 12:39 p.m. UTC | #3 
Em Wed, Mar 29, 2023 at 08:07:42AM +0300, Adrian Hunter escreveu:
> On 16/03/23 21:41, Adrian Hunter wrote:
> > Hi
> > 
> > Here are 3 small fixes resulting from a report from a
> > kernel test robot:
> > 
> > https://lore.kernel.org/oe-lkp/202303061424.6ad43294-yujie.liu@intel.com
> > 
> > Issues were revealed due to the use of build
> > option:
> > 
> > 	EXTRA_CFLAGS="-fsanitize=undefined -fsanitize=address"
> > 
> > 
> > Adrian Hunter (3):
> >       perf symbols: Fix use-after-free in get_plt_got_name()
> >       perf symbols: Fix unaligned access in get_x86_64_plt_disp()
> >       perf tools: Avoid warning in do_realloc_array_as_needed()
> > 
> >  tools/perf/util/symbol-elf.c | 10 ++++++++--
> >  tools/perf/util/util.c       |  3 ++-
> >  2 files changed, 10 insertions(+), 3 deletions(-)
> 
> These seem to be still outstanding.

Thanks for the reminder, applied.

- Arnaldo