> + /* verify MAC addr is valid */
> + if (!is_zero_ether_addr(mac) && !is_valid_ether_addr(mac) &&
> + is_multicast_ether_addr(mac)) {This is really odd as verification goes; Currently this is a very elaborate way of checking for multicast, but I guess it's probably a mistake.
