linux-rdma.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] osmtest/osmt_service.c: In osmt_run_service_records_flow, add missing status
@ 2010-09-27 13:40 Hal Rosenstock
       [not found] ` <20100927134040.GA1321-Wuw85uim5zDR7s880joybQ@public.gmane.org>
  0 siblings, 1 reply; 2+ messages in thread
From: Hal Rosenstock @ 2010-09-27 13:40 UTC (permalink / raw)
  To: sashak-smomgflXvOZWk0Htik3J/w; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA


When deleting ServiceRecord for service_name[6] fails, error status should
be set.

Signed-off-by: Hal Rosenstock <hal-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
---
diff --git a/opensm/osmtest/osmt_service.c b/opensm/osmtest/osmt_service.c
index a203b0c..891fbe3 100644
--- a/opensm/osmtest/osmt_service.c
+++ b/opensm/osmtest/osmt_service.c
@@ -1578,6 +1578,7 @@ ib_api_status_t osmt_run_service_records_flow(IN osmtest_t * const p_osmt)
 		OSM_LOG(&p_osmt->log, OSM_LOG_ERROR, "ERR 4A32: "
 			"Failed to delete service name: %s\n",
 			(char *)service_name[6]);
+		status = IB_ERROR;
 		goto Exit;
 	}
 
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] osmtest/osmt_service.c: In osmt_run_service_records_flow, add missing status
       [not found] ` <20100927134040.GA1321-Wuw85uim5zDR7s880joybQ@public.gmane.org>
@ 2010-11-30 15:45   ` Sasha Khapyorsky
  0 siblings, 0 replies; 2+ messages in thread
From: Sasha Khapyorsky @ 2010-11-30 15:45 UTC (permalink / raw)
  To: Hal Rosenstock; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA

On 09:40 Mon 27 Sep     , Hal Rosenstock wrote:
> 
> When deleting ServiceRecord for service_name[6] fails, error status should
> be set.
> 
> Signed-off-by: Hal Rosenstock <hal-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>

Applied. Thanks.

Sasha
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-11-30 15:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-27 13:40 [PATCH] osmtest/osmt_service.c: In osmt_run_service_records_flow, add missing status Hal Rosenstock
     [not found] ` <20100927134040.GA1321-Wuw85uim5zDR7s880joybQ@public.gmane.org>
2010-11-30 15:45   ` Sasha Khapyorsky

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).