diff options
| author | Denis V. Lunev <den@openvz.org> | 2008-07-10 16:53:39 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-07-10 16:53:39 -0700 |
| commit | fe785bee05f08d37b34b7399d003b74199274ce4 (patch) | |
| tree | d87c30deefc4c35d5794879b3430470d554ae2a6 /net/netlabel/netlabel_unlabeled.c | |
| parent | 2e655571c618434c24ac2ca989374fdd84470d6d (diff) | |
| download | lwn-fe785bee05f08d37b34b7399d003b74199274ce4.tar.gz lwn-fe785bee05f08d37b34b7399d003b74199274ce4.zip | |
netlabel: netlink_unicast calls kfree_skb on error path by itself
So, no need to kfree_skb here on the error path. In this case we can
simply return.
Signed-off-by: Denis V. Lunev <den@openvz.org>
Acked-by: Paul Moore <paul.moore@hp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/netlabel/netlabel_unlabeled.c')
| -rw-r--r-- | net/netlabel/netlabel_unlabeled.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/net/netlabel/netlabel_unlabeled.c b/net/netlabel/netlabel_unlabeled.c index 52b2611a6eb6..56f80872924e 100644 --- a/net/netlabel/netlabel_unlabeled.c +++ b/net/netlabel/netlabel_unlabeled.c @@ -1107,11 +1107,7 @@ static int netlbl_unlabel_list(struct sk_buff *skb, struct genl_info *info) goto list_failure; genlmsg_end(ans_skb, data); - - ret_val = genlmsg_reply(ans_skb, info); - if (ret_val != 0) - goto list_failure; - return 0; + return genlmsg_reply(ans_skb, info); list_failure: kfree_skb(ans_skb); |