Re: [PATCH] perf: fix infinite loop with corrupted header

From: Sonny Rao
Date: Thu Sep 26 2013 - 13:34:03 EST


On Thu, Sep 26, 2013 at 6:34 AM, David Ahern <dsahern@xxxxxxxxx> wrote:
> On 9/25/13 11:20 PM, Sonny Rao wrote:
>>
>> We recently ran into a corrupt perf data file which mostly looked okay
>> but the section size for data was set to 0. This caused perf report to
>> get into an infinite loop in __perf_session_process_events(). Let's
>> just avoid this by bailing early and reporting it if there's an
>> invalid header.
>
>
> Been suggested before:
> https://lkml.org/lkml/2013/5/9/405
>
> Other changes went in around that time as well. Are you still seeing the
> loop on latest source?

I'm still seeing it on 3.12-rc1. I haven't tested anything newer or
in the perf git tree.
Thanks

>
> David
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/