On Jan 6 2008 11:22, Herbert Xu wrote: >@@ -271,6 +271,7 @@ static int raw_send_hdrinc(struct sock *sk, void *from, >size_t length, > int hh_len; > struct iphdr *iph; > struct sk_buff *skb; >+ unsigned int iphlen; > int err; > > if (length > rt->u.dst.dev->mtu) { >@@ -304,7 +305,8 @@ static int raw_send_hdrinc(struct sock *sk, void *from, >size_t length, > goto error_fault; > > /* We don't modify invalid header */ >- if (length >= sizeof(*iph) && iph->ihl * 4U <= length) { >+ iphlen = iph->ihl * 4; >+ if (iphlen >= sizeof(*iph) && iphlen <= length) {
Humm, this could use ip_hdrlen(skb) :-) -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html