Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Sven Schnelle <svens@linux.ibm.com>
To:     Steven Rostedt <rostedt@goodmis.org>
Cc:     linux-kernel@vger.kernel.org
Subject: Re: BUG: KASAN: slab-out-of-bounds in print_synth_event+0xa68/0xa78
References: <yt9dsf8zfhw8.fsf@linux.ibm.com>
        <20230804115033.34c2b5af@gandalf.local.home>
        <yt9da5v66a4v.fsf@linux.ibm.com>
        <20230804133601.40cb5fe9@gandalf.local.home>
Date:   Mon, 07 Aug 2023 08:08:49 +0200
In-Reply-To: <20230804133601.40cb5fe9@gandalf.local.home> (Steven Rostedt's
        message of "Fri, 4 Aug 2023 13:36:01 -0400")
Message-ID: <yt9d7cq7s7ta.fsf@linux.ibm.com>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

Steven Rostedt <rostedt@goodmis.org> writes:

> On Fri, 04 Aug 2023 18:32:48 +0200
> Sven Schnelle <svens@linux.ibm.com> wrote:
>
>> > Can you show where exactly the above line is?  
>> 
>> It is:
>> 
>> (gdb) list *(print_synth_event+0xa68)
>> 0x5e4f60 is in print_synth_event (/home/svens/ibmgit/linux/kernel/trace/trace_events_synth.c:410).
>> 405                             p = (void *)entry + data_offset;
>> 406                             end = (void *)p + len - (sizeof(long) - 1);
>> 407
>> 408                             trace_seq_printf(s, "%s=STACK:\n", se->fields[i]->name);
>> 409
>> 410                             for (; *p && p < end; p++)
>
> I think that conditional needs to be swapped. Can you test the following
> change and see if the problem disappears?

Unfortunately that doesn't change anything, still exactly the same splat.