Search Linux Wireless

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 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.

Regards
Alan
--
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