Hi Shinoda, On Mon, Aug 27, 2012 at 6:28 AM, LEO Airwarosu Yoichi Shinoda <[email protected]> wrote: > > Various small fixes for net/mac80211/cfg.c:mpath_set_pinfo(): > Initialize *pinfo before filling members in, handle MESH_PATH_RESOLVED > correctly, and remove bogus assignment; result in correct display > of FLAGS values and meaningful EXPTIME for expired paths in iw utility. > > Signed-off-by: Yoichi Shinoda <[email protected]> Acked-by: Thomas Pedersen <[email protected]> > --- > net/mac80211/cfg.c | 9 ++++----- > 1 files changed, 4 insertions(+), 5 deletions(-) > > diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c > index d41974a..d69e38b 100644 > --- a/net/mac80211/cfg.c > +++ b/net/mac80211/cfg.c > @@ -1378,6 +1378,8 @@ static void mpath_set_pinfo(struct mesh_path *mpath, u8 > *next_hop, > else > memset(next_hop, 0, ETH_ALEN); > > + memset(pinfo, 0, sizeof(struct mpath_info)); > +
This is probably up to personal preference, but IMO 'sizeof(*p)' is a little nicer since the compiler already knows the pointer type. > pinfo->generation = mesh_paths_generation; > > pinfo->filled = MPATH_INFO_FRAME_QLEN | > @@ -1396,7 +1398,6 @@ static void mpath_set_pinfo(struct mesh_path *mpath, u8 > *next_hop, > pinfo->discovery_timeout = > jiffies_to_msecs(mpath->discovery_timeout); > pinfo->discovery_retries = mpath->discovery_retries; > - pinfo->flags = 0; > if (mpath->flags & MESH_PATH_ACTIVE) > pinfo->flags |= NL80211_MPATH_FLAG_ACTIVE; > if (mpath->flags & MESH_PATH_RESOLVING) > @@ -1405,10 +1406,8 @@ static void mpath_set_pinfo(struct mesh_path *mpath, > u8 *next_hop, > pinfo->flags |= NL80211_MPATH_FLAG_SN_VALID; > if (mpath->flags & MESH_PATH_FIXED) > pinfo->flags |= NL80211_MPATH_FLAG_FIXED; > - if (mpath->flags & MESH_PATH_RESOLVING) > - pinfo->flags |= NL80211_MPATH_FLAG_RESOLVING; > - > - pinfo->flags = mpath->flags; > + if (mpath->flags & MESH_PATH_RESOLVED) > + pinfo->flags |= NL80211_MPATH_FLAG_RESOLVED; > } > > static int ieee80211_get_mpath(struct wiphy *wiphy, struct net_device *dev, > -- > 1.7.4.1 > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html _______________________________________________ Devel mailing list [email protected] http://lists.open80211s.org/cgi-bin/mailman/listinfo/devel
