[PATCH 1/2] Change wpa_printf to wpa_msg in scan.c

Ben Greear greearb
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 mailing list