ipv6:remove useless check
authorYang Hongyang <yanghy@cn.fujitsu.com>
Mon, 13 Apr 2009 15:51:00 +0000 (15:51 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 14 Apr 2009 09:21:41 +0000 (02:21 -0700)
After switch (rthdr->type) {...},the check below is completely useless.Because:
if the type is 2,then hdrlen must be 2 and segments_left must be 1,clearly the
check is redundant;if the type is not 2,then goto sticky_done,the check is useless
too.

Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com>
Reviewed-by: Shan Wei <shanwei@cn.fujitsu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/ipv6_sockglue.c

index d31df0f4bc9aa3ce017dc798a2af3be61a7a5c9a..a7fdf9a27f15e4b16cf36cb714f58e238921f451 100644 (file)
@@ -380,10 +380,6 @@ static int do_ipv6_setsockopt(struct sock *sk, int level, int optname,
                        default:
                                goto sticky_done;
                        }
-
-                       if ((rthdr->hdrlen & 1) ||
-                           (rthdr->hdrlen >> 1) != rthdr->segments_left)
-                               goto sticky_done;
                }
 
                retv = 0;