diff options
author | Colin Ian King <colin.king@canonical.com> | 2021-04-15 12:06:54 +0100 |
---|---|---|
committer | Vinod Koul <vkoul@kernel.org> | 2021-04-20 15:45:50 +0530 |
commit | 28ac8e03c43dfc6a703aa420d18222540b801120 (patch) | |
tree | 8f8a5e1331309fc4ae832c979860c93f60f67c56 /crypto/cbc.c | |
parent | 0b0f93cf913b96f7a6f3d6b1ab5dc697d7b9d0f6 (diff) | |
download | lwn-28ac8e03c43dfc6a703aa420d18222540b801120.tar.gz lwn-28ac8e03c43dfc6a703aa420d18222540b801120.zip |
dmaengine: idxd: Fix potential null dereference on pointer status
There are calls to idxd_cmd_exec that pass a null status pointer however
a recent commit has added an assignment to *status that can end up
with a null pointer dereference. The function expects a null status
pointer sometimes as there is a later assignment to *status where
status is first null checked. Fix the issue by null checking status
before making the assignment.
Addresses-Coverity: ("Explicit null dereferenced")
Fixes: 89e3becd8f82 ("dmaengine: idxd: check device state before issue command")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Dave Jiang <dave.jiang@intel.com>
Link: https://lore.kernel.org/r/20210415110654.1941580-1-colin.king@canonical.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'crypto/cbc.c')
0 files changed, 0 insertions, 0 deletions