diff options
author | Kangjie Lu <kjlu@umn.edu> | 2019-05-14 15:44:49 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-06-15 11:53:59 +0200 |
commit | 2a89e4c5ee2ee5964bc8b974f120e1a8eded25e9 (patch) | |
tree | ea60444ddb21e2e3703a2feb58eee05de6fd2d6b /drivers/rapidio | |
parent | 768292d053619b2725b846ed2bf556bf40f43de2 (diff) | |
download | linux-stable-2a89e4c5ee2ee5964bc8b974f120e1a8eded25e9.tar.gz linux-stable-2a89e4c5ee2ee5964bc8b974f120e1a8eded25e9.tar.bz2 linux-stable-2a89e4c5ee2ee5964bc8b974f120e1a8eded25e9.zip |
rapidio: fix a NULL pointer dereference when create_workqueue() fails
[ Upstream commit 23015b22e47c5409620b1726a677d69e5cd032ba ]
In case create_workqueue fails, the fix releases resources and returns
-ENOMEM to avoid NULL pointer dereference.
Signed-off-by: Kangjie Lu <kjlu@umn.edu>
Acked-by: Alexandre Bounine <alex.bou9@gmail.com>
Cc: Matt Porter <mporter@kernel.crashing.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'drivers/rapidio')
-rw-r--r-- | drivers/rapidio/rio_cm.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/drivers/rapidio/rio_cm.c b/drivers/rapidio/rio_cm.c index bad0e0ea4f30..ef989a15aefc 100644 --- a/drivers/rapidio/rio_cm.c +++ b/drivers/rapidio/rio_cm.c @@ -2145,6 +2145,14 @@ static int riocm_add_mport(struct device *dev, mutex_init(&cm->rx_lock); riocm_rx_fill(cm, RIOCM_RX_RING_SIZE); cm->rx_wq = create_workqueue(DRV_NAME "/rxq"); + if (!cm->rx_wq) { + riocm_error("failed to allocate IBMBOX_%d on %s", + cmbox, mport->name); + rio_release_outb_mbox(mport, cmbox); + kfree(cm); + return -ENOMEM; + } + INIT_WORK(&cm->rx_work, rio_ibmsg_handler); cm->tx_slot = 0; |