diff options
author | Gerrit Renker <gerrit@erg.abdn.ac.uk> | 2006-11-13 16:31:50 +0100 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2006-12-03 06:22:21 +0100 |
commit | 865e9022d88ceedd89fa1079a6e1f9266ccd3711 (patch) | |
tree | bc276752f832226f9a506f1fe65bff000bea557e /drivers/net | |
parent | [DCCP]: Remove forward declarations in timer.c (diff) | |
download | linux-865e9022d88ceedd89fa1079a6e1f9266ccd3711.tar.xz linux-865e9022d88ceedd89fa1079a6e1f9266ccd3711.zip |
[DCCP]: Remove redundant statements in init_sequence (ISS)
This patch removes the following redundancies:
1) The test skb->protocol == htons(ETH_P_IPV6) in dccp_v6_init_sequence
is always true since
* dccp_v6_conn_request() is the only calling function
* dccp_v6_conn_request() redirects all skb's with ETH_P_IP to
dccp_v4_conn_request()
2) The first argument, `struct sock *sk', of dccp_v{4,6}_init_sequence()
is never used.
(This is similar for tcp_v{4,6}_init_sequence, an analogous patch has been
submitted to netdev and merged.)
By the way - are the `sport' / `dport' arguments in the right order?
I have made them consistent among calls but they seem to be in the
reverse order.
Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>
Signed-off-by: Arnaldo Carvalho de Melo <acme@mandriva.com>
Diffstat (limited to 'drivers/net')
0 files changed, 0 insertions, 0 deletions