Commit bfb27814 authored by Johannes Berg's avatar Johannes Berg

mac80211: ethtool: memset the whole sinfo struct to 0

Rather than just setting the valid flags to 0 set the
whole struct to 0 since other places might rely on it.
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent 35f4962c
...@@ -4,6 +4,7 @@ ...@@ -4,6 +4,7 @@
* Copied from cfg.c - originally * Copied from cfg.c - originally
* Copyright 2006-2010 Johannes Berg <johannes@sipsolutions.net> * Copyright 2006-2010 Johannes Berg <johannes@sipsolutions.net>
* Copyright 2014 Intel Corporation (Author: Johannes Berg) * Copyright 2014 Intel Corporation (Author: Johannes Berg)
* Copyright (C) 2018 Intel Corporation
* *
* This file is GPLv2 as found in COPYING. * This file is GPLv2 as found in COPYING.
*/ */
...@@ -106,7 +107,7 @@ static void ieee80211_get_stats(struct net_device *dev, ...@@ -106,7 +107,7 @@ static void ieee80211_get_stats(struct net_device *dev,
if (!(sta && !WARN_ON(sta->sdata->dev != dev))) if (!(sta && !WARN_ON(sta->sdata->dev != dev)))
goto do_survey; goto do_survey;
sinfo.filled = 0; memset(&sinfo, 0, sizeof(sinfo));
sta_set_sinfo(sta, &sinfo); sta_set_sinfo(sta, &sinfo);
i = 0; i = 0;
...@@ -133,7 +134,7 @@ static void ieee80211_get_stats(struct net_device *dev, ...@@ -133,7 +134,7 @@ static void ieee80211_get_stats(struct net_device *dev,
if (sta->sdata->dev != dev) if (sta->sdata->dev != dev)
continue; continue;
sinfo.filled = 0; memset(&sinfo, 0, sizeof(sinfo));
sta_set_sinfo(sta, &sinfo); sta_set_sinfo(sta, &sinfo);
i = 0; i = 0;
ADD_STA_STATS(sta); ADD_STA_STATS(sta);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment