From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jay Vosburgh Subject: Re: [PATCH] bonding:delete a dereference before check Date: Mon, 13 Jun 2011 15:11:01 -0700 Message-ID: <18226.1308003061@death> References: <20110611081214.GA21427@shale.localdomain> <1307975410-13523-1-git-send-email-panweiping3@gmail.com> Cc: error27@gmail.com, Andy Gospodarek (supporter:BONDING DRIVER) , netdev@vger.kernel.org (open list:BONDING DRIVER), linux-kernel@vger.kernel.org (open list) To: Weiping Pan Return-path: Received: from e3.ny.us.ibm.com ([32.97.182.143]:42127 "EHLO e3.ny.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753009Ab1FMWLH (ORCPT ); Mon, 13 Jun 2011 18:11:07 -0400 In-reply-to: <1307975410-13523-1-git-send-email-panweiping3@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: Weiping Pan wrote: >Dan Carpenter found that there was a dereference before a check, >added in 56d00c677de0(bonding:delete lacp_fast from ad_bond_info). > >Signed-off-by: Weiping Pan Signed-off-by: Jay Vosburgh -J > drivers/net/bonding/bond_3ad.c | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > >diff --git a/drivers/net/bonding/bond_3ad.c b/drivers/net/bonding/bond_3ad.c >index 6122725..77da2e8 100644 >--- a/drivers/net/bonding/bond_3ad.c >+++ b/drivers/net/bonding/bond_3ad.c >@@ -1900,7 +1900,6 @@ void bond_3ad_initialize(struct bonding *bond, u16 tick_resolution) > int bond_3ad_bind_slave(struct slave *slave) > { > struct bonding *bond = bond_get_bond_by_slave(slave); >- int lacp_fast = bond->params.lacp_fast; > struct port *port; > struct aggregator *aggregator; > >@@ -1916,7 +1915,7 @@ int bond_3ad_bind_slave(struct slave *slave) > // port initialization > port = &(SLAVE_AD_INFO(slave).port); > >- ad_initialize_port(port, lacp_fast); >+ ad_initialize_port(port, bond->params.lacp_fast); > > port->slave = slave; > port->actor_port_number = SLAVE_AD_INFO(slave).id; >-- >1.7.4.4