This fixes the other occurence of the array overrun in ucsi.c, this time in ucsi_unregister_altmodes. Signed-off-by: Zwane Mwaikambo <zwane@xxxxxxxxx> --- diff --git a/drivers/usb/typec/ucsi/ucsi.c b/drivers/usb/typec/ucsi/ucsi.c index d0c63afaf..79061705e 100644 --- a/drivers/usb/typec/ucsi/ucsi.c +++ b/drivers/usb/typec/ucsi/ucsi.c @@ -479,7 +480,10 @@ static void ucsi_unregister_altmodes(struct ucsi_connector *con, u8 recipient) return; } - while (adev[i]) { + for (i = 0; i < UCSI_MAX_ALTMODES; i++) { + if (!adev[i]) + break; + if (recipient == UCSI_RECIPIENT_SOP && (adev[i]->svid == USB_TYPEC_DP_SID || (adev[i]->svid == USB_TYPEC_NVIDIA_VLINK_SID && @@ -488,7 +492,7 @@ static void ucsi_unregister_altmodes(struct ucsi_connector *con, u8 recipient) ucsi_displayport_remove_partner((void *)pdev); } typec_unregister_altmode(adev[i]); - adev[i++] = NULL; + adev[i] = NULL; } }