summaryrefslogtreecommitdiffstats
path: root/net/caif
diff options
context:
space:
mode:
authorYueHaibing <yuehaibing@huawei.com>2018-07-19 04:27:13 +0200
committerDavid S. Miller <davem@davemloft.net>2018-07-22 01:14:39 +0200
commit64119e05f7b31e83e2555f6782e6cdc8f81c63f4 (patch)
tree87d44bf9e7cecbc6fa4cbb7e6d45f8ec32b62b4a /net/caif
parentbonding: set default miimon value for non-arp modes if not set (diff)
downloadlinux-64119e05f7b31e83e2555f6782e6cdc8f81c63f4.tar.xz
linux-64119e05f7b31e83e2555f6782e6cdc8f81c63f4.zip
net: caif: Add a missing rcu_read_unlock() in caif_flow_cb
Add a missing rcu_read_unlock in the error path Fixes: c95567c80352 ("caif: added check for potential null return") Signed-off-by: YueHaibing <yuehaibing@huawei.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/caif')
-rw-r--r--net/caif/caif_dev.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/net/caif/caif_dev.c b/net/caif/caif_dev.c
index e0adcd123f48..711d7156efd8 100644
--- a/net/caif/caif_dev.c
+++ b/net/caif/caif_dev.c
@@ -131,8 +131,10 @@ static void caif_flow_cb(struct sk_buff *skb)
caifd = caif_get(skb->dev);
WARN_ON(caifd == NULL);
- if (caifd == NULL)
+ if (!caifd) {
+ rcu_read_unlock();
return;
+ }
caifd_hold(caifd);
rcu_read_unlock();