[PATCH] Remove references to time_t/time()
Jouni Malinen
j
Mon Sep 12 12:27:19 PDT 2011
On Mon, Sep 12, 2011 at 04:33:11PM +0200, Per Ekman wrote:
> I encountered some more calls to time().
Thanks, applied.
> Would you consider removing the
> <time.h> include from utils/includes.h? That would catch these
> calls. With a vanilla build only os_unix.c complains and perhaps it
> should include time.h directly.
There are couple of other files that need time.h for functions that do
not have os_*() wrappers. Anyway, those can include time.h directly, so
yes, I did remove time.h from includes.h. It should be noted that
sys/time.h is also getting included there, but that had some more ifdef
processing around it, so I did not want to move that to os_unix.c.
> As an aside, trunk does not seem to build without WPS enabled
> because drv_callbacks.c:hostapd_notif_assoc() calls wps_is_20()
> which is not compiled when WPS is disabled.
Thanks! Fixed.
--
Jouni Malinen PGP id EFC895FA
More information about the Hostap
mailing list