2013-12-06 04:50:39

by Wang Weidong

Subject: [PATCH] wireless: fix build warning of used uninitialized variable

nl80211.c: In function __cfg80211_wdev_from_attrs
nl80211.c:77: warning: wdev_id may be used uninitialized in this function

Signed-off-by: Wang Weidong <[email protected]>
---
net/wireless/nl80211.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c
index 138dc3b..6894229 100644
--- a/net/wireless/nl80211.c
+++ b/net/wireless/nl80211.c
@@ -74,7 +74,7 @@ __cfg80211_wdev_from_attrs(struct net *netns, struct nlattr **attrs)
struct wireless_dev *result = NULL;
bool have_ifidx = attrs[NL80211_ATTR_IFINDEX];
bool have_wdev_id = attrs[NL80211_ATTR_WDEV];
- u64 wdev_id;
+ u64 wdev_id = 0;
int wiphy_idx = -1;
int ifidx = -1;

--
1.6.0.2

2013-12-06 11:11:54

by Wang Weidong

Subject: Re: [PATCH] wireless: fix build warning of used uninitialized variable

On 2013/12/6 12:49, Wang Weidong wrote:
> nl80211.c: In function __cfg80211_wdev_from_attrs
> nl80211.c:77: warning: wdev_id may be used uninitialized in this function
>

Sorry. My complier version is too low so that I got this warning.
Just ignore it.

Regards.
Wang

> Signed-off-by: Wang Weidong <[email protected]>
> ---
> net/wireless/nl80211.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c
> index 138dc3b..6894229 100644
> --- a/net/wireless/nl80211.c
> +++ b/net/wireless/nl80211.c
> @@ -74,7 +74,7 @@ __cfg80211_wdev_from_attrs(struct net *netns, struct nlattr **attrs)
> struct wireless_dev *result = NULL;
> bool have_ifidx = attrs[NL80211_ATTR_IFINDEX];
> bool have_wdev_id = attrs[NL80211_ATTR_WDEV];
> - u64 wdev_id;
> + u64 wdev_id = 0;
> int wiphy_idx = -1;
> int ifidx = -1;
>
>