diff mbox

[opensm] osmtest.c: Fixed missing assignment of return value from function osmtest_get_port_rec_by_num

Message ID 536792CA.4040707@dev.mellanox.co.il (mailing list archive)
State Rejected
Headers show

Commit Message

Hal Rosenstock May 5, 2014, 1:31 p.m. UTC 
From: Tomer Cohen <tomerco@mellanox.com>
Date: Sun, 4 May 2014 11:38:01 +0300

found by Coverity

Signed-off-by: Tomer Cohen <tomerco@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
---
 osmtest/osmtest.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)
diff mbox

Patch

diff --git a/osmtest/osmtest.c b/osmtest/osmtest.c
index 857df86..d7aef60 100644
--- a/osmtest/osmtest.c
+++ b/osmtest/osmtest.c
@@ -5097,9 +5097,9 @@  osmtest_validate_single_port_rec_lid(IN osmtest_t * const p_osmt,
 	memset(&context, 0, sizeof(context));
 
 	context.p_osmt = p_osmt;
-	osmtest_get_port_rec_by_num(p_osmt,
-				    p_port->rec.lid,
-				    p_port->rec.port_num, &context);
+	status = osmtest_get_port_rec_by_num(p_osmt,
+					     p_port->rec.lid,
+					     p_port->rec.port_num, &context);
 	if (status != IB_SUCCESS) {
 		OSM_LOG(&p_osmt->log, OSM_LOG_ERROR, "ERR 0108: "
 			"ib_query failed (%s)\n", ib_get_err_str(status));