Commit 59a39ca3 authored by Dan Carpenter's avatar Dan Carpenter Committed by Roland Dreier

RDMA/ocrdma: Fix off by one in ocrdma_query_gid()

The ->sgid_tbl[] array has OCRDMA_MAX_SGID number of elements so this
test is off by one.  ->sgid_tbl is allocated in ocrdma_alloc_resources().
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Acked-by: default avatarSelvin Xavier <selvin.xavier@emulex.com>
Signed-off-by: default avatarRoland Dreier <roland@purestorage.com>
parent f3070e7e
...@@ -53,7 +53,7 @@ int ocrdma_query_gid(struct ib_device *ibdev, u8 port, ...@@ -53,7 +53,7 @@ int ocrdma_query_gid(struct ib_device *ibdev, u8 port,
dev = get_ocrdma_dev(ibdev); dev = get_ocrdma_dev(ibdev);
memset(sgid, 0, sizeof(*sgid)); memset(sgid, 0, sizeof(*sgid));
if (index > OCRDMA_MAX_SGID) if (index >= OCRDMA_MAX_SGID)
return -EINVAL; return -EINVAL;
memcpy(sgid, &dev->sgid_tbl[index], sizeof(*sgid)); memcpy(sgid, &dev->sgid_tbl[index], sizeof(*sgid));
......
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