diff options
author | Javier Martinez Canillas <javier@dowhile0.org> | 2012-06-26 22:22:20 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-06-26 22:39:37 (GMT) |
commit | 2da049bd5f9b0dbd688519fdb6688a4895fe8395 (patch) | |
tree | 038ad1358269b1883733c0b6e2017ac171b00731 /drivers/hv | |
parent | e8abca1f30239f6c979f60df49305a602672cb94 (diff) | |
download | linux-2da049bd5f9b0dbd688519fdb6688a4895fe8395.tar.xz |
staging: gdm72xx: fix an skb memory leak
The NLMSG_PUT() macro contains a hidden goto that jumps to the
nlmsg_failure label. Since the sk_buff was allocated before the macro,
jumping to the nlmsg_failure label leaks the memory allocated for it.
Calling kfree() before returning would fix it, but is better to avoid
using this error prone macro and use nlmsg_put() instead.
Also, use nlmsg_data() instead of NLMSG_DATA() to check type.
Signed-off-by: Javier Martinez Canillas <javier@dowhile0.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/hv')
0 files changed, 0 insertions, 0 deletions