Messages in this thread | | | Date | Thu, 18 Dec 2008 18:18:36 +0100 (CET) | From | Julia Lawall <> | Subject | Re: [PATCH 7/11] drivers/net/wireless/ath9k: Move a dereference below a NULL test |
| |
Indeed the code seems to have complete changed.
julia
On Thu, 18 Dec 2008, John W. Linville wrote:
> On Tue, Dec 16, 2008 at 04:14:28PM +0100, Julia Lawall wrote: > > From: Julia Lawall <julia@diku.dk> > > > > In each case, if the NULL test is necessary, then the dereference should be > > moved below the NULL test. > > > > The semantic patch that makes this change is as follows: > > (http://www.emn.fr/x-info/coccinelle/) > > > > // <smpl> > > @@ > > type T; > > expression E; > > identifier i,fld; > > statement S; > > @@ > > > > - T i = E->fld; > > + T i; > > ... when != E > > when != i > > if (E == NULL) S > > + i = E->fld; > > // </smpl> > > > > Signed-off-by: Julia Lawall <julia@diku.dk> > > > > --- > > drivers/net/wireless/ath9k/hw.c | 3 ++- > > 1 files changed, 2 insertions(+), 1 deletions(-) > > > > diff --git a/drivers/net/wireless/ath9k/hw.c b/drivers/net/wireless/ath9k/hw.c > > index 98bc25c..7d1dac6 100644 > > --- a/drivers/net/wireless/ath9k/hw.c > > +++ b/drivers/net/wireless/ath9k/hw.c > > @@ -5581,11 +5581,12 @@ static inline bool ath9k_hw_run_init_cals(struct ath_hal *ah, > > struct ath9k_channel ichan; > > bool isCalDone; > > struct hal_cal_list *currCal = ahp->ah_cal_list_curr; > > - const struct hal_percal_data *calData = currCal->calData; > > + const struct hal_percal_data *calData; > > int i; > > > > if (currCal == NULL) > > return false; > > + calData = currCal->calData; > > > > ichan.CalValid = 0; > > This patch doesn't seem to apply. I don't see this code anywhere. > Perhaps this is against some older version? > > John > -- > John W. Linville Linux should be at the core > linville@tuxdriver.com of your literate lifestyle. >
| |