cleanup, new protocol, renaming of some functions, GUI has to update!

12 files changed, 164 insertions, 152 deletions

diff --git a/noncore/net/wellenreiter/libwellenreiter/source/Makefile b/noncore/net/wellenreiter/libwellenreiter/source/Makefile
index 3670d96..51b5ff3 100644
--- a/noncore/net/wellenreiter/libwellenreiter/source/Makefile
+++ b/noncore/net/wellenreiter/libwellenreiter/source/Makefile
@@ -1,17 +1,17 @@
 # $Id$
 
-INCLUDES = 
-LIBRARIES =
-LIBOBJ = proto.o sock.o log.o cardmode.o sniff.o
-CXX = g++ -Wall -pedantic -g  $(INCLUDES) -DDEBUG
+OBJ = wl_proto.o wl_sock.o wl_log.o cardmode.o sniff.o
+CPP = g++
+CPPFLAGS = -Wall -pedantic -g -DDEBUG
 
 static:	libwellenreiter.a
-libwellenreiter.a: $(LIBOBJ)
-	ar -cr libwellenreiter.a $(LIBOBJ)
+libwellenreiter.a: $(OBJ)
+	ar -cr libwellenreiter.a $(OBJ)
 
 shared:	libwellenreiter.so
-libwellenreiter.so: $(LIBOBJ)
-	$(CXX) -shared -o libwellenreiter.so $(LIBOBJ)
+libwellenreiter.so: $(OBJ)
+	$(CPP) $(CPPFLAGS) -shared -o libwellenreiter.so $(OBJ)
 
 clean:
 	rm -f *.o *.a *.so *~
+
diff --git a/noncore/net/wellenreiter/libwellenreiter/source/cardmode.cc b/noncore/net/wellenreiter/libwellenreiter/source/cardmode.cc
index 3aaa4a7..c29f4d0 100644
--- a/noncore/net/wellenreiter/libwellenreiter/source/cardmode.cc
+++ b/noncore/net/wellenreiter/libwellenreiter/source/cardmode.cc
@@ -6,5 +6,5 @@
 
 #include "cardmode.hh"
-#include "log.hh"
+#include "wl_log.hh"
 
 /* main card into monitor function */
@@ -13,5 +13,4 @@ int card_into_monitormode (pcap_t **orighandle, char *device, int cardtype)
   char CiscoRFMON[35] = "/proc/driver/aironet/";
   FILE *CISCO_CONFIG_FILE;
-  pcap_t *handle = (pcap_t*)orighandle;
 
   /* Checks if we have a device to sniff on */
@@ -34,5 +33,5 @@ int card_into_monitormode (pcap_t **orighandle, char *device, int cardtype)
   {
       /* bring the sniffer into rfmon  mode */
-      snprintf(CiscoRFMON, sizeof(CiscoRFMON), DEFAULT_PATH, device);
+      snprintf(CiscoRFMON, sizeof(CiscoRFMON) - 1, DEFAULT_PATH, device);
       if((CISCO_CONFIG_FILE = fopen(CiscoRFMON,"w")) == NULL)
       {
@@ -48,5 +47,5 @@ int card_into_monitormode (pcap_t **orighandle, char *device, int cardtype)
   {
       char wlanngcmd[62];
-      snprintf(wlanngcmd, sizeof(wlanngcmd), "%s %s lnxreq_wlansniff channel=1 enable=true", WLANCTL_PATH, device);
+      snprintf(wlanngcmd, sizeof(wlanngcmd) - 1, "%s %s lnxreq_wlansniff channel=1 enable=true", WLANCTL_PATH, device);
       if (system(wlanngcmd) != 0)
       {
@@ -69,5 +68,5 @@ int card_set_promisc_up (const char *device)
   int retval=0;
 
-  snprintf(ifconfigcmd, sizeof(ifconfigcmd), SBIN_PATH, device);
+  snprintf(ifconfigcmd, sizeof(ifconfigcmd) - 1, SBIN_PATH, device);
   retval = system(ifconfigcmd);
 
diff --git a/noncore/net/wellenreiter/libwellenreiter/source/cardmode.hh b/noncore/net/wellenreiter/libwellenreiter/source/cardmode.hh
index b85b7b7..6ec5b95 100644
--- a/noncore/net/wellenreiter/libwellenreiter/source/cardmode.hh
+++ b/noncore/net/wellenreiter/libwellenreiter/source/cardmode.hh
@@ -11,10 +11,6 @@
 #include <sys/socket.h>
 #include <netinet/in.h>
-
-extern "C"
-{
 #include <net/bpf.h>
 #include <pcap.h>  
-}
 
 /* Defines, used for the card setup */
diff --git a/noncore/net/wellenreiter/libwellenreiter/source/proto.cc b/noncore/net/wellenreiter/libwellenreiter/source/proto.cc
deleted file mode 100644
index a1228e4..0000000
--- a/noncore/net/wellenreiter/libwellenreiter/source/proto.cc
+++ b/dev/null
@@ -1,109 +0,0 @@
-/*
- * Communication protocol
- *
- * $Id$
- */
-
-#include "proto.hh"
-#include "sock.hh"
-#include "log.hh"
-
-/* Send found network to GUI */
-int send_network_found (const char *guihost, int guiport, void *structure)
-{
-  wl_network_t *ptr;
-  char buffer[2048];
-  char temp[5];
-
-  ptr = (wl_network_t *)structure;
-
-  memset(buffer, 0, 2048);
-
-  /* Type = Found new net */
-  memcpy(buffer, "01", 2);
-
-  /* Set Net-type */
-  memset(temp, 0, sizeof(temp));
-  snprintf(temp, 2, "%d", ptr->net_type);
-  memcpy(buffer + 2, temp, 1);
-
-  /* Set channel */
-  memset(temp, 0, sizeof(temp));
-
-  if(ptr->channel < 10)
-    snprintf(temp, 3, "0%d", ptr->channel);
-  else
-    snprintf(temp, 3, "%d", ptr->channel);
-
-  memcpy(buffer + 3, temp, 2);
-
-  /* Set WEP y/n */
-  memset(temp, 0, sizeof(temp));
-  snprintf(temp, 2, "%d", ptr->wep);
-  memcpy(buffer + 5, temp, 1);
-
-  /* Set MAC address */
-  memcpy(buffer + 6, ptr->mac, 17);
-
-  /* Set lenght of ssid */
-  memset(temp, 0, sizeof(temp));
-
-  if(ptr->ssid_len > 99)
-    snprintf(temp, 4, "%d", ptr->ssid_len);
-  else if(ptr->ssid_len < 10)
-    snprintf(temp, 4, "00%d", ptr->ssid_len);
-  else
-    snprintf(temp, 4, "0%d", ptr->ssid_len);
-
-  memcpy(buffer + 23, temp, 3);
-
-  /* Set ssid */
-  memcpy(buffer + 26, ptr->bssid, ptr->ssid_len);
-
-  /* Send prepared buffer to GUI */
-#ifdef DEBUG
-  wl_loginfo("Sending network to GUI: '%s'", buffer);
-#endif
-
-  sendcomm(guihost, guiport, buffer);
-
-  return 1;
-}
-
-/* Fill buffer into structur */
-int get_network_found (void *structure, const char *buffer)
-{
-  wl_network_t *ptr;
-  char temp[512];
-
-  ptr = (wl_network_t *)structure;
-
-  /* Get net type */
-  memset(temp, 0, sizeof(temp));
-  memcpy(temp, buffer + 2, 1);
-  ptr->net_type = atoi(temp);
-
-  /* Get channel */
-  memset(temp, 0, sizeof(temp));
-  memcpy(temp, buffer + 3, 2);
-  ptr->channel = atoi(temp);
-
-  /* Set WEP y/n */
-  memset(temp, 0, sizeof(temp));
-  memcpy(temp, buffer + 5, 1);
-  ptr->wep = atoi(temp);
-
-  /* Set MAC address */
-  memcpy(ptr->mac, buffer + 6, 17);
-  ptr->mac[17]='\0';
-
-  /* Set lenght of ssid */
-  memset(temp, 0, sizeof(temp));
-  memcpy(temp, buffer + 23, 3);
-  ptr->ssid_len = atoi(temp);
-
-  /* Set ssid */
-  memcpy(ptr->bssid, buffer + 26, ptr->ssid_len + 1);
-
-  return 1;
-}
diff --git a/noncore/net/wellenreiter/libwellenreiter/source/sniff.cc b/noncore/net/wellenreiter/libwellenreiter/source/sniff.cc
index c703052..e2169ca 100644
--- a/noncore/net/wellenreiter/libwellenreiter/source/sniff.cc
+++ b/noncore/net/wellenreiter/libwellenreiter/source/sniff.cc
@@ -8,6 +8,6 @@
 #include "ieee802_11.hh"
 #include "extract.hh"
-#include "log.hh"
-#include "proto.hh"
+#include "wl_log.hh"
+#include "wl_proto.hh"
 
 /* Main function, checks packets */
diff --git a/noncore/net/wellenreiter/libwellenreiter/source/sniff.hh b/noncore/net/wellenreiter/libwellenreiter/source/sniff.hh
index c7108ac..713b383 100644
--- a/noncore/net/wellenreiter/libwellenreiter/source/sniff.hh
+++ b/noncore/net/wellenreiter/libwellenreiter/source/sniff.hh
@@ -13,10 +13,6 @@
 #include <netinet/in.h>
 #include <arpa/inet.h>
-
-extern "C"
-{
 #include <net/bpf.h>
 #include <pcap.h>  
-}
 
 #define   NONBROADCASTING "non-broadcasting"
diff --git a/noncore/net/wellenreiter/libwellenreiter/source/log.cc b/noncore/net/wellenreiter/libwellenreiter/source/wl_log.cc
index af7f909..7adaba8 100644
--- a/noncore/net/wellenreiter/libwellenreiter/source/log.cc
+++ b/noncore/net/wellenreiter/libwellenreiter/source/wl_log.cc
@@ -5,5 +5,5 @@
  */
 
-#include "log.hh"
+#include "wl_log.hh"
 
 /* Log to syslog INFO */
diff --git a/noncore/net/wellenreiter/libwellenreiter/source/log.hh b/noncore/net/wellenreiter/libwellenreiter/source/wl_log.hh
index 8f6e543..8f6e543 100644
--- a/noncore/net/wellenreiter/libwellenreiter/source/log.hh
+++ b/noncore/net/wellenreiter/libwellenreiter/source/wl_log.hh
diff --git a/noncore/net/wellenreiter/libwellenreiter/source/wl_proto.cc b/noncore/net/wellenreiter/libwellenreiter/source/wl_proto.cc
new file mode 100644
index 0000000..f15523f
--- a/dev/null
+++ b/noncore/net/wellenreiter/libwellenreiter/source/wl_proto.cc
@@ -0,0 +1,132 @@
+/*
+ * Communication protocol
+ *
+ * $Id$
+ */
+
+#include "wl_proto.hh"
+#include "wl_log.hh"
+#include "wl_sock.hh"
+
+/* Adds a field to the buffer */
+int add_field(char *buffer, char *string, int len)
+{
+  char newlen[5];
+  
+  /* 3 Byte = Length */
+  snprintf(newlen, sizeof(newlen) - 1, "%.3d", len);
+  memcpy(buffer, newlen, 3);
+  
+  /* Length bytes = Value */
+  memcpy(buffer + 3, string, atoi(newlen));
+  
+  /* Return length of attached field */
+  return (atoi(newlen) + 3);
+}
+
+int get_field(const char *buffer, char *out)
+{
+  char len[5];
+
+  /* Get length of value */
+  memcpy(len, buffer, 3);
+
+  /* Copy buffer to out pointer */
+  memset(out, 0, atoi(len) + 1);
+  memcpy(out, buffer + 3, atoi(len));
+
+  return atoi(len) + 3;
+}
+
+/* Send found network to GUI */
+int send_network_found (const char *guihost, int guiport, void *structure)
+{
+  wl_network_t *ptr;
+  char buffer[2048];
+  char temp[128];
+  int retval=0, len=0;
+
+  memset(temp, 0, sizeof(temp));
+
+  ptr = (wl_network_t *)structure;
+
+  /* Type = Found new net (without length field) */
+  memset(temp, 0, sizeof(temp));
+  snprintf(temp, sizeof(temp), "%.2d", NETFOUND);
+  memcpy(buffer, temp, 2);
+  len += 2;
+
+  /* Set Net-type */
+  memset(temp, 0, sizeof(temp));
+  snprintf(temp, sizeof(temp), "%d", ptr->net_type);
+  retval = add_field(buffer + len, temp, 1);
+  len += retval;
+
+  /* Set channel */
+  memset(temp, 0, sizeof(temp));
+  snprintf(temp, sizeof(temp), "%.2d", ptr->channel);
+  retval = add_field(buffer + len, temp, 2); 
+  len += retval;
+
+  /* Set WEP y/n */
+  memset(temp, 0, sizeof(temp));
+  snprintf(temp, sizeof(temp), "%d", ptr->wep);
+  retval = add_field(buffer + len, temp, 1);
+  len += retval;
+
+  /* Set Mac */
+  retval = add_field(buffer + len, (char *)ptr->mac, 17);
+  len += retval;
+
+  /* Set ssid */
+  retval = add_field(buffer + len, (char *)ptr->bssid, ptr->ssid_len);
+
+  /* Send prepared buffer to GUI */
+#ifdef DEBUG
+  wl_loginfo("Sending network to GUI: '%s'", buffer);
+#endif
+
+  wl_send(guihost, guiport, buffer);
+
+  return 1;
+}
+
+/* Fill buffer into structur */
+int get_network_found (void *structure, const char *buffer)
+{
+  wl_network_t *ptr;
+  char temp[512];
+  int retval=0, len=0;
+
+  ptr = (wl_network_t *)structure;
+  
+  /* packet type already determined */
+  len += 2;
+
+  /* Get net type */
+  memset(temp, 0, sizeof(temp));
+  retval = get_field(buffer + len, temp);
+  len += retval;
+  ptr->net_type = atoi(temp); 
+   
+  /* Get channel */
+  memset(temp, 0, sizeof(temp));
+  retval = get_field(buffer + len, temp);
+  len += retval;
+  ptr->channel = atoi(temp);
+
+  /* Set WEP y/n */
+  memset(temp, 0, sizeof(temp));
+  retval = get_field(buffer + len, temp);
+  len += retval;
+  ptr->wep = atoi(temp);
+
+  /* Set MAC address */
+  retval = get_field(buffer + len, ptr->mac);
+  len += retval;
+
+  /* Set BSSID */
+  retval = get_field(buffer + len, ptr->bssid);
+
+  return 1;
+}
diff --git a/noncore/net/wellenreiter/libwellenreiter/source/proto.hh b/noncore/net/wellenreiter/libwellenreiter/source/wl_proto.hh
index e3e9f50..f755589 100644
--- a/noncore/net/wellenreiter/libwellenreiter/source/proto.hh
+++ b/noncore/net/wellenreiter/libwellenreiter/source/wl_proto.hh
@@ -1,6 +1,6 @@
 /* $Id$ */
 
-#ifndef PROTO_HH
-#define PROTO_HH
+#ifndef WLPROTO_HH
+#define WLPROTO_HH
 
 #include <stdio.h>
@@ -26,3 +26,3 @@ typedef struct {
 } wl_network_t;
 
-#endif /* PROTO_HH */
+#endif /* WLPROTO_HH */
diff --git a/noncore/net/wellenreiter/libwellenreiter/source/sock.cc b/noncore/net/wellenreiter/libwellenreiter/source/wl_sock.cc
index 90fc20a..5714afb 100644
--- a/noncore/net/wellenreiter/libwellenreiter/source/sock.cc
+++ b/noncore/net/wellenreiter/libwellenreiter/source/wl_sock.cc
@@ -5,9 +5,9 @@
  */
 
-#include "sock.hh"
-#include "log.hh"
+#include "wl_sock.hh"
+#include "wl_log.hh"
 
 /* Setup UDP Socket for incoming commands */
-int commsock(const char *host, int port)
+int wl_setupsock(const char *host, int port)
 {
   struct sockaddr_in saddr;
@@ -36,5 +36,5 @@ int commsock(const char *host, int port)
 
 /* Send a string to commsock */
-int sendcomm(const char *host, int port, const char *string, ...) 
+int wl_send(const char *host, int port, const char *string, ...) 
 {
   int sock;
@@ -64,5 +64,5 @@ int sendcomm(const char *host, int port, const char *string, ...)
     wl_logerr("Cannot write to socket: %s", strerror(errno));
     close(sock);
-    return 0;
+    return -1;
   }
 
@@ -70,21 +70,19 @@ int sendcomm(const char *host, int port, const char *string, ...)
     wl_logerr("Cannot close socket: %s", strerror(errno));
 
-  return 1;
+  return 0;
 }
 
 /* Check for new messages on commsock */
-int recvcomm(int *sock, char *out, int maxlen)
+int wl_recv(int *sock, char *out, int maxlen)
 {
   struct sockaddr_in *cliaddr;
   socklen_t len=sizeof(struct sockaddr);
-  char buffer[128], retval[3];
-
-  memset(buffer, 0, sizeof(buffer));
-  if(recvfrom(*sock, buffer, sizeof(buffer)-1, 0, (struct sockaddr *)cliaddr, &len) < 0)
-    return -1;
+  char retval[3];
 
   memset(out, 0, maxlen);
-  memcpy(out, buffer, maxlen - 1);
+  if(recvfrom(*sock, out, maxlen - 1, 0, (struct sockaddr *)cliaddr, &len) < 0)
+    return -1;
 
+  /* Get packet type and return it */
   memset(retval, 0, sizeof(retval));
   memcpy(retval, out, 2);
diff --git a/noncore/net/wellenreiter/libwellenreiter/source/sock.hh b/noncore/net/wellenreiter/libwellenreiter/source/wl_sock.hh
index 611e335..6ddbaef 100644
--- a/noncore/net/wellenreiter/libwellenreiter/source/sock.hh
+++ b/noncore/net/wellenreiter/libwellenreiter/source/wl_sock.hh
@@ -14,7 +14,7 @@
 #include <errno.h>
 
-int commsock(const char *, int);
-int sendcomm(const char *, int, const char *, ...);
-int recvcomm(int *, char *, int);
+int wl_setupsock(const char *, int);
+int wl_send(const char *, int, const char *, ...);
+int wl_recv(int *, char *, int);
 
 #endif /* WLSOCK_HH */