Commit 16014929 authored by Dennis Dalessandro's avatar Dennis Dalessandro Committed by Greg Kroah-Hartman

IB/hfi1: Remove incorrect IS_ERR check

commit 2b16056f upstream.

Remove IS_ERR check from caching code as the function being called does
not actually return error pointers.

Fixes: f19bd643: "IB/hfi1: Prevent NULL pointer deferences in caching code"
Reported-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: default avatarDean Luick <dean.luick@intel.com>
Signed-off-by: default avatarDennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: default avatarDoug Ledford <dledford@redhat.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 46e47543
...@@ -1152,7 +1152,7 @@ static int pin_vector_pages(struct user_sdma_request *req, ...@@ -1152,7 +1152,7 @@ static int pin_vector_pages(struct user_sdma_request *req,
rb_node = hfi1_mmu_rb_extract(pq->handler, rb_node = hfi1_mmu_rb_extract(pq->handler,
(unsigned long)iovec->iov.iov_base, (unsigned long)iovec->iov.iov_base,
iovec->iov.iov_len); iovec->iov.iov_len);
if (rb_node && !IS_ERR(rb_node)) if (rb_node)
node = container_of(rb_node, struct sdma_mmu_node, rb); node = container_of(rb_node, struct sdma_mmu_node, rb);
else else
rb_node = NULL; rb_node = NULL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment