[PATCH 1/2] Change wpa_printf to wpa_msg in scan.c
Thu Feb 10 10:58:09 PST 2011
On 02/10/2011 10:52 AM, Jouni Malinen wrote:
> On Thu, Feb 10, 2011 at 10:39:00AM -0800, Ben Greear wrote:
>> One thing about passing null to wpa_msg: Doesn't that
>> allow us to get messages over the cli interface tool, where
>> wpa_printf does not?
> No, the context pointer is needed for that to work.
>> It seemed to me that converting all wpa_printfs to wpa_msg
>> (or wpa_dbg) was going to be more flexible going forward.
> For places where the suitable context pointer is available, yes.
>> We could do a check for NULL and instead of printing
>> "NULL:" is my patches did, it could just leave out the "NULL:",
>> or maybe make more informed decisions about passing to
>> the cli, log files, etc.
> I actually already changed that to never use "NULL: " prefix.
Ok, thanks. I'll do any future wpa_printf conversion
patches as you suggest.
Ben Greear <greearb at candelatech.com>
Candela Technologies Inc http://www.candelatech.com
More information about the Hostap