diff options
| author | David S. Miller <davem@davemloft.net> | 2005-08-23 19:50:09 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2005-08-23 19:50:09 +0200 |
| commit | dc16aaf29d64b8c5e0b88f49a4d541edf5b61e42 (patch) | |
| tree | 0f1dcf6e5f9f57989d0aca1a565fa56701ed7556 /net/rose/rose_route.c | |
| parent | [TCP]: Document non-trivial locking path in tcp_v{4,6}_get_port(). (diff) | |
| download | linux-dc16aaf29d64b8c5e0b88f49a4d541edf5b61e42.tar.xz linux-dc16aaf29d64b8c5e0b88f49a4d541edf5b61e42.zip | |
[ROSE]: Fix missing unlocks in rose_route_frame()
Noticed by Coverity checker.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/rose/rose_route.c')
| -rw-r--r-- | net/rose/rose_route.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/net/rose/rose_route.c b/net/rose/rose_route.c index ff73ebb912b8..46b23217a353 100644 --- a/net/rose/rose_route.c +++ b/net/rose/rose_route.c @@ -994,8 +994,10 @@ int rose_route_frame(struct sk_buff *skb, ax25_cb *ax25) * 1. The frame isn't for us, * 2. It isn't "owned" by any existing route. */ - if (frametype != ROSE_CALL_REQUEST) /* XXX */ - return 0; + if (frametype != ROSE_CALL_REQUEST) { /* XXX */ + ret = 0; + goto out; + } len = (((skb->data[3] >> 4) & 0x0F) + 1) / 2; len += (((skb->data[3] >> 0) & 0x0F) + 1) / 2; |