Remove unused functions, memory handler
authorNeutron Soutmun <neo.neutron@gmail.com>
Mon, 1 Sep 2008 10:53:20 +0000 (17:53 +0700)
committerNeutron Soutmun <neo.neutron@gmail.com>
Sun, 2 Nov 2008 19:43:11 +0000 (02:43 +0700)
2008-09-01  Neutron Soutmun <neo.neutron@gmail.com>

* src/rahunasd.c: Remove unnecessary functions.
* src/rh-xmlrpc-server.c: Free the allocated memory of the the vars that
  calling rh_string_get_sep().

ChangeLog
src/rahunasd.c
src/rh-xmlrpc-server.c

index c1aede5..903b569 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2008-09-01  Neutron Soutmun <neo.neutron@gmail.com>
+
+       * src/rahunasd.c: Remove unnecessary functions.
+       * src/rh-xmlrpc-server.c: Free the allocated memory of the the vars that 
+         calling rh_string_get_sep().
+
 2008-08-31  Neutron Soutmun <neo.neutron@gmail.com>
 
        * +src/rh-utils.{h,c}: Add new utility functions.
index 211e1ba..f5d698a 100644 (file)
@@ -32,9 +32,6 @@ int ipset_flush();
 struct rahunas_map* rh_init_map();
 int rh_init_members(struct rahunas_map *map);
 
-int parse_set_header(const char *in, struct rahunas_map *map);
-int parse_set_list(const char *in, struct rahunas_map *map);
-
 int send_xmlrpc_stopacct(struct rahunas_map *map, uint32_t id);
 
 /* Declaration */
@@ -230,91 +227,6 @@ int getline(int fd, char *buf, size_t size)
        return (current - buf);
 }
 
-
-int parse_set_header(const char *in, struct rahunas_map *map)
-{
-       in_addr_t first_ip;
-       in_addr_t last_ip;
-       char *ip_from_start = NULL;
-  char *ip_from_end = NULL;
-       char *ip_to_start = NULL;
-  char *ip_to_end = NULL;
-       char *ip_from = NULL;
-  char *ip_to = NULL;
-
-       if (!map || !in)
-         return (-1);
-
-  ip_from_start = strstr(in, "from:");
-       if (ip_from_start == NULL)
-         return (-1);
-
-       ip_to_start = strstr(in, "to:");
-       if (ip_to_start == NULL)
-         return (-1);
-
-  ip_from_start += 6;
-       ip_from_end = ip_to_start - 1;
-       ip_to_start += 4;
-       ip_to_end = in + strlen(in);
-
-       ip_from = strndup(ip_from_start, ip_from_end - ip_from_start);
-       ip_to   = strndup(ip_to_start, ip_to_end - ip_to_start);
-       logmsg(RH_LOG_NORMAL, "First IP: %s", ip_from);
-       logmsg(RH_LOG_NORMAL, "Last IP: %s", ip_to);
-
-       first_ip = inet_addr(ip_from);
-  memcpy(&map->first_ip, &first_ip, sizeof(in_addr_t));
-
-       last_ip = inet_addr(ip_to);
-  memcpy(&map->last_ip, &last_ip, sizeof(in_addr_t));
-
-       map->size = ntohl(map->last_ip) - ntohl(map->first_ip) + 1;
-
-       logmsg(RH_LOG_NORMAL, "Set Size: %lu", map->size);
-       free(ip_from);
-       free(ip_to);
-
-       return 0;
-}
-
-int parse_set_list(const char *in, struct rahunas_map *map)
-{
-       char *sep = NULL;
-  char *sess_ip = NULL;
-       char *sess_idle_time = NULL;
-       uint32_t  id;
-
-       struct rahunas_member *members;
-
-       if (!map || !in)
-         return (-1);
-
-  if (!map->members)
-    return (-1);
-       
-       members = map->members;
-
-  // Found members
-  DP("Parse from list: %s", in);
-       sep = strstr(in, ":");
-       sess_ip = strndup(in, sep - in);
-  if (sess_ip == NULL)
-    return (-1);
-       id = iptoid(map, sess_ip);
-       sess_idle_time = strndup(sep + 1, strstr(in, "seconds") - sep - 1);
-  if (sess_idle_time == NULL)
-    return (-1);
-
-  members[id].flags = 1;
-  members[id].expired = atoi(sess_idle_time) < IDLE_THRESHOLD ? 0 : 1;
-
-  free(sess_ip);
-  free(sess_idle_time);
-       return id;
-}
-
 int send_xmlrpc_stopacct(struct rahunas_map *map, uint32_t id) {
   struct rahunas_member *members = NULL;
   GNetXmlRpcClient *client = NULL;
index 5af88c5..30e1fb3 100644 (file)
@@ -54,7 +54,7 @@ int do_startsession(GNetXmlRpcServer *server,
 
        if (id < 0) {
     *reply_string = g_strdup("Invalid IP Address");
-               return 0;
+               goto cleanup;
        }
 
   res = set_adtip(rahunas_set, ip, mac_address, IP_SET_OP_ADD_IP);
@@ -93,13 +93,18 @@ int do_startsession(GNetXmlRpcServer *server,
   *reply_string = g_strdup_printf("Greeting! Got: IP %s, User %s, ID %s", 
                                         ip, members[id].username, 
                                                                                                                                         members[id].session_id);
-  g_free(ip);
-       return 0;
+  goto cleanup;
 
 out:
     *reply_string = g_strdup("Invalid input parameters");
-    g_free(ip);
-               return 0;
+    goto cleanup;
+
+cleanup:
+  g_free(ip);
+  g_free(username);
+  g_free(session_id);
+  g_free(mac_address);
+  return 0;
 }
 
 int do_stopsession(GNetXmlRpcServer *server,
@@ -139,7 +144,7 @@ int do_stopsession(GNetXmlRpcServer *server,
 
        if (id < 0) {
     *reply_string = g_strdup("Invalid IP Address");
-               return 0;
+    goto cleanup;
        }
 
        if (members[id].flags) {
@@ -165,21 +170,25 @@ int do_stopsession(GNetXmlRpcServer *server,
  
         *reply_string = g_strdup_printf("Client IP %s was removed!", 
                                                          idtoip(map, id));
-           return 0;
       } else {
          *reply_string = g_strdup_printf("Client IP %s error remove!", 
                                                        idtoip(map, id));
-                         return 0;
       }
+      goto cleanup;
                }       
   }
 
   *reply_string = g_strdup_printf("%s", ip);
-       return 0;
+  goto cleanup;
 
 out:
-    *reply_string = g_strdup("Invalid input parameters");
-               return 0;
+  *reply_string = g_strdup("Invalid input parameters");
+  goto cleanup;
+
+cleanup:
+  g_free(ip);
+  g_free(mac_address);
+  return 0;
 }
 
 int do_getsessioninfo(GNetXmlRpcServer *server,
@@ -209,7 +218,7 @@ int do_getsessioninfo(GNetXmlRpcServer *server,
 
        if (id < 0) {
     *reply_string = g_strdup("Invalid IP Address");
-               return 0;
+    return 0;
        }
 
        if (members[id].flags) {