The ci_req->hw_buf can be NULL, test whether it is and if so,
avoid accessing it. Else, the system may crash.

Signed-off-by: Marek Vasut <ma...@denx.de>
Cc: Peter Chen <peter.c...@nxp.com>
Cc: Li Jun <jun...@nxp.com>
Cc: Peng Fan <peng....@nxp.com>
---
 drivers/usb/gadget/ci_udc.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/usb/gadget/ci_udc.c b/drivers/usb/gadget/ci_udc.c
index 226a9e6d671..542684c1c30 100644
--- a/drivers/usb/gadget/ci_udc.c
+++ b/drivers/usb/gadget/ci_udc.c
@@ -402,6 +402,9 @@ align:
 
 flush:
        hwaddr = (unsigned long)ci_req->hw_buf;
+       if (!hwaddr)
+               return 0;
+
        aligned_used_len = roundup(req->length, ARCH_DMA_MINALIGN);
        flush_dcache_range(hwaddr, hwaddr + aligned_used_len);
 
@@ -415,7 +418,7 @@ static void ci_debounce(struct ci_req *ci_req, int in)
        unsigned long hwaddr = (unsigned long)ci_req->hw_buf;
        uint32_t aligned_used_len;
 
-       if (in)
+       if (in || !hwaddr)
                return;
 
        aligned_used_len = roundup(req->actual, ARCH_DMA_MINALIGN);
-- 
2.34.1

Reply via email to