diff options
| author | Hannes Eder <hannes@hanneseder.net> | 2009-02-14 12:48:44 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-02-18 04:34:49 +0100 |
| commit | b1ae1e99bd9f611f16a09c1bca2a0a38e67b88c3 (patch) | |
| tree | df9b742f01d081f4767041327d121cf1b2a1a32f /drivers/net/wireless/strip.c | |
| parent | drivers/net/wan: fix sparse warning: symbol shadows an earlier one (diff) | |
| download | linux-b1ae1e99bd9f611f16a09c1bca2a0a38e67b88c3.tar.xz linux-b1ae1e99bd9f611f16a09c1bca2a0a38e67b88c3.zip | |
drivers/net/wireless: fix sparse warning: context imbalance
Impact: Attribute function with __acquires(...) resp. __releases(...).
Fix this sparse warnings:
drivers/net/wireless/strip.c:955:21: warning: context imbalance in 'strip_seq_start' - wrong count at exit
drivers/net/wireless/strip.c:975:13: warning: context imbalance in 'strip_seq_stop' - unexpected unlock
Signed-off-by: Hannes Eder <hannes@hanneseder.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wireless/strip.c')
| -rw-r--r-- | drivers/net/wireless/strip.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/wireless/strip.c b/drivers/net/wireless/strip.c index d6bf8d2ef8ea..39f525efdc8d 100644 --- a/drivers/net/wireless/strip.c +++ b/drivers/net/wireless/strip.c @@ -950,6 +950,7 @@ static struct strip *strip_get_idx(loff_t pos) } static void *strip_seq_start(struct seq_file *seq, loff_t *pos) + __acquires(RCU) { rcu_read_lock(); return *pos ? strip_get_idx(*pos - 1) : SEQ_START_TOKEN; @@ -973,6 +974,7 @@ static void *strip_seq_next(struct seq_file *seq, void *v, loff_t *pos) } static void strip_seq_stop(struct seq_file *seq, void *v) + __releases(RCU) { rcu_read_unlock(); } |