[PATCH 3/3] nl80211: make eloop sockets non-blocking
Johannes Berg
johannes
Sat Oct 26 03:23:52 PDT 2013
From: Johannes Berg <johannes.berg at intel.com>
To avoid a problem where the beacon socket occasionally
blocks, mark any sockets on the eloop as non-blocking.
The previous patch reordered the code to never send a
command after a socket was put on the eloop, but now also
invalidate the nl handle pointer while it's on there.
Signed-hostap: Johannes Berg <johannes.berg at intel.com>
---
src/drivers/driver_nl80211.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/src/drivers/driver_nl80211.c b/src/drivers/driver_nl80211.c
index 2021a9c..8ce5c2a 100644
--- a/src/drivers/driver_nl80211.c
+++ b/src/drivers/driver_nl80211.c
@@ -139,16 +139,25 @@ static void nl_destroy_handles(struct nl_handle **handle)
*handle = NULL;
}
+#if __WORDSIZE == 64
+#define ELOOP_SOCKET_INVALID 0x8888888888888889ULL
+#else
+#define ELOOP_SOCKET_INVALID 0x88888889ULL
+#endif
+
static void nl80211_register_eloop_read(struct nl_handle **handle,
eloop_sock_handler handler,
void *eloop_data)
{
+ nl_socket_set_nonblocking(*handle);
eloop_register_read_sock(nl_socket_get_fd(*handle), handler,
eloop_data, *handle);
+ *handle = (void *)(((unsigned long)*handle) ^ ELOOP_SOCKET_INVALID);
}
static void nl80211_destroy_eloop_handle(struct nl_handle **handle)
{
+ *handle = (void *)(((unsigned long)*handle) ^ ELOOP_SOCKET_INVALID);
eloop_unregister_read_sock(nl_socket_get_fd(*handle));
nl_destroy_handles(handle);
}
--
1.8.4.rc3
More information about the Hostap
mailing list