Re: [PATCH 0/3] misc: fastrpc: fix memory corruption

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Johan,

On 29/08/2022 09:05, Johan Hovold wrote:
The fastrpc driver uses a fixed-sized array to store its sessions but
missing and broken sanity checks could lead to memory beyond the array
being corrupted.

This specifically happens on SC8280XP platforms that use 14 sessions for
the compute DSP.

Thanks for doing this.

I see that we hit this issue once again, and the way we are fixing it is not really scalable. We should really get rid of FASTRPC_MAX_SESSIONS.

We should allocate the sessions dynamically based in the child node count and qcom,nsessions.



thanks,
Srini

These are all needed for 6.0.

Johan


Johan Hovold (3):
   misc: fastrpc: fix memory corruption on probe
   misc: fastrpc: fix memory corruption on open
   misc: fastrpc: increase maximum session count

  drivers/misc/fastrpc.c | 14 +++++++++-----
  1 file changed, 9 insertions(+), 5 deletions(-)




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux