Commit fcf801cc authored by Yunfeng Ye's avatar Yunfeng Ye Committed by Herbert Xu

crypto: chtls - remove the redundant check in chtls_recvmsg()

A warning message reported by a static analysis tool:
  "
  Either the condition 'if(skb)' is redundant or there is possible null
  pointer dereference: skb.
  "

Remove the unused redundant check.
Signed-off-by: default avatarYunfeng Ye <yeyunfeng@huawei.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent b981744e
...@@ -1841,8 +1841,7 @@ int chtls_recvmsg(struct sock *sk, struct msghdr *msg, size_t len, ...@@ -1841,8 +1841,7 @@ int chtls_recvmsg(struct sock *sk, struct msghdr *msg, size_t len,
tp->urg_data = 0; tp->urg_data = 0;
if (avail + offset >= skb->len) { if (avail + offset >= skb->len) {
if (likely(skb)) chtls_free_skb(sk, skb);
chtls_free_skb(sk, skb);
buffers_freed++; buffers_freed++;
if (copied >= target && if (copied >= target &&
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment