diff options
| author | Dan Carpenter <[email protected]> | 2018-01-09 23:03:46 +0300 |
|---|---|---|
| committer | Doug Ledford <[email protected]> | 2018-01-10 16:48:09 -0500 |
| commit | 57194fa763bfa1a0908f30d4c77835beaa118fcb (patch) | |
| tree | 5c5adaa5b07c08e423a32d8d1d1f1f270c16b89a | |
| parent | cd52cb26e7ead5093635e98e07e221e4df482d34 (diff) | |
IB/hfi1: Prevent a NULL dereference
In the original code, we set "fd->uctxt" to NULL and then dereference it
which will cause an Oops.
Fixes: f2a3bc00a03c ("IB/hfi1: Protect context array set/clear with spinlock")
Cc: <[email protected]> # 4.14.x
Signed-off-by: Dan Carpenter <[email protected]>
Reviewed-by: Michael J. Ruhl <[email protected]>
Signed-off-by: Doug Ledford <[email protected]>
| -rw-r--r-- | drivers/infiniband/hw/hfi1/file_ops.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/infiniband/hw/hfi1/file_ops.c b/drivers/infiniband/hw/hfi1/file_ops.c index 7750a9c38b06..1df7da47f431 100644 --- a/drivers/infiniband/hw/hfi1/file_ops.c +++ b/drivers/infiniband/hw/hfi1/file_ops.c @@ -763,11 +763,11 @@ static int complete_subctxt(struct hfi1_filedata *fd) } if (ret) { - hfi1_rcd_put(fd->uctxt); - fd->uctxt = NULL; spin_lock_irqsave(&fd->dd->uctxt_lock, flags); __clear_bit(fd->subctxt, fd->uctxt->in_use_ctxts); spin_unlock_irqrestore(&fd->dd->uctxt_lock, flags); + hfi1_rcd_put(fd->uctxt); + fd->uctxt = NULL; } return ret; |