The fastrpc_buf_free function currently does not handle the case where
the input buffer pointer (buf) is NULL. This can lead to a null pointer
dereference, causing a crash or undefined behavior when the function
attempts to access members of the buf structure. Add a NULL check to
ensure safe handling of NULL pointers and prevent potential crashes.

Fixes: c68cfb718c8f9 ("misc: fastrpc: Add support for context Invoke method")
Cc: sta...@kernel.org
Signed-off-by: Ekansh Gupta <ekansh.gu...@oss.qualcomm.com>
---
 drivers/misc/fastrpc.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/misc/fastrpc.c b/drivers/misc/fastrpc.c
index 7b7a22c91fe4..ca3721365ddc 100644
--- a/drivers/misc/fastrpc.c
+++ b/drivers/misc/fastrpc.c
@@ -394,6 +394,9 @@ static int fastrpc_map_lookup(struct fastrpc_user *fl, int 
fd,
 
 static void fastrpc_buf_free(struct fastrpc_buf *buf)
 {
+       if (!buf)
+               return;
+
        dma_free_coherent(buf->dev, buf->size, buf->virt,
                          FASTRPC_PHYS(buf->phys));
        kfree(buf);
-- 
2.34.1

Reply via email to