On Fri, Mar 21, 2025 at 10:33:36AM +0800, Herbert Xu wrote:
> On Thu, Mar 20, 2025 at 05:24:57PM +0000, Cabiddu, Giovanni wrote:
> >
> > > +static int acomp_reqchain_finish(struct acomp_req_chain *state,
> > > + int err, u32 mask)
> > > +{
> > > + struct acomp_req *req0 = state->req0;
> > > + struct acomp_req *req = state->cur;
> > > + struct acomp_req *n;
> > > +
> > > + acomp_reqchain_virt(state, err);
> > > +
> > > + if (req != req0)
> > I'm hitting a NULL pointer dereference at this point as req0 is NULL.
>
> Yes sorry, my testing was screwed up. When I thought I was testing
> the chaining fallback path I was actually testing the shash
> fall-through path.
No problem. The set fix that you sent, "Fix synchronous fallback path for
request chaining", resolved the issue.
Thanks,
--
Giovanni
