John W. Linville wrote:
> drivers/net/wireless/b43legacy/main.c: In function ‘b43legacy_op_dev_config’:
> drivers/net/wireless/b43legacy/main.c:2468: warning: ‘up_dev’ may be used uninitialized in this function
>
> Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx>
> ---
> drivers/net/wireless/b43legacy/main.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/net/wireless/b43legacy/main.c b/drivers/net/wireless/b43legacy/main.c
> index c1324e3..fb996c2 100644
> --- a/drivers/net/wireless/b43legacy/main.c
> +++ b/drivers/net/wireless/b43legacy/main.c
> @@ -2465,7 +2465,7 @@ static void b43legacy_put_phy_into_reset(struct b43legacy_wldev *dev)
> static int b43legacy_switch_phymode(struct b43legacy_wl *wl,
> unsigned int new_mode)
> {
> - struct b43legacy_wldev *up_dev;
> + struct b43legacy_wldev *uninitialized_var(up_dev);
> struct b43legacy_wldev *down_dev;
> int err;
> bool gmode = 0;
It seems that some versions of gcc don't do the flow analysis
correctly, so
ACK
Larry
--
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
