diff options
| author |   Johannes Berg <johannes@sipsolutions.net> | 2008-02-23 15:17:20 +0100 |
|---|---|---|
| committer |   John W. Linville <linville@tuxdriver.com> | 2008-03-06 15:30:42 -0500 |
| commit | dc0b0f7d1e34b797b98e4d16122b3ea6f775154c (patch) | |
| tree | 6b1bbfa17735243eb8c8f9bd394ffe169bd7c22b /net/mac80211/mesh_hwmp.c | |
| parent | mac80211: clean up mesh code (diff) | |
| download | linux-dev-dc0b0f7d1e34b797b98e4d16122b3ea6f775154c.tar.xz linux-dev-dc0b0f7d1e34b797b98e4d16122b3ea6f775154c.zip | |
mac80211: mesh hwmp locking fixes
This fixes missing unlocks noticed by sparse.
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211/mesh_hwmp.c')
| -rw-r--r-- | net/mac80211/mesh_hwmp.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/net/mac80211/mesh_hwmp.c b/net/mac80211/mesh_hwmp.c index 9a501aae48eb..c2f40ef418cf 100644 --- a/net/mac80211/mesh_hwmp.c +++ b/net/mac80211/mesh_hwmp.c @@ -264,8 +264,10 @@ static u32 hwmp_route_info_get(struct net_device *dev, rcu_read_lock(); sta = sta_info_get(local, mgmt->sa); - if (!sta) + if (!sta) { + rcu_read_unlock(); return 0; + } last_hop_metric = airtime_link_metric_get(local, sta); /* Update and check originator routing info */ @@ -293,6 +295,7 @@ static u32 hwmp_route_info_get(struct net_device *dev, break; default: sta_info_put(sta); + rcu_read_unlock(); return 0; } new_metric = orig_metric + last_hop_metric; |