Re: [PATCH] mesh: Avoid STA expiration timer truncation to u32

2015-01-15 Thread Masashi Honma
2015-01-16 14:49 GMT+09:00 Kalle Valo :
> For mac80211 patches please use prefix "mac80211: ".

Thanks. I will fix it.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] mesh: Avoid STA expiration timer truncation to u32

2015-01-15 Thread Kalle Valo
Masashi Honma  writes:

> On some combination of plink_timeout and HZ, the STA expiration timer will be
> unexpectedly truncated to u32. Maybe there is a question "Who sets such a 
> large
> number to plink_timeout ?". At least wpa_supplicant will set 0x to
> plink_timeout to disable this timer because wpa_supplicant has it's own
> expiration mechanism.
>
> Signed-off-by: Masashi Honma 

For mac80211 patches please use prefix "mac80211: ".

-- 
Kalle Valo
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] mesh: Avoid STA expiration timer truncation to u32

2015-01-15 Thread Masashi Honma
On some combination of plink_timeout and HZ, the STA expiration timer will be
unexpectedly truncated to u32. Maybe there is a question "Who sets such a large
number to plink_timeout ?". At least wpa_supplicant will set 0x to
plink_timeout to disable this timer because wpa_supplicant has it's own
expiration mechanism.

Signed-off-by: Masashi Honma 
---
 net/mac80211/mesh.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/net/mac80211/mesh.c b/net/mac80211/mesh.c
index 0c8b2a7..bb721a0 100644
--- a/net/mac80211/mesh.c
+++ b/net/mac80211/mesh.c
@@ -574,7 +574,8 @@ static void ieee80211_mesh_housekeeping(struct 
ieee80211_sub_if_data *sdata)
struct ieee80211_if_mesh *ifmsh = &sdata->u.mesh;
u32 changed;
 
-   ieee80211_sta_expire(sdata, ifmsh->mshcfg.plink_timeout * HZ);
+   ieee80211_sta_expire(sdata, ifmsh->mshcfg.plink_timeout *
+(unsigned long)HZ);
mesh_path_expire(sdata);
 
changed = mesh_accept_plinks_update(sdata);
-- 
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html