[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 3 Mar 2010 08:36:03 +0100
From: Jiri Pirko <jpirko@...hat.com>
To: Pavel Roskin <proski@....org>
Cc: netdev@...r.kernel.org, davem@...emloft.net
Subject: Re: [net-2.6 PATCH] af_packet: move strict addr_len check right
before dev_[mc/unicast]_[add/del]
Wed, Mar 03, 2010 at 08:01:10AM CET, proski@....org wrote:
>Quoting Jiri Pirko <jpirko@...hat.com>:
>
>>@@ -1734,7 +1738,7 @@ static int packet_mc_add(struct sock *sk,
>>struct packet_mreq_max *mreq)
>> goto done;
>>
>> err = -EINVAL;
>>- if (mreq->mr_alen != dev->addr_len)
>>+ if (mreq->mr_alen > dev->addr_len)
>> goto done;
>>
>> err = -ENOBUFS;
>
>The patch looks good, but did you mean to include this change? It's
>not described.
Sure - this is revert of the bit from my original patch. I think it's clear from
description.
Jirka
>
>--
>Regards,
>Pavel Roskin
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists