On Monday 16 July 2007 11:34, Johannes Berg wrote: > On Sun, 2007-07-15 at 17:09 -0700, Michael Wu wrote: > > > > - priv->hwaddr = conf->mac_addr; > > + priv->hwaddr = conf->mac_addr ? conf->mac_addr : dev->wiphy->perm_addr; > > Do you allow monitor mode? If so, this isn't correct unless the hardware > is explicitly told to not do anything (most hw doesn't and we use a zero > mac addr for that). conf->mac_addr is NULL in monitor mode for a reason: > we don't want to have any mac address so we don't ACK packets. I'd suggest changing this behaviour in the stack, as this bug will be made again for new drivers in the future. What about pointing mac_addr to a 00:00:00:00:00 const static mac address, instead of passing a NULL pointer? - To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
