On Fri, 6 Jan 2006, Patrick McHardy wrote:
> Kris Katterjohn wrote:
> > This uses is_multicast_ether_addr() because it has recently been changed to
> > do
> > the same thing these seperate tests are doing.
>
> > --- x/net/atm/br2684.c 2006-01-02 21:21:10.000000000 -0600
> > +++ y/net/atm/br2684.c 2006-01-06 12:34:47.000000000 -0600
> > @@ -295,7 +295,7 @@ static inline __be16 br_type_trans(struc
> > unsigned char *rawp;
> > eth = eth_hdr(skb);
> >
> > - if (*eth->h_dest & 1) {
> > + if (is_multicast_ether_addr(eth->h_dest)) {
>
> This is not equivalent, is_multicast_ether_addr() ignores
> addresses starting with 0xff.
It used to. Not today afaict.
--
~Randy
-
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
