Re: [PATCH v2] perf ordered_events: fix crash in free_dup_event()

2018-08-10 Thread Jiri Olsa
On Fri, Aug 10, 2018 at 01:21:18AM -0700, Stephane Eranian wrote: > On Thu, Aug 9, 2018 at 1:07 AM Jiri Olsa wrote: > > > > On Wed, Aug 08, 2018 at 03:33:20PM -0700, Stephane Eranian wrote: > > > This patch fixes a bug in ordered_event.c:alloc_event(). > > > An ordered_event struct was not

Re: [PATCH v2] perf ordered_events: fix crash in free_dup_event()

2018-08-10 Thread Jiri Olsa
On Fri, Aug 10, 2018 at 01:21:18AM -0700, Stephane Eranian wrote: > On Thu, Aug 9, 2018 at 1:07 AM Jiri Olsa wrote: > > > > On Wed, Aug 08, 2018 at 03:33:20PM -0700, Stephane Eranian wrote: > > > This patch fixes a bug in ordered_event.c:alloc_event(). > > > An ordered_event struct was not

Re: [PATCH v2] perf ordered_events: fix crash in free_dup_event()

2018-08-10 Thread Stephane Eranian
On Thu, Aug 9, 2018 at 1:07 AM Jiri Olsa wrote: > > On Wed, Aug 08, 2018 at 03:33:20PM -0700, Stephane Eranian wrote: > > This patch fixes a bug in ordered_event.c:alloc_event(). > > An ordered_event struct was not initialized properly potentially > > causing crashes later on in free_dup_event()

Re: [PATCH v2] perf ordered_events: fix crash in free_dup_event()

2018-08-10 Thread Stephane Eranian
On Thu, Aug 9, 2018 at 1:07 AM Jiri Olsa wrote: > > On Wed, Aug 08, 2018 at 03:33:20PM -0700, Stephane Eranian wrote: > > This patch fixes a bug in ordered_event.c:alloc_event(). > > An ordered_event struct was not initialized properly potentially > > causing crashes later on in free_dup_event()

Re: [PATCH v2] perf ordered_events: fix crash in free_dup_event()

2018-08-09 Thread Jiri Olsa
On Wed, Aug 08, 2018 at 03:33:20PM -0700, Stephane Eranian wrote: > This patch fixes a bug in ordered_event.c:alloc_event(). > An ordered_event struct was not initialized properly potentially > causing crashes later on in free_dup_event() depending on the > content of the memory. If it was NULL,

Re: [PATCH v2] perf ordered_events: fix crash in free_dup_event()

2018-08-09 Thread Jiri Olsa
On Wed, Aug 08, 2018 at 03:33:20PM -0700, Stephane Eranian wrote: > This patch fixes a bug in ordered_event.c:alloc_event(). > An ordered_event struct was not initialized properly potentially > causing crashes later on in free_dup_event() depending on the > content of the memory. If it was NULL,

[PATCH v2] perf ordered_events: fix crash in free_dup_event()

2018-08-08 Thread Stephane Eranian
This patch fixes a bug in ordered_event.c:alloc_event(). An ordered_event struct was not initialized properly potentially causing crashes later on in free_dup_event() depending on the content of the memory. If it was NULL, then it would work fine, otherwise, it could cause crashes such as: $ perf

[PATCH v2] perf ordered_events: fix crash in free_dup_event()

2018-08-08 Thread Stephane Eranian
This patch fixes a bug in ordered_event.c:alloc_event(). An ordered_event struct was not initialized properly potentially causing crashes later on in free_dup_event() depending on the content of the memory. If it was NULL, then it would work fine, otherwise, it could cause crashes such as: $ perf