Messages in this thread | | | Date | Thu, 14 Nov 2013 18:29:02 -0200 | Subject | Re: [PATCH] net/hsr: Fix possible leak in 'hsr_get_node_status()' | From | Geyslan Gregório Bem <> |
| |
2013/11/14 Geyslan G. Bem <geyslan@gmail.com>: > If 'hsr_get_node_data()' returns error, going directly to 'fail' label > doesn't free the memory pointed by 'skb_out'. > > Signed-off-by: Geyslan G. Bem <geyslan@gmail.com> > --- > net/hsr/hsr_netlink.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/net/hsr/hsr_netlink.c b/net/hsr/hsr_netlink.c > index 4e66bf6..aef8429 100644 > --- a/net/hsr/hsr_netlink.c > +++ b/net/hsr/hsr_netlink.c > @@ -249,7 +249,7 @@ static int hsr_get_node_status(struct sk_buff *skb_in, struct genl_info *info) > &hsr_node_if2_age, > &hsr_node_if2_seq); > if (res < 0) > - goto fail; > + goto nla_put_failure; > > res = nla_put(skb_out, HSR_A_NODE_ADDR, ETH_ALEN, > nla_data(info->attrs[HSR_A_NODE_ADDR])); > -- > 1.8.4.2 >
It's a Coverity catch: CID 1128855.
-- Regards,
Geyslan G. Bem hackingbits.com
| |