diff options
author | Pan Bian <bianpan2016@163.com> | 2017-04-23 03:09:19 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-04-24 15:51:30 -0400 |
commit | 78302fd405769c9a9379e9adda119d533dce2eed (patch) | |
tree | 78cc4f053d8cac193e683af4d6e2e7dca7b54fd2 /net/tipc/node.c | |
parent | a50fe0ffd76fbd17af36209a16caf0b8ad901fef (diff) |
tipc: check return value of nlmsg_new
Function nlmsg_new() will return a NULL pointer if there is no enough
memory, and its return value should be checked before it is used.
However, in function tipc_nl_node_get_monitor(), the validation of the
return value of function nlmsg_new() is missed. This patch fixes the
bug.
Signed-off-by: Pan Bian <bianpan2016@163.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/node.c')
-rw-r--r-- | net/tipc/node.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/tipc/node.c b/net/tipc/node.c index 01b1f077603e..aeef8011ac7d 100644 --- a/net/tipc/node.c +++ b/net/tipc/node.c | |||
@@ -2098,6 +2098,8 @@ int tipc_nl_node_get_monitor(struct sk_buff *skb, struct genl_info *info) | |||
2098 | int err; | 2098 | int err; |
2099 | 2099 | ||
2100 | msg.skb = nlmsg_new(NLMSG_GOODSIZE, GFP_KERNEL); | 2100 | msg.skb = nlmsg_new(NLMSG_GOODSIZE, GFP_KERNEL); |
2101 | if (!msg.skb) | ||
2102 | return -ENOMEM; | ||
2101 | msg.portid = info->snd_portid; | 2103 | msg.portid = info->snd_portid; |
2102 | msg.seq = info->snd_seq; | 2104 | msg.seq = info->snd_seq; |
2103 | 2105 | ||