diff options
| author | luo penghao <luo.penghao@zte.com.cn> | 2021-11-04 07:26:21 +0100 |
|---|---|---|
| committer | Steffen Klassert <steffen.klassert@secunet.com> | 2021-11-18 19:15:05 +0100 |
| commit | 2e1809208a4a52e883527c288b4fca61a90100b2 (patch) | |
| tree | b4798a876bac4c6fe975d3d02e5fd7a8442e04f5 /net/xfrm | |
| parent | ipv6/esp6: Remove structure variables and alignment statements (diff) | |
| download | linux-2e1809208a4a52e883527c288b4fca61a90100b2.tar.xz linux-2e1809208a4a52e883527c288b4fca61a90100b2.zip | |
xfrm: Remove duplicate assignment
The statement in the switch is repeated with the statement at the
beginning of the while loop, so this statement is meaningless.
The clang_analyzer complains as follows:
net/xfrm/xfrm_policy.c:3392:2 warning:
Value stored to 'exthdr' is never read
Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: luo penghao <luo.penghao@zte.com.cn>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
Diffstat (limited to 'net/xfrm')
| -rw-r--r-- | net/xfrm/xfrm_policy.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/net/xfrm/xfrm_policy.c b/net/xfrm/xfrm_policy.c index 1a06585022ab..edc673e78114 100644 --- a/net/xfrm/xfrm_policy.c +++ b/net/xfrm/xfrm_policy.c @@ -3392,7 +3392,6 @@ decode_session6(struct sk_buff *skb, struct flowi *fl, bool reverse) case NEXTHDR_DEST: offset += ipv6_optlen(exthdr); nexthdr = exthdr->nexthdr; - exthdr = (struct ipv6_opt_hdr *)(nh + offset); break; case IPPROTO_UDP: case IPPROTO_UDPLITE: |