diff mbox

[4/6] osm_ucast_ftree.c: fix dereferencing null variable

Message ID 51B768DC.8060003@dev.mellanox.co.il (mailing list archive)
State Accepted
Delegated to: Hal Rosenstock
Headers show

Commit Message

Hal Rosenstock June 11, 2013, 6:13 p.m. UTC 
From 35f4419e6f2451f2124e826e8e93b02c733d068c Mon Sep 17 00:00:00 2001
From: Dan Ben Yosef <danby@dev.mellanox.co.il>
Date: Sun, 2 Dec 2012 16:07:49 +0200
Subject: [PATCH 4/6] osm_ucast_ftree.c: fix dereferencing null variable

Dereferencing null variable "p_port".

Signed-off-by: Dan Ben Yosef <danby@dev.mellanox.co.il>
---
 opensm/osm_ucast_ftree.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
diff mbox

Patch

diff --git a/opensm/osm_ucast_ftree.c b/opensm/osm_ucast_ftree.c
index a293803..decfc82 100644
--- a/opensm/osm_ucast_ftree.c
+++ b/opensm/osm_ucast_ftree.c
@@ -2591,6 +2591,8 @@  fabric_route_downgoing_by_going_up(IN ftree_fabric_t * p_ftree,
 		   trying to balance these routes - always pick port 0. */
 		p_min_port = NULL;
 		ports_num = (uint16_t) cl_ptr_vector_get_size(&p_group->ports);
+		if(ports_num == 0)
+			continue;
 		for (j = 0; j < ports_num; j++) {
 			cl_ptr_vector_at(&p_group->ports, j, (void *)&p_port);
 			if (!p_min_port) {