[hipl-dev] [Branch ~hipl-core/hipl/trunk] Rev 5367: Simplify a series of connection structure assignments.

  • From: noreply@xxxxxxxxxxxxx
  • To: HIPL core team <hipl-dev@xxxxxxxxxxxxx>
  • Date: Tue, 04 Jan 2011 16:25:50 -0000

------------------------------------------------------------
revno: 5367
author: Stefan Götz <stefan.goetz@xxxxxxxxxxxxxxxxx>
committer: Diego Biurrun <diego@xxxxxxxxxx>
branch nick: hipl
timestamp: Tue 2011-01-04 17:14:19 +0100
message:
  Simplify a series of connection structure assignments.
  
  This will make a future uncrustify run simpler as it was choking on this code.
modified:
  firewall/conntrack.c


--
lp:hipl
https://code.launchpad.net/~hipl-core/hipl/trunk

Your team HIPL core team is subscribed to branch lp:hipl.
To unsubscribe from this branch go to 
https://code.launchpad.net/~hipl-core/hipl/trunk/+edit-subscription
=== modified file 'firewall/conntrack.c'
--- firewall/conntrack.c        2011-01-04 14:10:46 +0000
+++ firewall/conntrack.c        2011-01-04 16:14:19 +0000
@@ -444,34 +444,26 @@
 #endif
 
     //original direction tuple
-    connection->original.state                   = HIP_STATE_UNASSOCIATED;
-    connection->original.direction               = ORIGINAL_DIR;
-    connection->original.esp_tuples              = NULL;
-    connection->original.connection              = connection;
-    connection->original.hip_tuple               = malloc(sizeof(struct 
hip_tuple));
-    memset(connection->original.hip_tuple, 0, sizeof(struct hip_tuple));
-    connection->original.hip_tuple->tuple        = &connection->original;
-    connection->original.hip_tuple->data         = malloc(sizeof(struct 
hip_data));
-    memset(connection->original.hip_tuple->data, 0, sizeof(struct hip_data));
-    memcpy(&connection->original.hip_tuple->data->src_hit, &data->src_hit, 
sizeof(struct in6_addr));
-    memcpy(&connection->original.hip_tuple->data->dst_hit, &data->dst_hit, 
sizeof(struct in6_addr));
-    connection->original.hip_tuple->data->src_hi = NULL;
-    connection->original.hip_tuple->data->verify = NULL;
+    connection->original.state                    = HIP_STATE_UNASSOCIATED;
+    connection->original.direction                = ORIGINAL_DIR;
+    connection->original.esp_tuples               = NULL;
+    connection->original.connection               = connection;
+    connection->original.hip_tuple                = malloc(sizeof(struct 
hip_tuple));
+    connection->original.hip_tuple->tuple         = &connection->original;
+    connection->original.hip_tuple->data          = calloc(1, sizeof(struct 
hip_data));
+    connection->original.hip_tuple->data->src_hit = data->src_hit;
+    connection->original.hip_tuple->data->dst_hit = data->dst_hit;
 
     //reply direction tuple
-    connection->reply.state                      = HIP_STATE_UNASSOCIATED;
-    connection->reply.direction                  = REPLY_DIR;
-    connection->reply.esp_tuples                 = NULL;
-    connection->reply.connection                 = connection;
-    connection->reply.hip_tuple                  = malloc(sizeof(struct 
hip_tuple));
-    memset(connection->reply.hip_tuple, 0, sizeof(struct hip_tuple));
-    connection->reply.hip_tuple->tuple           = &connection->reply;
-    connection->reply.hip_tuple->data            = malloc(sizeof(struct 
hip_data));
-    memset(connection->reply.hip_tuple->data, 0, sizeof(struct hip_data));
-    memcpy(&connection->reply.hip_tuple->data->src_hit, &data->dst_hit, 
sizeof(struct in6_addr));
-    memcpy(&connection->reply.hip_tuple->data->dst_hit, &data->src_hit, 
sizeof(struct in6_addr));
-    connection->reply.hip_tuple->data->src_hi    = NULL;
-    connection->reply.hip_tuple->data->verify    = NULL;
+    connection->reply.state                    = HIP_STATE_UNASSOCIATED;
+    connection->reply.direction                = REPLY_DIR;
+    connection->reply.esp_tuples               = NULL;
+    connection->reply.connection               = connection;
+    connection->reply.hip_tuple                = malloc(sizeof(struct 
hip_tuple));
+    connection->reply.hip_tuple->tuple         = &connection->reply;
+    connection->reply.hip_tuple->data          = calloc(1, sizeof(struct 
hip_data));
+    connection->reply.hip_tuple->data->src_hit = data->dst_hit;
+    connection->reply.hip_tuple->data->dst_hit = data->src_hit;
 
     //add tuples to list
     hip_list = append_to_list(hip_list, connection->original.hip_tuple);

Other related posts:

  • » [hipl-dev] [Branch ~hipl-core/hipl/trunk] Rev 5367: Simplify a series of connection structure assignments. - noreply