diff options
author | J. Bruce Fields <bfields@redhat.com> | 2012-08-17 20:32:27 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2012-08-21 17:07:50 -0400 |
commit | f23abfdb94fda3108441530cb4a813088d3f9176 (patch) | |
tree | 31ae521555cb313a168e6adc304121cf39f0ac00 /net/sunrpc | |
parent | a10fded18eb275e7e97e57e7542b16c665fe61ec (diff) |
svcrpc: minor udp code cleanup
Order the code in a more boring way.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'net/sunrpc')
-rw-r--r-- | net/sunrpc/svcsock.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/net/sunrpc/svcsock.c b/net/sunrpc/svcsock.c index c7a7b14f54ed..06ae8a755349 100644 --- a/net/sunrpc/svcsock.c +++ b/net/sunrpc/svcsock.c | |||
@@ -620,10 +620,7 @@ static int svc_udp_recvfrom(struct svc_rqst *rqstp) | |||
620 | if (!svc_udp_get_dest_address(rqstp, cmh)) { | 620 | if (!svc_udp_get_dest_address(rqstp, cmh)) { |
621 | net_warn_ratelimited("svc: received unknown control message %d/%d; dropping RPC reply datagram\n", | 621 | net_warn_ratelimited("svc: received unknown control message %d/%d; dropping RPC reply datagram\n", |
622 | cmh->cmsg_level, cmh->cmsg_type); | 622 | cmh->cmsg_level, cmh->cmsg_type); |
623 | out_free: | 623 | goto out_free; |
624 | trace_kfree_skb(skb, svc_udp_recvfrom); | ||
625 | skb_free_datagram_locked(svsk->sk_sk, skb); | ||
626 | return 0; | ||
627 | } | 624 | } |
628 | rqstp->rq_daddrlen = svc_addr_len(svc_daddr(rqstp)); | 625 | rqstp->rq_daddrlen = svc_addr_len(svc_daddr(rqstp)); |
629 | 626 | ||
@@ -662,6 +659,10 @@ out_free: | |||
662 | serv->sv_stats->netudpcnt++; | 659 | serv->sv_stats->netudpcnt++; |
663 | 660 | ||
664 | return len; | 661 | return len; |
662 | out_free: | ||
663 | trace_kfree_skb(skb, svc_udp_recvfrom); | ||
664 | skb_free_datagram_locked(svsk->sk_sk, skb); | ||
665 | return 0; | ||
665 | } | 666 | } |
666 | 667 | ||
667 | static int | 668 | static int |