From: Jaeyun <jy1210.jung@samsung.com>
Date: Mon, 5 Dec 2022 10:40:11 +0000 (+0900)
Subject: [Handle] macro to check handle
X-Git-Tag: accepted/tizen/unified/20221226.070011~5
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3a3d4d06117b2ac8c37921c740ee2f174a1e3a7d;p=platform%2Fupstream%2Fnnstreamer-edge.git

[Handle] macro to check handle

Macro to check handle in nns-edge.
The magic key would be changed when a handle is destroyed. If handle is not valid, its mutex lock is also invalid.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>
---

diff --git a/src/libnnstreamer-edge/nnstreamer-edge-data.c b/src/libnnstreamer-edge/nnstreamer-edge-data.c
index 5fc7df2..848f4c9 100644
--- a/src/libnnstreamer-edge/nnstreamer-edge-data.c
+++ b/src/libnnstreamer-edge/nnstreamer-edge-data.c
@@ -34,7 +34,7 @@ nns_edge_data_create (nns_edge_data_h * data_h)
   }
 
   nns_edge_lock_init (ed);
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (ed, NNS_EDGE_MAGIC);
   nns_edge_metadata_create (&ed->metadata);
 
   *data_h = ed;
@@ -56,15 +56,13 @@ nns_edge_data_destroy (nns_edge_data_h data_h)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, given edge data is invalid.");
-    nns_edge_unlock (ed);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_lock (ed);
+  nns_edge_handle_set_magic (ed, NNS_EDGE_MAGIC_DEAD);
 
   for (i = 0; i < ed->num; i++) {
     if (ed->data[i].destroy_cb)
@@ -107,15 +105,11 @@ nns_edge_data_is_valid (nns_edge_data_h data_h)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, edge data handle is invalid.");
-    nns_edge_unlock (ed);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_unlock (ed);
   return NNS_EDGE_ERROR_NONE;
 }
 
@@ -141,14 +135,13 @@ nns_edge_data_copy (nns_edge_data_h data_h, nns_edge_data_h * new_data_h)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, edge data handle is invalid.");
-    nns_edge_unlock (ed);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (ed);
+
   ret = nns_edge_data_create (new_data_h);
   if (ret != NNS_EDGE_ERROR_NONE) {
     nns_edge_loge ("Failed to create new data handle.");
@@ -206,14 +199,13 @@ nns_edge_data_add (nns_edge_data_h data_h, void *data, nns_size_t data_len,
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, given edge data is invalid.");
-    nns_edge_unlock (ed);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (ed);
+
   if (ed->num >= NNS_EDGE_DATA_LIMIT) {
     nns_edge_loge ("Cannot add data, the maximum number of edge data is %d.",
         NNS_EDGE_DATA_LIMIT);
@@ -250,14 +242,13 @@ nns_edge_data_get (nns_edge_data_h data_h, unsigned int index, void **data,
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, given edge data is invalid.");
-    nns_edge_unlock (ed);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (ed);
+
   if (index >= ed->num) {
     nns_edge_loge
         ("Invalid param, the number of edge data is %u but requested %uth data.",
@@ -292,17 +283,15 @@ nns_edge_data_get_count (nns_edge_data_h data_h, unsigned int *count)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, given edge data is invalid.");
-    nns_edge_unlock (ed);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (ed);
   *count = ed->num;
-
   nns_edge_unlock (ed);
+
   return NNS_EDGE_ERROR_NONE;
 }
 
@@ -327,17 +316,15 @@ nns_edge_data_set_info (nns_edge_data_h data_h, const char *key,
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, given edge data is invalid.");
-    nns_edge_unlock (ed);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (ed);
   ret = nns_edge_metadata_set (ed->metadata, key, value);
-
   nns_edge_unlock (ed);
+
   return ret;
 }
 
@@ -366,17 +353,15 @@ nns_edge_data_get_info (nns_edge_data_h data_h, const char *key, char **value)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, given edge data is invalid.");
-    nns_edge_unlock (ed);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (ed);
   ret = nns_edge_metadata_get (ed->metadata, key, value);
-
   nns_edge_unlock (ed);
+
   return ret;
 }
 
@@ -396,17 +381,15 @@ nns_edge_data_serialize_meta (nns_edge_data_h data_h, void **data,
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, given edge data is invalid.");
-    nns_edge_unlock (ed);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (ed);
   ret = nns_edge_metadata_serialize (ed->metadata, data, data_len);
-
   nns_edge_unlock (ed);
+
   return ret;
 }
 
@@ -426,17 +409,15 @@ nns_edge_data_deserialize_meta (nns_edge_data_h data_h, const void *data,
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, given edge data is invalid.");
-    nns_edge_unlock (ed);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (ed);
   ret = nns_edge_metadata_deserialize (ed->metadata, data, data_len);
-
   nns_edge_unlock (ed);
+
   return ret;
 }
 
@@ -460,14 +441,12 @@ nns_edge_data_serialize (nns_edge_data_h data_h, void **data, nns_size_t * len)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, given edge data is invalid.");
-    nns_edge_unlock (ed);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (ed);
   header_len = sizeof (nns_edge_data_header_s);
 
   data_len = 0;
@@ -533,14 +512,12 @@ nns_edge_data_deserialize (nns_edge_data_h data_h, const void *data,
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (ed);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (ed)) {
+  if (!nns_edge_handle_is_valid (ed)) {
     nns_edge_loge ("Invalid param, given edge data is invalid.");
-    ret = NNS_EDGE_ERROR_INVALID_PARAMETER;
-    goto error;
+    return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (ed);
   header = (nns_edge_data_header_s *) data;
 
   /* Check mem size */
diff --git a/src/libnnstreamer-edge/nnstreamer-edge-event.c b/src/libnnstreamer-edge/nnstreamer-edge-event.c
index f40fd4d..8712772 100644
--- a/src/libnnstreamer-edge/nnstreamer-edge-event.c
+++ b/src/libnnstreamer-edge/nnstreamer-edge-event.c
@@ -80,7 +80,7 @@ nns_edge_event_create (nns_edge_event_e event, nns_edge_event_h * event_h)
     return NNS_EDGE_ERROR_OUT_OF_MEMORY;
   }
 
-  ee->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (ee, NNS_EDGE_MAGIC);
   ee->event = event;
 
   *event_h = ee;
@@ -97,12 +97,12 @@ nns_edge_event_destroy (nns_edge_event_h event_h)
 
   ee = (nns_edge_event_s *) event_h;
 
-  if (!NNS_EDGE_MAGIC_IS_VALID (ee)) {
+  if (!nns_edge_handle_is_valid (ee)) {
     nns_edge_loge ("Invalid param, given edge event is invalid.");
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  ee->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (ee, NNS_EDGE_MAGIC_DEAD);
 
   if (ee->data.destroy_cb)
     ee->data.destroy_cb (ee->data.data);
@@ -122,7 +122,7 @@ nns_edge_event_set_data (nns_edge_event_h event_h, void *data,
 
   ee = (nns_edge_event_s *) event_h;
 
-  if (!NNS_EDGE_MAGIC_IS_VALID (ee)) {
+  if (!nns_edge_handle_is_valid (ee)) {
     nns_edge_loge ("Invalid param, given edge event is invalid.");
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
@@ -153,7 +153,7 @@ nns_edge_event_get_type (nns_edge_event_h event_h, nns_edge_event_e * event)
 
   ee = (nns_edge_event_s *) event_h;
 
-  if (!NNS_EDGE_MAGIC_IS_VALID (ee)) {
+  if (!nns_edge_handle_is_valid (ee)) {
     nns_edge_loge ("Invalid param, given edge event is invalid.");
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
@@ -178,7 +178,7 @@ nns_edge_event_parse_new_data (nns_edge_event_h event_h,
 
   ee = (nns_edge_event_s *) event_h;
 
-  if (!NNS_EDGE_MAGIC_IS_VALID (ee)) {
+  if (!nns_edge_handle_is_valid (ee)) {
     nns_edge_loge ("Invalid param, given edge event is invalid.");
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
@@ -206,7 +206,7 @@ nns_edge_event_parse_capability (nns_edge_event_h event_h, char **capability)
 
   ee = (nns_edge_event_s *) event_h;
 
-  if (!NNS_EDGE_MAGIC_IS_VALID (ee)) {
+  if (!nns_edge_handle_is_valid (ee)) {
     nns_edge_loge ("Invalid param, given edge event is invalid.");
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
diff --git a/src/libnnstreamer-edge/nnstreamer-edge-internal.c b/src/libnnstreamer-edge/nnstreamer-edge-internal.c
index c7b2536..1a8b2dc 100644
--- a/src/libnnstreamer-edge/nnstreamer-edge-internal.c
+++ b/src/libnnstreamer-edge/nnstreamer-edge-internal.c
@@ -232,7 +232,7 @@ _nns_edge_cmd_init (nns_edge_cmd_s * cmd, nns_edge_cmd_e c, int64_t cid)
     return;
 
   memset (cmd, 0, sizeof (nns_edge_cmd_s));
-  cmd->info.magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (&cmd->info, NNS_EDGE_MAGIC);
   cmd->info.cmd = c;
   cmd->info.client_id = cid;
   cmd->info.num = 0;
@@ -250,7 +250,7 @@ _nns_edge_cmd_clear (nns_edge_cmd_s * cmd)
   if (!cmd)
     return;
 
-  cmd->info.magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (&cmd->info, NNS_EDGE_MAGIC_DEAD);
 
   for (i = 0; i < cmd->info.num; i++) {
     SAFE_FREE (cmd->mem[i]);
@@ -276,7 +276,7 @@ _nns_edge_cmd_is_valid (nns_edge_cmd_s * cmd)
 
   command = (int) cmd->info.cmd;
 
-  if (!NNS_EDGE_MAGIC_IS_VALID (&cmd->info) ||
+  if (!nns_edge_handle_is_valid (&cmd->info) ||
       (command < 0 || command >= _NNS_EDGE_CMD_END)) {
     return false;
   }
@@ -632,7 +632,7 @@ _nns_edge_message_handler (void *thread_data)
     struct pollfd poll_fd;
 
     /* Validate edge handle */
-    if (!NNS_EDGE_MAGIC_IS_VALID (eh)) {
+    if (!nns_edge_handle_is_valid (eh)) {
       nns_edge_loge ("The edge handle is invalid, it would be expired.");
       break;
     }
@@ -1177,7 +1177,7 @@ nns_edge_create_handle (const char *id, nns_edge_connect_type_e connect_type,
   }
 
   nns_edge_lock_init (eh);
-  eh->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (eh, NNS_EDGE_MAGIC);
   eh->id = STR_IS_VALID (id) ? nns_edge_strdup (id) :
       nns_edge_strdup_printf ("%lld", (long long) nns_edge_generate_id ());
   eh->connect_type = connect_type;
@@ -1212,14 +1212,13 @@ nns_edge_start (nns_edge_h edge_h)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (eh);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (eh)) {
+  if (!nns_edge_handle_is_valid (eh)) {
     nns_edge_loge ("Invalid param, given edge handle is invalid.");
-    nns_edge_unlock (eh);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (eh);
+
   if (eh->port <= 0) {
     eh->port = nns_edge_get_available_port ();
     if (eh->port <= 0) {
@@ -1308,14 +1307,14 @@ nns_edge_release_handle (nns_edge_h edge_h)
     nns_edge_loge ("Invalid param, given edge handle is null.");
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
-  nns_edge_lock (eh);
 
-  if (!NNS_EDGE_MAGIC_IS_VALID (eh)) {
+  if (!nns_edge_handle_is_valid (eh)) {
     nns_edge_loge ("Invalid param, given edge handle is invalid.");
-    nns_edge_unlock (eh);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (eh);
+
   switch (eh->connect_type) {
     case NNS_EDGE_CONNECT_TYPE_HYBRID:
       if (NNS_EDGE_ERROR_NONE != nns_edge_mqtt_close (eh->broker_h)) {
@@ -1332,7 +1331,7 @@ nns_edge_release_handle (nns_edge_h edge_h)
   }
 
   /* Clear event callback and handles */
-  eh->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (eh, NNS_EDGE_MAGIC_DEAD);
   eh->event_cb = NULL;
   eh->user_data = NULL;
   eh->broker_h = NULL;
@@ -1389,14 +1388,13 @@ nns_edge_set_event_callback (nns_edge_h edge_h, nns_edge_event_cb cb,
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (eh);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (eh)) {
+  if (!nns_edge_handle_is_valid (eh)) {
     nns_edge_loge ("Invalid param, given edge handle is invalid.");
-    nns_edge_unlock (eh);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (eh);
+
   ret = nns_edge_event_invoke_callback (eh->event_cb, eh->user_data,
       NNS_EDGE_EVENT_CALLBACK_RELEASED, NULL, 0, NULL);
   if (ret != NNS_EDGE_ERROR_NONE) {
@@ -1437,14 +1435,13 @@ nns_edge_connect (nns_edge_h edge_h, const char *dest_host, int dest_port)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (eh);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (eh)) {
+  if (!nns_edge_handle_is_valid (eh)) {
     nns_edge_loge ("Invalid param, given edge handle is invalid.");
-    nns_edge_unlock (eh);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (eh);
+
   if (!eh->event_cb) {
     nns_edge_loge ("NNStreamer-edge event callback is not registered.");
     nns_edge_unlock (eh);
@@ -1547,17 +1544,15 @@ nns_edge_disconnect (nns_edge_h edge_h)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (eh);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (eh)) {
+  if (!nns_edge_handle_is_valid (eh)) {
     nns_edge_loge ("Invalid param, given edge handle is invalid.");
-    nns_edge_unlock (eh);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (eh);
   _nns_edge_remove_all_connection (eh);
-
   nns_edge_unlock (eh);
+
   return NNS_EDGE_ERROR_NONE;
 }
 
@@ -1607,14 +1602,13 @@ nns_edge_send (nns_edge_h edge_h, nns_edge_data_h data_h)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (eh);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (eh)) {
+  if (!nns_edge_handle_is_valid (eh)) {
     nns_edge_loge ("Invalid param, given edge handle is invalid.");
-    nns_edge_unlock (eh);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (eh);
+
   if (!_nns_edge_is_connected (eh)) {
     nns_edge_loge ("There is no available connection.");
     nns_edge_unlock (eh);
@@ -1666,14 +1660,13 @@ nns_edge_set_info (nns_edge_h edge_h, const char *key, const char *value)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (eh);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (eh)) {
+  if (!nns_edge_handle_is_valid (eh)) {
     nns_edge_loge ("Invalid param, given edge handle is invalid.");
-    nns_edge_unlock (eh);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (eh);
+
   if (0 == strcasecmp (key, "CAPS") || 0 == strcasecmp (key, "CAPABILITY")) {
     SAFE_FREE (eh->caps_str);
     eh->caps_str = nns_edge_strdup (value);
@@ -1765,14 +1758,13 @@ nns_edge_get_info (nns_edge_h edge_h, const char *key, char **value)
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
-  nns_edge_lock (eh);
-
-  if (!NNS_EDGE_MAGIC_IS_VALID (eh)) {
+  if (!nns_edge_handle_is_valid (eh)) {
     nns_edge_loge ("Invalid param, given edge handle is invalid.");
-    nns_edge_unlock (eh);
     return NNS_EDGE_ERROR_INVALID_PARAMETER;
   }
 
+  nns_edge_lock (eh);
+
   if (0 == strcasecmp (key, "CAPS") || 0 == strcasecmp (key, "CAPABILITY")) {
     *value = nns_edge_strdup (eh->caps_str);
   } else if (0 == strcasecmp (key, "IP") || 0 == strcasecmp (key, "HOST")) {
diff --git a/src/libnnstreamer-edge/nnstreamer-edge-util.h b/src/libnnstreamer-edge/nnstreamer-edge-util.h
index e2302f8..f6a5123 100644
--- a/src/libnnstreamer-edge/nnstreamer-edge-util.h
+++ b/src/libnnstreamer-edge/nnstreamer-edge-util.h
@@ -55,7 +55,8 @@ extern "C" {
 
 #define NNS_EDGE_MAGIC 0xfeedfeed
 #define NNS_EDGE_MAGIC_DEAD 0xdeaddead
-#define NNS_EDGE_MAGIC_IS_VALID(h) ((h) && (h)->magic == NNS_EDGE_MAGIC)
+#define nns_edge_handle_is_valid(h) ((h) && *((uint32_t *)(h)) == NNS_EDGE_MAGIC)
+#define nns_edge_handle_set_magic(h,m) do { if (h) *((uint32_t *)(h)) = (m); } while (0)
 
 #define nns_edge_lock_init(h) do { pthread_mutex_init (&(h)->lock, NULL); } while (0)
 #define nns_edge_lock_destroy(h) do { pthread_mutex_destroy (&(h)->lock); } while (0)
diff --git a/tests/unittest_nnstreamer-edge.cc b/tests/unittest_nnstreamer-edge.cc
index e9456a8..e3756b4 100644
--- a/tests/unittest_nnstreamer-edge.cc
+++ b/tests/unittest_nnstreamer-edge.cc
@@ -319,20 +319,18 @@ TEST(edge, startInvalidParam01_n)
 TEST(edge, startInvalidParam02_n)
 {
   nns_edge_h edge_h;
-  nns_edge_handle_s *eh;
   int ret;
 
   ret = nns_edge_create_handle ("temp-id", NNS_EDGE_CONNECT_TYPE_TCP,
       NNS_EDGE_NODE_TYPE_QUERY_CLIENT, &edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  eh = (nns_edge_handle_s *) edge_h;
-  eh->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_start (edge_h);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  eh->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_release_handle (edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -355,20 +353,18 @@ TEST(edge, releaseHandleInvalidParam01_n)
 TEST(edge, releaseHandleInvalidParam02_n)
 {
   nns_edge_h edge_h;
-  nns_edge_handle_s *eh;
   int ret;
 
   ret = nns_edge_create_handle ("temp-id", NNS_EDGE_CONNECT_TYPE_TCP,
       NNS_EDGE_NODE_TYPE_QUERY_CLIENT, &edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  eh = (nns_edge_handle_s *) edge_h;
-  eh->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_release_handle (edge_h);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  eh->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_release_handle (edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -422,7 +418,6 @@ TEST(edge, setEventCbInvalidParam01_n)
 TEST(edge, setEventCbInvalidParam02_n)
 {
   nns_edge_h edge_h;
-  nns_edge_handle_s *eh;
   ne_test_data_s *_td;
   int ret;
 
@@ -433,13 +428,12 @@ TEST(edge, setEventCbInvalidParam02_n)
       NNS_EDGE_NODE_TYPE_QUERY_CLIENT, &edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  eh = (nns_edge_handle_s *) edge_h;
-  eh->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_set_event_callback (edge_h, _test_edge_event_cb, _td);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  eh->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_release_handle (edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -464,7 +458,6 @@ TEST(edge, connectInvalidParam01_n)
 TEST(edge, connectInvalidParam02_n)
 {
   nns_edge_h edge_h;
-  nns_edge_handle_s *eh;
   int ret;
 
   ret = nns_edge_create_handle ("temp-id", NNS_EDGE_CONNECT_TYPE_TCP,
@@ -474,13 +467,12 @@ TEST(edge, connectInvalidParam02_n)
   ret = nns_edge_set_event_callback (edge_h, _test_edge_event_cb, NULL);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  eh = (nns_edge_handle_s *) edge_h;
-  eh->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_connect (edge_h, "127.0.0.1", 80);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  eh->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_release_handle (edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -574,20 +566,18 @@ TEST(edge, disconnectInvalidParam01_n)
 TEST(edge, disconnectInvalidParam02_n)
 {
   nns_edge_h edge_h;
-  nns_edge_handle_s *eh;
   int ret;
 
   ret = nns_edge_create_handle ("temp-id", NNS_EDGE_CONNECT_TYPE_TCP,
       NNS_EDGE_NODE_TYPE_QUERY_CLIENT, &edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  eh = (nns_edge_handle_s *) edge_h;
-  eh->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_disconnect (edge_h);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  eh->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_release_handle (edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -621,7 +611,6 @@ TEST(edge, sendInvalidParam02_n)
 {
   nns_edge_h edge_h;
   nns_edge_data_h data_h;
-  nns_edge_handle_s *eh;
   int ret;
 
   ret = nns_edge_create_handle ("temp-id", NNS_EDGE_CONNECT_TYPE_TCP,
@@ -634,13 +623,12 @@ TEST(edge, sendInvalidParam02_n)
   ret = nns_edge_data_set_info (data_h, "client_id", "10");
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  eh = (nns_edge_handle_s *) edge_h;
-  eh->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_send (edge_h, data_h);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  eh->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_release_handle (edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -685,20 +673,18 @@ TEST(edge, setInfoInvalidParam01_n)
 TEST(edge, setInfoInvalidParam02_n)
 {
   nns_edge_h edge_h;
-  nns_edge_handle_s *eh;
   int ret;
 
   ret = nns_edge_create_handle ("temp-id", NNS_EDGE_CONNECT_TYPE_TCP,
       NNS_EDGE_NODE_TYPE_QUERY_CLIENT, &edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  eh = (nns_edge_handle_s *) edge_h;
-  eh->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_set_info (edge_h, "caps", "temp-caps");
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  eh->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_release_handle (edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -952,7 +938,6 @@ TEST(edge, getInfoInvalidParam01_n)
 TEST(edge, getInfoInvalidParam02_n)
 {
   nns_edge_h edge_h;
-  nns_edge_handle_s *eh;
   char *value;
   int ret;
 
@@ -960,13 +945,12 @@ TEST(edge, getInfoInvalidParam02_n)
       NNS_EDGE_NODE_TYPE_QUERY_CLIENT, &edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  eh = (nns_edge_handle_s *) edge_h;
-  eh->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_get_info (edge_h, "temp-key", &value);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  eh->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (edge_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_release_handle (edge_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -1079,19 +1063,17 @@ TEST(edgeData, destroyInvalidParam01_n)
 TEST(edgeData, destroyInvalidParam02_n)
 {
   nns_edge_data_h data_h;
-  nns_edge_data_s *ed;
   int ret;
 
   ret = nns_edge_data_create (&data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) data_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -1132,19 +1114,17 @@ TEST(edgeData, validateInvalidParam01_n)
 TEST(edgeData, validateInvalidParam02_n)
 {
   nns_edge_data_h data_h;
-  nns_edge_data_s *ed;
   int ret;
 
   ret = nns_edge_data_create (&data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) data_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_is_valid (data_h);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -1228,19 +1208,17 @@ TEST(edgeData, copyInvalidParam01_n)
 TEST(edgeData, copyInvalidParam02_n)
 {
   nns_edge_data_h src_h, desc_h;
-  nns_edge_data_s *ed;
   int ret;
 
   ret = nns_edge_data_create (&src_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) src_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (src_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_copy (src_h, &desc_h);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (src_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (src_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -1320,7 +1298,6 @@ TEST(edgeData, addInvalidParam01_n)
 TEST(edgeData, addInvalidParam02_n)
 {
   nns_edge_data_h data_h;
-  nns_edge_data_s *ed;
   void *data;
   nns_size_t data_len;
   int ret;
@@ -1332,13 +1309,12 @@ TEST(edgeData, addInvalidParam02_n)
   ret = nns_edge_data_create (&data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) data_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_add (data_h, data, data_len, NULL);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -1453,7 +1429,6 @@ TEST(edgeData, getInvalidParam01_n)
 TEST(edgeData, getInvalidParam02_n)
 {
   nns_edge_data_h data_h;
-  nns_edge_data_s *ed;
   void *data, *result;
   nns_size_t data_len, result_len;
   int ret;
@@ -1468,13 +1443,12 @@ TEST(edgeData, getInvalidParam02_n)
   ret = nns_edge_data_add (data_h, data, data_len, NULL);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) data_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_get (data_h, 0, &result, &result_len);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -1588,7 +1562,6 @@ TEST(edgeData, getCountInvalidParam01_n)
 TEST(edgeData, getCountInvalidParam02_n)
 {
   nns_edge_data_h data_h;
-  nns_edge_data_s *ed;
   void *data;
   nns_size_t data_len;
   unsigned int count;
@@ -1604,13 +1577,12 @@ TEST(edgeData, getCountInvalidParam02_n)
   ret = nns_edge_data_add (data_h, data, data_len, NULL);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) data_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_get_count (data_h, &count);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -1664,19 +1636,17 @@ TEST(edgeData, setInfoInvalidParam01_n)
 TEST(edgeData, setInfoInvalidParam02_n)
 {
   nns_edge_data_h data_h;
-  nns_edge_data_s *ed;
   int ret;
 
   ret = nns_edge_data_create (&data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) data_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_set_info (data_h, "temp-key", "temp-value");
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -1736,7 +1706,6 @@ TEST(edgeData, getInfoInvalidParam01_n)
 TEST(edgeData, getInfoInvalidParam02_n)
 {
   nns_edge_data_h data_h;
-  nns_edge_data_s *ed;
   char *value = NULL;
   int ret;
 
@@ -1746,13 +1715,12 @@ TEST(edgeData, getInfoInvalidParam02_n)
   ret = nns_edge_data_set_info (data_h, "temp-key", "temp-value");
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) data_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_get_info (data_h, "temp-key", &value);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -1841,7 +1809,6 @@ TEST(edgeData, serializeInvalidParam01_n)
 TEST(edgeData, serializeInvalidParam02_n)
 {
   nns_edge_data_h data_h;
-  nns_edge_data_s *ed;
   void *data;
   nns_size_t data_len;
   int ret;
@@ -1852,13 +1819,12 @@ TEST(edgeData, serializeInvalidParam02_n)
   ret = nns_edge_data_set_info (data_h, "temp-key", "temp-value");
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) data_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_serialize_meta (data_h, &data, &data_len);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -1942,7 +1908,6 @@ TEST(edgeData, deserializeInvalidParam01_n)
 TEST(edgeData, deserializeInvalidParam02_n)
 {
   nns_edge_data_h data_h;
-  nns_edge_data_s *ed;
   void *data;
   nns_size_t data_len;
   int ret;
@@ -1956,13 +1921,12 @@ TEST(edgeData, deserializeInvalidParam02_n)
   ret = nns_edge_data_serialize_meta (data_h, &data, &data_len);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) data_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_deserialize_meta (data_h, data, data_len);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -2126,7 +2090,6 @@ TEST(edgeDataSerialize, invalidParam01_n)
 TEST(edgeData, invalidParam02_n)
 {
   nns_edge_data_h data_h;
-  nns_edge_data_s *ed;
   void *data;
   nns_size_t data_len;
   int ret;
@@ -2137,13 +2100,12 @@ TEST(edgeData, invalidParam02_n)
   ret = nns_edge_data_set_info (data_h, "temp-key", "temp-value");
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) data_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_serialize (data_h, &data, &data_len);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -2213,7 +2175,6 @@ TEST(edgeDataDeserialize, invalidParam01_n)
 TEST(edgeDataDeserialize, invalidParam02_n)
 {
   nns_edge_data_h data_h;
-  nns_edge_data_s *ed;
   void *data;
   nns_size_t data_len;
   int ret;
@@ -2227,13 +2188,12 @@ TEST(edgeDataDeserialize, invalidParam02_n)
   ret = nns_edge_data_serialize (data_h, &data, &data_len);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ed = (nns_edge_data_s *) data_h;
-  ed->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_data_deserialize (data_h, data, data_len);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ed->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (data_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_data_destroy (data_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -2337,7 +2297,6 @@ TEST(edgeEvent, destroyInvalidParam01_n)
 TEST(edgeEvent, destroyInvalidParam02_n)
 {
   nns_edge_event_h event_h;
-  nns_edge_event_s *ee;
   void *data;
   nns_size_t data_len;
   int ret;
@@ -2352,13 +2311,12 @@ TEST(edgeEvent, destroyInvalidParam02_n)
   ret = nns_edge_event_set_data (event_h, data, data_len, free);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ee = (nns_edge_event_s *) event_h;
-  ee->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (event_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_event_destroy (event_h);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ee->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (event_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_event_destroy (event_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -2441,7 +2399,6 @@ TEST(edgeEvent, setDataInvalidParam03_n)
 TEST(edgeEvent, setDataInvalidParam04_n)
 {
   nns_edge_event_h event_h;
-  nns_edge_event_s *ee;
   void *data;
   nns_size_t data_len;
   int ret;
@@ -2453,13 +2410,12 @@ TEST(edgeEvent, setDataInvalidParam04_n)
   ret = nns_edge_event_create (NNS_EDGE_EVENT_CUSTOM, &event_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ee = (nns_edge_event_s *) event_h;
-  ee->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (event_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_event_set_data (event_h, data, data_len, NULL);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ee->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (event_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_event_destroy (event_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -2524,19 +2480,17 @@ TEST(edgeEvent, getTypeInvalidParam03_n)
 {
   nns_edge_event_h event_h;
   nns_edge_event_e event;
-  nns_edge_event_s *ee;
   int ret;
 
   ret = nns_edge_event_create (NNS_EDGE_EVENT_CUSTOM, &event_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ee = (nns_edge_event_s *) event_h;
-  ee->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (event_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_event_get_type (event_h, &event);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ee->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (event_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_event_destroy (event_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -2649,19 +2603,17 @@ TEST(edgeEvent, parseNewDataInvalidParam03_n)
 {
   nns_edge_event_h event_h;
   nns_edge_data_h data_h;
-  nns_edge_event_s *ee;
   int ret;
 
   ret = nns_edge_event_create (NNS_EDGE_EVENT_NEW_DATA_RECEIVED, &event_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ee = (nns_edge_event_s *) event_h;
-  ee->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (event_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_event_parse_new_data (event_h, &data_h);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ee->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (event_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_event_destroy (event_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
@@ -2747,20 +2699,18 @@ TEST(edgeEvent, parseCapabilityInvalidParam02_n)
 TEST(edgeEvent, parseCapabilityInvalidParam03_n)
 {
   nns_edge_event_h event_h;
-  nns_edge_event_s *ee;
   char *caps = NULL;
   int ret;
 
   ret = nns_edge_event_create (NNS_EDGE_EVENT_CAPABILITY, &event_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);
 
-  ee = (nns_edge_event_s *) event_h;
-  ee->magic = NNS_EDGE_MAGIC_DEAD;
+  nns_edge_handle_set_magic (event_h, NNS_EDGE_MAGIC_DEAD);
 
   ret = nns_edge_event_parse_capability (event_h, &caps);
   EXPECT_NE (ret, NNS_EDGE_ERROR_NONE);
 
-  ee->magic = NNS_EDGE_MAGIC;
+  nns_edge_handle_set_magic (event_h, NNS_EDGE_MAGIC);
 
   ret = nns_edge_event_destroy (event_h);
   EXPECT_EQ (ret, NNS_EDGE_ERROR_NONE);