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 sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html