On 11/23/06, Gerrit Renker <[EMAIL PROTECTED]> wrote:

@@ -1022,6 +1047,9 @@ static void ccid3_hc_rx_packet_recv(stru
        if (DCCP_SKB_CB(skb)->dccpd_type == DCCP_PKT_ACK)
                return;

+       payload_size = skb->len - dccp_hdr(skb)->dccph_doff * 4;

Should't we use (skb->len - dccp_hdr_len(skb)) to take into account
the different DCCP packet size header lengths? We can get payload in
other kinds of packets than just DATA, and even on DATAACK we should
not compute the sizeof(struct dccp_hdr_ack_bits) bytes into the
payload, no?

- Arnaldo
-
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to