diff options
author | Gerrit Renker <gerrit@erg.abdn.ac.uk> | 2007-10-04 17:42:19 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-10-10 19:54:35 -0400 |
commit | 6c583248083c30c5305ec561e79f666ca465b376 (patch) | |
tree | b23e2787e6ce0208395aa7b879b924470592c8e3 /net/dccp | |
parent | ee196c2186d24d82088c94962598470e5abc081f (diff) |
[CCID2]: Remove redundant case block
skb's passed to ccid2_hc_tx_send_packet() are headerless, the packet
type is decided later, in dccp_write_xmit(). Therefore the first test
of the switch/case block is always true, the others are never reached.
Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dccp')
-rw-r--r-- | net/dccp/ccids/ccid2.c | 14 |
1 files changed, 1 insertions, 13 deletions
diff --git a/net/dccp/ccids/ccid2.c b/net/dccp/ccids/ccid2.c index 5114a2d30bfd..1dff4188f3f5 100644 --- a/net/dccp/ccids/ccid2.c +++ b/net/dccp/ccids/ccid2.c | |||
@@ -126,19 +126,7 @@ static int ccid2_hc_tx_alloc_seq(struct ccid2_hc_tx_sock *hctx) | |||
126 | 126 | ||
127 | static int ccid2_hc_tx_send_packet(struct sock *sk, struct sk_buff *skb) | 127 | static int ccid2_hc_tx_send_packet(struct sock *sk, struct sk_buff *skb) |
128 | { | 128 | { |
129 | struct ccid2_hc_tx_sock *hctx; | 129 | struct ccid2_hc_tx_sock *hctx = ccid2_hc_tx_sk(sk); |
130 | |||
131 | switch (DCCP_SKB_CB(skb)->dccpd_type) { | ||
132 | case 0: /* XXX data packets from userland come through like this */ | ||
133 | case DCCP_PKT_DATA: | ||
134 | case DCCP_PKT_DATAACK: | ||
135 | break; | ||
136 | /* No congestion control on other packets */ | ||
137 | default: | ||
138 | return 0; | ||
139 | } | ||
140 | |||
141 | hctx = ccid2_hc_tx_sk(sk); | ||
142 | 130 | ||
143 | ccid2_pr_debug("pipe=%d cwnd=%d\n", hctx->ccid2hctx_pipe, | 131 | ccid2_pr_debug("pipe=%d cwnd=%d\n", hctx->ccid2hctx_pipe, |
144 | hctx->ccid2hctx_cwnd); | 132 | hctx->ccid2hctx_cwnd); |