Commit 865e9022 authored by Gerrit Renker's avatar Gerrit Renker Committed by David S. Miller

[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: default avatarGerrit Renker <gerrit@erg.abdn.ac.uk>
Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@mandriva.com>
parent 4ed800d0
...@@ -366,8 +366,7 @@ void dccp_v4_send_check(struct sock *sk, int unused, struct sk_buff *skb) ...@@ -366,8 +366,7 @@ void dccp_v4_send_check(struct sock *sk, int unused, struct sk_buff *skb)
EXPORT_SYMBOL_GPL(dccp_v4_send_check); EXPORT_SYMBOL_GPL(dccp_v4_send_check);
static inline u64 dccp_v4_init_sequence(const struct sock *sk, static inline u64 dccp_v4_init_sequence(const struct sk_buff *skb)
const struct sk_buff *skb)
{ {
return secure_dccp_sequence_number(skb->nh.iph->daddr, return secure_dccp_sequence_number(skb->nh.iph->daddr,
skb->nh.iph->saddr, skb->nh.iph->saddr,
...@@ -663,7 +662,7 @@ int dccp_v4_conn_request(struct sock *sk, struct sk_buff *skb) ...@@ -663,7 +662,7 @@ int dccp_v4_conn_request(struct sock *sk, struct sk_buff *skb)
*/ */
dreq = dccp_rsk(req); dreq = dccp_rsk(req);
dreq->dreq_isr = dcb->dccpd_seq; dreq->dreq_isr = dcb->dccpd_seq;
dreq->dreq_iss = dccp_v4_init_sequence(sk, skb); dreq->dreq_iss = dccp_v4_init_sequence(skb);
dreq->dreq_service = service; dreq->dreq_service = service;
if (dccp_v4_send_response(sk, req, NULL)) if (dccp_v4_send_response(sk, req, NULL))
......
...@@ -76,20 +76,12 @@ static inline void dccp_v6_send_check(struct sock *sk, int unused_value, ...@@ -76,20 +76,12 @@ static inline void dccp_v6_send_check(struct sock *sk, int unused_value,
dh->dccph_checksum = dccp_v6_csum_finish(skb, &np->saddr, &np->daddr); dh->dccph_checksum = dccp_v6_csum_finish(skb, &np->saddr, &np->daddr);
} }
static __u32 dccp_v6_init_sequence(struct sock *sk, struct sk_buff *skb) static inline __u32 dccp_v6_init_sequence(const struct sk_buff *skb)
{ {
const struct dccp_hdr *dh = dccp_hdr(skb); return secure_tcpv6_sequence_number(skb->nh.ipv6h->daddr.s6_addr32,
skb->nh.ipv6h->saddr.s6_addr32,
if (skb->protocol == htons(ETH_P_IPV6)) dccp_hdr(skb)->dccph_dport,
return secure_tcpv6_sequence_number(skb->nh.ipv6h->daddr.s6_addr32, dccp_hdr(skb)->dccph_sport );
skb->nh.ipv6h->saddr.s6_addr32,
dh->dccph_dport,
dh->dccph_sport);
return secure_dccp_sequence_number(skb->nh.iph->daddr,
skb->nh.iph->saddr,
dh->dccph_dport,
dh->dccph_sport);
} }
static void dccp_v6_err(struct sk_buff *skb, struct inet6_skb_parm *opt, static void dccp_v6_err(struct sk_buff *skb, struct inet6_skb_parm *opt,
...@@ -491,7 +483,7 @@ static int dccp_v6_conn_request(struct sock *sk, struct sk_buff *skb) ...@@ -491,7 +483,7 @@ static int dccp_v6_conn_request(struct sock *sk, struct sk_buff *skb)
*/ */
dreq = dccp_rsk(req); dreq = dccp_rsk(req);
dreq->dreq_isr = dcb->dccpd_seq; dreq->dreq_isr = dcb->dccpd_seq;
dreq->dreq_iss = dccp_v6_init_sequence(sk, skb); dreq->dreq_iss = dccp_v6_init_sequence(skb);
dreq->dreq_service = service; dreq->dreq_service = service;
if (dccp_v6_send_response(sk, req, NULL)) if (dccp_v6_send_response(sk, req, NULL))
......
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