diff options
| author | Pan Bian <bianpan2016@163.com> | 2017-04-23 09:09:19 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-04-24 21:51:30 +0200 |
| commit | 78302fd405769c9a9379e9adda119d533dce2eed (patch) | |
| tree | 78cc4f053d8cac193e683af4d6e2e7dca7b54fd2 /net/tipc | |
| parent | lwtunnel: check return value of nla_nest_start (diff) | |
| download | linux-78302fd405769c9a9379e9adda119d533dce2eed.tar.xz linux-78302fd405769c9a9379e9adda119d533dce2eed.zip | |
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')
| -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) int err; msg.skb = nlmsg_new(NLMSG_GOODSIZE, GFP_KERNEL); + if (!msg.skb) + return -ENOMEM; msg.portid = info->snd_portid; msg.seq = info->snd_seq; |