| Message ID | 1473109698-31408-17-git-send-email-jgunthorpe@obsidianresearch.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
On 9/5/2016 5:08 PM, Jason Gunthorpe wrote: > The code was attempting to pass int* instead of uint32_t* to > umad_register2, which is not OK. > > Signed-off-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com> Thanks. Applied. Reviewed-by: Hal Rosenstock <hal@mellanox.com> -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/libibumad/tests/umad_reg2_compat.c b/libibumad/tests/umad_reg2_compat.c index 413a3bec6d58..6dd4a48a59b2 100644 --- a/libibumad/tests/umad_reg2_compat.c +++ b/libibumad/tests/umad_reg2_compat.c @@ -154,7 +154,7 @@ void test_fall_back(void) { int rc = 0; struct umad_reg_attr reg_attr; - int agent_id; + uint32_t agent_id; int fd; fd = open_test_device(); diff --git a/libibumad/tests/umad_register2.c b/libibumad/tests/umad_register2.c index c2d7846038bd..ed7e816bb63c 100644 --- a/libibumad/tests/umad_register2.c +++ b/libibumad/tests/umad_register2.c @@ -92,8 +92,8 @@ void test_fail(void) { int rc = 0; struct umad_reg_attr reg_attr; - int agent_id; - int agent_id2; + uint32_t agent_id; + uint32_t agent_id2; int fd; printf("\n *****\nBegin invalid tests\n"); @@ -200,7 +200,7 @@ void test_oui(void) { int rc = 0; struct umad_reg_attr reg_attr; - int agent_id; + uint32_t agent_id; int fd; printf("\n *****\nStart valid oui tests\n");
The code was attempting to pass int* instead of uint32_t* to umad_register2, which is not OK. Signed-off-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com> --- libibumad/tests/umad_reg2_compat.c | 2 +- libibumad/tests/umad_register2.c | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-)