Return-path: Received: from mx4.wp.pl ([212.77.101.8]:13858 "EHLO mx4.wp.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753333Ab3FGOEE (ORCPT ); Fri, 7 Jun 2013 10:04:04 -0400 Date: Fri, 7 Jun 2013 16:04:02 +0200 From: Jakub =?UTF-8?B?S2ljacWEc2tp?= To: Felix Fietkau Cc: linux-wireless@vger.kernel.org, Helmut Schaa , Johannes Berg , nietrywialneprzejscie@gmail.com Subject: Re: [RFC 4/5] mac80211: enforce address verification on monitors Message-ID: <20130607160402.49ea16c3@north> (sfid-20130607_160409_657773_53C9857A) In-Reply-To: <51B1E4F3.9020509@openwrt.org> References: <1370612179-24385-1-git-send-email-moorray3@wp.pl> <1370612179-24385-5-git-send-email-moorray3@wp.pl> <51B1E4F3.9020509@openwrt.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-wireless-owner@vger.kernel.org List-ID: On Fri, 07 Jun 2013 15:49:39 +0200, Felix Fietkau wrote: > On 2013-06-07 3:36 PM, Jakub Kicinski wrote: >> From: Jakub Kicinski >> >> Mac address of passive monitor have to treated >> the same as address any other interface because >> type of interface can be changed or monitor can >> be later put into active mode. > It can't. There's checks preventing that. I'd prefer to leave that part > of the code as it is, the MAC address for normal monitor interfaces > should be ignored completely. I must have misread the code regarding active monitors then. Main motivation behind this patch was the first part though - user can change type of interface and once ignored address of monitor now becomes address of AP, sta etc. -- kuba