commit 0cb9870ddfa1b2fec50debe6d6333cbcb3df1e7e
author Patrick Delaunay <patrick.delaunay@foss.st.com> Tue Nov 02 11:09:12 2021 +0100
committer Patrick Delaunay <patrick.delaunay@foss.st.com> Thu Nov 04 09:48:44 2021 +0100
tree 63cc9b1468ace1e0fd420838dcf2159ac4fa74a5
parent 89ff55fef4b74beda62d04b01428f3e587a4fb4d

fix(drivers/usb): fix Null pointer dereferences in usb_core_set_config

Correct the invalid test on NULL pointer pdev->class in
usb_core_set_config function.

This patch fix the coverity errors:

  ** CID 373790:  Null pointer dereferences  (FORWARD_NULL)
  /drivers/usb/usb_device.c: 182 in usb_core_set_config()

Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Change-Id: I83e43261bafa2d47f800e56df0b047a6c58a1e29

drivers/usb/usb_device.c

diff --git a/drivers/usb/usb_device.c b/drivers/usb/usb_device.c
index ce02d4f..27b78e7 100644
--- a/drivers/usb/usb_device.c
+++ b/drivers/usb/usb_device.c
@@ -174,7 +174,7 @@
 			pdev->dev_config = cfgidx;
 			pdev->class->de_init(pdev, cfgidx);
 		} else if (cfgidx != pdev->dev_config) {
-			if (pdev->class != NULL) {
+			if (pdev->class == NULL) {
 				usb_core_ctl_error(pdev);
 				return;
 			}