diff mbox series

dmaengine: idxd: fix uninit var for alt_drv

Message ID 162689250332.2114335.636367120454420852.stgit@djiang5-desk3.ch.intel.com (mailing list archive)
State Accepted
Commit 568b2126466f926a10be0b53b40c2d6ae056d8d6
Headers show
Series dmaengine: idxd: fix uninit var for alt_drv | expand

Commit Message

Dave Jiang July 21, 2021, 6:35 p.m. UTC
0-day detected uninitialized alt_drv variable in the bind_store() function.
The branch can be taken when device is not idxd device or wq 'struct
device'. Init alt_drv to NULL.

Fixes: 6e7f3ee97bbe ("dmaengine: idxd: move dsa_drv support to compatible mode")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Dave Jiang <dave.jiang@intel.com>
---
 drivers/dma/idxd/compat.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Vinod Koul July 28, 2021, 12:27 p.m. UTC | #1
On 21-07-21, 11:35, Dave Jiang wrote:
> 0-day detected uninitialized alt_drv variable in the bind_store() function.
> The branch can be taken when device is not idxd device or wq 'struct
> device'. Init alt_drv to NULL.

Applied, thanks
diff mbox series

Patch

diff --git a/drivers/dma/idxd/compat.c b/drivers/dma/idxd/compat.c
index d67746ee0c1a..d7616c240dcd 100644
--- a/drivers/dma/idxd/compat.c
+++ b/drivers/dma/idxd/compat.c
@@ -34,7 +34,7 @@  static ssize_t bind_store(struct device_driver *drv, const char *buf, size_t cou
 {
 	struct bus_type *bus = drv->bus;
 	struct device *dev;
-	struct device_driver *alt_drv;
+	struct device_driver *alt_drv = NULL;
 	int rc = -ENODEV;
 	struct idxd_dev *idxd_dev;