Cleaned up and added some missing comments

2 files changed, 23 insertions, 7 deletions

diff --git a/plugin/firewall/eurephiafw_intf.h b/plugin/firewall/eurephiafw_intf.h
index c4fc065..88719e9 100644
--- a/plugin/firewall/eurephiafw_intf.h
+++ b/plugin/firewall/eurephiafw_intf.h
@@ -32,10 +32,26 @@
 #ifndef   	EUREPHIAFW_INTF_H_
 #define   	EUREPHIAFW_INTF_H_
 
-// Mandatory functions for all FW interfaces
-const char *(*eFWinterfaceVersion) (void);
-int (*eFWinterfaceAPIversion) (void);
+/**
+ * Mandatory function, contains driver information.
+ *
+ * @return Should retun a static string, containing the version information.
+ */
+const char *(*eFWinterfaceVersion) ();
+
+/**
+ * Mandatory function, contains driver information.
+ *
+ * @return Should retun an integer which correponds to the API level the interface driver uses.
+ */
+int (*eFWinterfaceAPIversion) ();
 
-void (*eFW_RunFirewall) (void *);
+/**
+ * The main routine of the firewall interface.  This function should not return before
+ * the firewall interface is supposed to shut down.
+ *
+ * @param fwargs efw_threaddata pointer, with needed information to communicate with the openvpn process.
+ */
+void (*eFW_RunFirewall) (void *fwargs);
 
 #endif 	    /* !EUREPHIAFW_INTF_H_ */
diff --git a/plugin/firewall/iptables/efw-iptables.c b/plugin/firewall/iptables/efw-iptables.c
index b686ca9..1329973 100644
--- a/plugin/firewall/iptables/efw-iptables.c
+++ b/plugin/firewall/iptables/efw-iptables.c
@@ -37,15 +37,15 @@
 #include <pthread.h>
 #include <sys/wait.h>
 
-#define EUREPHIA_FWINTF
+#define EUREPHIA_FWINTF         /**< Include the proper eurephiaFWINTF declaration in eurephiaCTX */
 #include <eurephiafw_struct.h>
 #include <eurephia_context.h>
 #include <eurephia_nullsafe.h>
 #include <eurephia_log.h>
 #include <eurephiafw_helpers.h>
 
-#define INTERFACEVER "1.0"
-#define INTERFACEAPIVER 1
+#define INTERFACEVER "1.0"      /**< The version of this firewall interface (driver) */
+#define INTERFACEAPIVER 1       /**< Define the API level this firewall interface uses. */
 
 /**
  * Mandatory function, contains driver information.