Search Linux Wireless

Re: net/wireless/reg.c:1196: warning: ‘channel_after’ may be used uninitialised in this function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 15, 2009 at 11:58:39AM +0100, Alan Jenkins wrote:
> [Corrected Luis address]
> 
> On 4/15/09, Alan Jenkins <sourcejedi.lkml@xxxxxxxxxxxxxx> wrote:
> > On 4/10/09, Luis R. Rodriguez <lrodriguez@xxxxxxxxxxx> wrote:
> >> Thanks pavel, please try this patch:
> >>
> >> From 94505af850bd0961a86e2786238ac0bbe0c44615 Mon Sep 17 00:00:00 2001
> >> From: Luis R. Rodriguez <lrodriguez@xxxxxxxxxxx>
> >> Date: Fri, 10 Apr 2009 11:07:43 -0700
> >> Subject: [PATCH] cfg80211: fix bug while trying to process obeacon hints
> >> on
> >> init
> >>
> >> During initialization we would not have received any beacons
> >> so skip processing reg beacon hints, also adds a check to
> >> reg_is_world_roaming() for last_request before accessing its
> >> fields.
> >
> > Hi again
> >
> > I'm building wireless-testing to test the rfkill-rewrite patches on
> > top of.  I need to apply the patch referenced above as well.
> >
> > When I applied the above patch, I noticed this unrelated warning
> >
> > net/wireless/reg.c: In function ‘wiphy_update_regulatory’:
> > net/wireless/reg.c:1196: warning: ‘channel_after’ may be used
> > uninitialised in this function
> > net/wireless/reg.c:1196: note: ‘channel_after’ was declared here
> > net/wireless/reg.c:1196: warning: ‘channel_before’ may be used
> > uninitialised in this function
> > net/wireless/reg.c:1196: note: ‘channel_before’ was declared here
> >
> > Looking at the code, I think it does need fixing.  There's even a
> > comment which says "this finds them (or not)" - and if they're not
> > found, channel_before / channel_after will be unset.

I reverted the (HT40-related) patch series that last touched that line.
Hopefully that resolves that warning.

> Um, also the new "ath" module taints the kernel
> 
> ath: module 'unspecified' taints kernel.
> 
> and I get a new BUG in freq_reg_info_regd.  I'll send the full trace
> as soon as someone reassures it is welcome and wireless-testing is
> really intended for testing at the current time :-).

Someone probably wants to see it. :-)

John
-- 
John W. Linville		Someday the world will need a hero, and you
linville@xxxxxxxxxxxxx			might be all we have.  Be ready.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux