* [PATCH] opensm/osm_torus.c: In dump_torus, make sure switch is present before dumping
@ 2013-02-04 9:36 Alex Netes
[not found] ` <20130204093600.GA13151-iQai9MGU/dze+A/uUDamNg@public.gmane.org>
0 siblings, 1 reply; 2+ messages in thread
From: Alex Netes @ 2013-02-04 9:36 UTC (permalink / raw)
To: linux-rdma-u79uwXL29TY76Z2rM5mHXA; +Cc: hal-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb
Fix segmentation fault in osm_torus.c.
Signed-off-by: Hal Rosenstock <hal-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Signed-off-by: Alex Netes <alexne-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
---
opensm/osm_torus.c | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/opensm/osm_torus.c b/opensm/osm_torus.c
index 1510233..e63cb40 100644
--- a/opensm/osm_torus.c
+++ b/opensm/osm_torus.c
@@ -7517,11 +7517,12 @@ void dump_torus(struct torus *t)
for (k = 0; k < z_sz; k++)
for (j = 0; j < y_sz; j++)
for (i = 0; i < x_sz; i++)
- fprintf(file, "switch %u,%u,%u GUID 0x%04"
- PRIx64 " (%s)\n",
- i, j, k,
- cl_ntoh64(t->sw[i][j][k]->n_id),
- t->sw[i][j][k]->osm_switch->p_node->print_desc);
+ if (t->sw[i][j][k])
+ fprintf(file, "switch %u,%u,%u GUID 0x%04"
+ PRIx64 " (%s)\n",
+ i, j, k,
+ cl_ntoh64(t->sw[i][j][k]->n_id),
+ t->sw[i][j][k]->osm_switch->p_node->print_desc);
fclose(file);
}
--
1.7.11.7
--
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] opensm/osm_torus.c: In dump_torus, make sure switch is present before dumping
[not found] ` <20130204093600.GA13151-iQai9MGU/dze+A/uUDamNg@public.gmane.org>
@ 2013-02-04 21:56 ` Jim Schutt
0 siblings, 0 replies; 2+ messages in thread
From: Jim Schutt @ 2013-02-04 21:56 UTC (permalink / raw)
To: Alex Netes
Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA,
hal-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb
On 02/04/2013 02:36 AM, Alex Netes wrote:
> Fix segmentation fault in osm_torus.c.
>
> Signed-off-by: Hal Rosenstock <hal-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
> Signed-off-by: Alex Netes <alexne-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Acked-by: Jim Schutt <jaschut-4OHPYypu0djtX7QSmKvirg@public.gmane.org>
--
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:[~2013-02-04 21:56 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-02-04 9:36 [PATCH] opensm/osm_torus.c: In dump_torus, make sure switch is present before dumping Alex Netes
[not found] ` <20130204093600.GA13151-iQai9MGU/dze+A/uUDamNg@public.gmane.org>
2013-02-04 21:56 ` Jim Schutt
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.