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

Reply via email to