| > - if (skb->len < (iph->ihl << 2) + 8) {
| > + if (skb->len < (iph->ihl << 2) + __dccp_basic_hdr_len(dh)) {
| > ICMP_INC_STATS_BH(net, ICMP_MIB_INERRORS);
| > return;
| > }
|
| You can't dereference "dh" before you know there is even
| space past offset "iph->ihl << 2". Yet that is what doing
| an unconditional __dccp_basic_hdr_len() call here is going
| to do.
|
Oh that was my fault. Thanks a lot for pointing this out.
Will work out a fixed/improved version for both patches,
test and then resubmit.
Gerrit
--
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
