Fix eloop namespace

Makefile

@@ -6,7 +6,7 @@ ROLE = dut
 VERSION = "2.1.0.42"
 
 OBJS = main.o eloop.o indigo_api.o indigo_packet.o utils.o wpa_ctrl.o
-CFLAGS += -g
+CFLAGS += -g -Wall -Wextra -Wpedantic -Werror
 
 ifeq ($(TYPE),laptop)
 CC = gcc

eloop.c

@@ -70,14 +70,14 @@ struct eloop_data {
 static struct eloop_data eloop;
 
 
-void eloop_init(void *user_data)
+void qt_eloop_init(void *user_data)
 {
 	memset(&eloop, 0, sizeof(eloop));
 	eloop.user_data = user_data;
 }
 
 
-int eloop_register_read_sock(int sock,
+int qt_eloop_register_read_sock(int sock,
 			     void (*handler)(int sock, void *eloop_ctx,
 					     void *sock_ctx),
 			     void *eloop_data, void *user_data)
@@ -103,7 +103,7 @@ int eloop_register_read_sock(int sock,
 }
 
 
-void eloop_unregister_read_sock(int sock)
+void qt_eloop_unregister_read_sock(int sock)
 {
 	int i;
 
@@ -125,7 +125,7 @@ void eloop_unregister_read_sock(int sock)
 }
 
 
-int eloop_register_timeout(unsigned int secs, unsigned int usecs,
+int qt_eloop_register_timeout(unsigned int secs, unsigned int usecs,
 			   void (*handler)(void *eloop_ctx, void *timeout_ctx),
 			   void *eloop_data, void *user_data)
 {
@@ -172,7 +172,7 @@ int eloop_register_timeout(unsigned int secs, unsigned int usecs,
 }
 
 
-int eloop_cancel_timeout(void (*handler)(void *eloop_ctx, void *sock_ctx),
+int qt_eloop_cancel_timeout(void (*handler)(void *eloop_ctx, void *sock_ctx),
 			 void *eloop_data, void *user_data)
 {
 	struct eloop_timeout *timeout, *prev, *next;
@@ -207,6 +207,7 @@ int eloop_cancel_timeout(void (*handler)(void *eloop_ctx, void *sock_ctx),
 #ifndef CONFIG_NATIVE_WINDOWS
 static void eloop_handle_alarm(int sig)
 {
+	(void) sig;
 	fprintf(stderr, "eloop: could not process SIGINT or SIGTERM in two "
 		"seconds. Looks like there\n"
 		"is a bug that ends up in a busy loop that "
@@ -268,7 +269,7 @@ static void eloop_process_pending_signals(void)
 }
 
 
-int eloop_register_signal(int sig,
+int qt_eloop_register_signal(int sig,
 			  void (*handler)(int sig, void *eloop_ctx,
 					  void *signal_ctx),
 			  void *user_data)
@@ -294,15 +295,15 @@ int eloop_register_signal(int sig,
 }
 
 
-void eloop_run(void)
+void qt_eloop_run(void)
 {
 	fd_set *rfds;
 	int i, res;
 	struct timeval tv, now;
 
 	rfds = malloc(sizeof(*rfds));
 	if (rfds == NULL) {
-		printf("eloop_run - malloc failed\n");
+		printf("qt_eloop_run - malloc failed\n");
 		return;
 	}
 
@@ -364,13 +365,13 @@ void eloop_run(void)
 }
 
 
-void eloop_terminate(void)
+void qt_eloop_terminate(void)
 {
 	eloop.terminate = 1;
 }
 
 
-void eloop_destroy(void)
+void qt_eloop_destroy(void)
 {
 	struct eloop_timeout *timeout, *prev;
 
@@ -385,7 +386,7 @@ void eloop_destroy(void)
 }
 
 
-int eloop_terminated(void)
+int qt_eloop_terminated(void)
 {
 	return eloop.terminate;
 }

eloop.h

@@ -23,21 +23,21 @@
 #ifndef ELOOP_H
 #define ELOOP_H
 
-/* Magic number for eloop_cancel_timeout() */
+/* Magic number for qt_eloop_cancel_timeout() */
 #define ELOOP_ALL_CTX (void *) -1
 
 /**
- * eloop_init() - Initialize global event loop data
+ * qt_eloop_init() - Initialize global event loop data
  * @user_data: Pointer to global data passed as eloop_ctx to signal handlers
  *
  * This function must be called before any other eloop_* function. user_data
  * can be used to configure a global (to the process) pointer that will be
  * passed as eloop_ctx parameter to signal handlers.
  */
-void eloop_init(void *user_data);
+void qt_eloop_init(void *user_data);
 
 /**
- * eloop_register_read_sock - Register handler for read events
+ * qt_eloop_register_read_sock - Register handler for read events
  * @sock: File descriptor number for the socket
  * @handler: Callback function to be called when data is available for reading
  * @eloop_data: Callback context data (eloop_ctx)
@@ -48,22 +48,22 @@ void eloop_init(void *user_data);
  * function will be called whenever data is available for reading from the
  * socket.
  */
-int eloop_register_read_sock(int sock,
+int qt_eloop_register_read_sock(int sock,
 			     void (*handler)(int sock, void *eloop_ctx,
 					     void *sock_ctx),
 			     void *eloop_data, void *user_data);
 
 /**
- * eloop_unregister_read_sock - Unregister handler for read events
+ * qt_eloop_unregister_read_sock - Unregister handler for read events
  * @sock: File descriptor number for the socket
  *
  * Unregister a read socket notifier that was previously registered with
- * eloop_register_read_sock().
+ * qt_eloop_register_read_sock().
  */
-void eloop_unregister_read_sock(int sock);
+void qt_eloop_unregister_read_sock(int sock);
 
 /**
- * eloop_register_timeout - Register timeout
+ * qt_eloop_register_timeout - Register timeout
  * @secs: Number of seconds to the timeout
  * @usecs: Number of microseconds to the timeout
  * @handler: Callback function to be called when timeout occurs
@@ -74,26 +74,26 @@ void eloop_unregister_read_sock(int sock);
  * Register a timeout that will cause the handler function to be called after
  * given time.
  */
-int eloop_register_timeout(unsigned int secs, unsigned int usecs,
+int qt_eloop_register_timeout(unsigned int secs, unsigned int usecs,
 			   void (*handler)(void *eloop_ctx, void *timeout_ctx),
 			   void *eloop_data, void *user_data);
 
 /**
- * eloop_cancel_timeout - Cancel timeouts
+ * qt_eloop_cancel_timeout - Cancel timeouts
  * @handler: Matching callback function
  * @eloop_data: Matching eloop_data or %ELOOP_ALL_CTX to match all
  * @user_data: Matching user_data or %ELOOP_ALL_CTX to match all
  * Returns: Number of cancelled timeouts
  *
  * Cancel matching <handler,eloop_data,user_data> timeouts registered with
- * eloop_register_timeout(). ELOOP_ALL_CTX can be used as a wildcard for
+ * qt_eloop_register_timeout(). ELOOP_ALL_CTX can be used as a wildcard for
  * cancelling all timeouts regardless of eloop_data/user_data.
  */
-int eloop_cancel_timeout(void (*handler)(void *eloop_ctx, void *sock_ctx),
+int qt_eloop_cancel_timeout(void (*handler)(void *eloop_ctx, void *sock_ctx),
 			 void *eloop_data, void *user_data);
 
 /**
- * eloop_register_signal - Register handler for signals
+ * qt_eloop_register_signal - Register handler for signals
  * @sig: Signal number (e.g., SIGHUP)
  * @handler: Callback function to be called when the signal is received
  * @user_data: Callback context data (signal_ctx)
@@ -106,47 +106,46 @@ int eloop_cancel_timeout(void (*handler)(void *eloop_ctx, void *sock_ctx),
  *
  * Signals are 'global' events and there is no local eloop_data pointer like
  * with other handlers. The global user_data pointer registered with
- * eloop_init() will be used as eloop_ctx for signal handlers.
+ * qt_eloop_init() will be used as eloop_ctx for signal handlers.
  */
-int eloop_register_signal(int sig,
+int qt_eloop_register_signal(int sig,
 			  void (*handler)(int sig, void *eloop_ctx,
 					  void *signal_ctx),
 			  void *user_data);
-
 /**
- * eloop_run - Start the event loop
+ * qt_eloop_run - Start the event loop
  *
  * Start the event loop and continue running as long as there are any
  * registered event handlers. This function is run after event loop has been
  * initialized with event_init() and one or more events have been registered.
  */
-void eloop_run(void);
+void qt_eloop_run(void);
 
 /**
- * eloop_terminate - Terminate event loop
+ * qt_eloop_terminate - Terminate event loop
  *
  * Terminate event loop even if there are registered events. This can be used
  * to request the program to be terminated cleanly.
  */
-void eloop_terminate(void);
+void qt_eloop_terminate(void);
 
 /**
- * eloop_destroy - Free any resources allocated for the event loop
+ * qt_eloop_destroy - Free any resources allocated for the event loop
  *
  * After calling eloop_destoy(), other eloop_* functions must not be called
- * before re-running eloop_init().
+ * before re-running qt_eloop_init().
  */
-void eloop_destroy(void);
+void qt_eloop_destroy(void);
 
 /**
- * eloop_terminated - Check whether event loop has been terminated
+ * qt_eloop_terminated - Check whether event loop has been terminated
  * Returns: 1 = event loop terminate, 0 = event loop still running
  *
- * This function can be used to check whether eloop_terminate() has been called
+ * This function can be used to check whether qt_eloop_terminate() has been called
  * to request termination of the event loop. This is normally used to abort
- * operations that may still be queued to be run when eloop_terminate() was
+ * operations that may still be queued to be run when qt_eloop_terminate() was
  * called.
  */
-int eloop_terminated(void);
+int qt_eloop_terminated(void);
 
 #endif /* ELOOP_H */

hs2_profile.h

@@ -180,7 +180,7 @@ struct tlv_to_profile hs2_profile[] = {
 };
 
 struct tlv_to_profile* find_tlv_hs2_profile(int tlv_id) {
-    int i;
+    unsigned int i;
     for (i = 0; i < ARRAY_SIZE(hs2_profile); i++) {
         if (tlv_id == hs2_profile[i].tlv_id) {
             return &hs2_profile[i];
@@ -190,7 +190,7 @@ struct tlv_to_profile* find_tlv_hs2_profile(int tlv_id) {
 }
 
 void attach_hs20_icons(char * buffer) {
-    int i;
+    unsigned int i;
     for (i = 0; i < ARRAY_SIZE(hs20_icon); i++) {
         strcat(buffer, hs20_icon[i]);
     }

indigo_api.c

@@ -277,7 +277,7 @@ struct indigo_tlv indigo_tlv_list[] = {
 
 /* Find the type of the API stucture by the ID from the list */
 char* get_api_type_by_id(int id) {
-    int i = 0;
+    unsigned int i = 0;
     for (i = 0; i < sizeof(indigo_api_list)/sizeof(struct indigo_api); i++) {
         if (id == indigo_api_list[i].type) {
             return indigo_api_list[i].name;
@@ -288,7 +288,7 @@ char* get_api_type_by_id(int id) {
 
 /* Find the API stucture by the ID from the list */
 struct indigo_api* get_api_by_id(int id) {
-    int i = 0;
+    unsigned int i = 0;
     for (i = 0; i < sizeof(indigo_api_list)/sizeof(struct indigo_api); i++) {
         if (id == indigo_api_list[i].type) {
             return &indigo_api_list[i];
@@ -299,7 +299,7 @@ struct indigo_api* get_api_by_id(int id) {
 
 /* Find the TLV by the ID from the list */
 struct indigo_tlv* get_tlv_by_id(int id) {
-    int i = 0;
+    unsigned int i = 0;
 
     for (i = 0; i < sizeof(indigo_tlv_list)/sizeof(struct indigo_tlv); i++) {
         if (id == indigo_tlv_list[i].id) {

indigo_api_callback.h

@@ -174,7 +174,7 @@ struct anqp_tlv_to_config_name anqp_maps[] = {
 };
 
 char* find_tlv_config_name(int tlv_id) {
-    int i;
+    unsigned int i;
     for (i = 0; i < sizeof(maps)/sizeof(struct tlv_to_config_name); i++) {
         if (tlv_id == maps[i].tlv_id) {
             return maps[i].config_name;
@@ -184,7 +184,7 @@ char* find_tlv_config_name(int tlv_id) {
 }
 
 struct tlv_to_config_name* find_tlv_config(int tlv_id) {
-    int i;
+    unsigned int i;
     for (i = 0; i < sizeof(maps)/sizeof(struct tlv_to_config_name); i++) {
         if (tlv_id == maps[i].tlv_id) {
             return &maps[i];
@@ -211,7 +211,7 @@ struct tlv_to_config_name wpas_global_maps[] = {
 };
 
 struct tlv_to_config_name* find_wpas_global_config_name(int tlv_id) {
-    int i;
+    unsigned int i;
     for (i = 0; i < sizeof(wpas_global_maps)/sizeof(struct tlv_to_config_name); i++) {
         if (tlv_id == wpas_global_maps[i].tlv_id) {
             return &wpas_global_maps[i];
@@ -234,8 +234,6 @@ struct tlv_to_config_name* find_generic_tlv_config(int tlv_id, struct tlv_to_con
 static int get_control_app_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int start_loopback_server(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int stop_loop_back_server_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
-static int send_loopback_data_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
-static int stop_loopback_data_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int create_bridge_network_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int assign_static_ip_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int get_mac_addr_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
@@ -253,14 +251,12 @@ static int send_ap_disconnect_handler(struct packet_wrapper *req, struct packet_
 static int set_ap_parameter_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int send_ap_btm_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int trigger_ap_channel_switch(struct packet_wrapper *req, struct packet_wrapper *resp);
-static int send_ap_arp_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int start_wps_ap_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int configure_ap_wsc_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 /* STA */
 static int stop_sta_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int configure_sta_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int associate_sta_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
-static int start_up_sta_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int send_sta_disconnect_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int send_sta_reconnect_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int send_sta_btm_query_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
@@ -270,11 +266,7 @@ static int set_sta_parameter_handler(struct packet_wrapper *req, struct packet_w
 static int set_sta_hs2_associate_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int sta_add_credential_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int set_sta_install_ppsmo_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
-static int set_sta_phy_mode_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
-static int set_sta_channel_width_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
-static int set_sta_power_save_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int start_wps_sta_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
-static int send_sta_icon_req_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 static int enable_wsc_sta_handler(struct packet_wrapper *req, struct packet_wrapper *resp);
 /* P2P */
 static int start_up_p2p_handler(struct packet_wrapper *req, struct packet_wrapper *resp);