devlink: double free in devlink_resource_fill()
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 21 Sep 2018 08:07:55 +0000 (11:07 +0300)
committerDavid S. Miller <davem@davemloft.net>
Sat, 22 Sep 2018 02:19:07 +0000 (19:19 -0700)
commit83fe9a966111b51a34f10c35e568e45bff34de48
tree11945dfd4538b07a1cd32e8093b20b8bbbb1b988
parente6ce3822a9f264b3f24c1acdc624131fb014f2f0
devlink: double free in devlink_resource_fill()

Smatch reports that devlink_dpipe_send_and_alloc_skb() frees the skb
on error so this is a double free.  We fixed a bunch of these bugs in
commit 7fe4d6dcbcb4 ("devlink: Remove redundant free on error path") but
we accidentally overlooked this one.

Fixes: d9f9b9a4d05f ("devlink: Add support for resource abstraction")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/devlink.c