From patchwork Tue Sep 7 14:53:54 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yevgeny Kliteynik X-Patchwork-Id: 161041 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter1.kernel.org (8.14.4/8.14.3) with ESMTP id o87EsOsD031573 for ; Tue, 7 Sep 2010 14:54:24 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757393Ab0IGOx6 (ORCPT ); Tue, 7 Sep 2010 10:53:58 -0400 Received: from mail.mellanox.co.il ([194.90.237.43]:53646 "EHLO mellanox.co.il" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1757387Ab0IGOx5 (ORCPT ); Tue, 7 Sep 2010 10:53:57 -0400 Received: from Internal Mail-Server by MTLPINE2 (envelope-from kliteyn@mellanox.co.il) with SMTP; 7 Sep 2010 17:53:16 +0300 Received: from [10.4.1.29] (10.4.1.29) by mtlmail01.mtl.com (10.0.8.12) with Microsoft SMTP Server id 8.2.254.0; Tue, 7 Sep 2010 17:53:54 +0300 Message-ID: <4C865202.9020105@mellanox.co.il> Date: Tue, 7 Sep 2010 17:53:54 +0300 From: Yevgeny Kliteynik Reply-To: kliteyn@dev.mellanox.co.il User-Agent: Thunderbird 1.5.0.5 (X11/20060719) MIME-Version: 1.0 To: Sasha Khapyorsky , Linux RDMA Subject: [PATCH] opensm/osm_mtree.c: removing useless 'if' statement Sender: linux-rdma-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-rdma@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.3 (demeter1.kernel.org [140.211.167.41]); Tue, 07 Sep 2010 14:54:24 +0000 (UTC) diff --git a/opensm/opensm/osm_mtree.c b/opensm/opensm/osm_mtree.c index b5ce73b..693d0d5 100644 --- a/opensm/opensm/osm_mtree.c +++ b/opensm/opensm/osm_mtree.c @@ -73,11 +73,10 @@ void osm_mtree_destroy(IN osm_mtree_node_t * p_mtn) if (p_mtn == NULL) return; - if (p_mtn->child_array != NULL) - for (i = 0; i < p_mtn->max_children; i++) - if ((p_mtn->child_array[i] != NULL) && - (p_mtn->child_array[i] != OSM_MTREE_LEAF)) - osm_mtree_destroy(p_mtn->child_array[i]); + for (i = 0; i < p_mtn->max_children; i++) + if ((p_mtn->child_array[i] != NULL) && + (p_mtn->child_array[i] != OSM_MTREE_LEAF)) + osm_mtree_destroy(p_mtn->child_array[i]); free(p_mtn); }