diff options
| author | Eric Dumazet <dada1@cosmosbay.com> | 2008-01-08 07:39:57 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-01-29 00:00:48 +0100 |
| commit | 40ccbf525e55fc6d1f3a88c1e98b13db4dd618db (patch) | |
| tree | 36ba0825f9c0d1797578834361b55dc844ecf744 /net/packet | |
| parent | [BLUETOOTH]: Use sockfd_put() (diff) | |
| download | linux-40ccbf525e55fc6d1f3a88c1e98b13db4dd618db.tar.xz linux-40ccbf525e55fc6d1f3a88c1e98b13db4dd618db.zip | |
[PACKET]: Fix sparse warnings in af_packet.c
CHECK net/packet/af_packet.c
net/packet/af_packet.c:1876:14: warning: context imbalance in 'packet_seq_start' - wrong count at exit
net/packet/af_packet.c:1888:13: warning: context imbalance in 'packet_seq_stop' - unexpected unlock
Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/packet')
| -rw-r--r-- | net/packet/af_packet.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/packet/af_packet.c b/net/packet/af_packet.c index 43e49f46ad43..b8b827c7062d 100644 --- a/net/packet/af_packet.c +++ b/net/packet/af_packet.c @@ -1870,6 +1870,7 @@ static inline struct sock *packet_seq_idx(struct net *net, loff_t off) } static void *packet_seq_start(struct seq_file *seq, loff_t *pos) + __acquires(seq_file_net(seq)->packet.sklist_lock) { struct net *net = seq_file_net(seq); read_lock(&net->packet.sklist_lock); @@ -1886,6 +1887,7 @@ static void *packet_seq_next(struct seq_file *seq, void *v, loff_t *pos) } static void packet_seq_stop(struct seq_file *seq, void *v) + __releases(seq_file_net(seq)->packet.sklist_lock) { struct net *net = seq_file_net(seq); read_unlock(&net->packet.sklist_lock); |