From 9aa7cd1d8d137bdb11f963af3e29ba4f421ab6b3 Mon Sep 17 00:00:00 2001
From: dimitri staessens <dimitri.staessens@intec.ugent.be>
Date: Wed, 30 Mar 2016 13:17:34 +0200
Subject: lib: fixes for instance_name

fixes wrong check, checks now use lazy evaluation
changed the order of instance_name_cpy to (dst, src)
---
 src/lib/instance_name.c | 26 ++++++++++----------------
 1 file changed, 10 insertions(+), 16 deletions(-)

(limited to 'src/lib/instance_name.c')

diff --git a/src/lib/instance_name.c b/src/lib/instance_name.c
index 1e61f790..0f666211 100644
--- a/src/lib/instance_name.c
+++ b/src/lib/instance_name.c
@@ -86,7 +86,7 @@ instance_name_t * instance_name_init_from(instance_name_t * dst,
 }
 
 instance_name_t * instance_name_init_with(instance_name_t * dst,
-                                          char *            name,
+                                          char *      name,
                                           uint16_t          id)
 {
         if (dst == NULL)
@@ -103,13 +103,11 @@ instance_name_t * instance_name_init_with(instance_name_t * dst,
 
 void instance_name_fini(instance_name_t * n)
 {
-        if (n == NULL)
+        if (n == NULL || n->name == NULL)
                 return;
 
-        if (n->name != NULL) {
-                free(n->name);
-                n->name = NULL;
-        }
+        free(n->name);
+        n->name = NULL;
 }
 
 void instance_name_destroy(instance_name_t * ptr)
@@ -122,17 +120,15 @@ void instance_name_destroy(instance_name_t * ptr)
         free(ptr);
 }
 
-int instance_name_cpy(const instance_name_t * src,
-             instance_name_t *       dst)
+int instance_name_cpy(instance_name_t *       dst,
+                      const instance_name_t * src)
 {
         instance_name_t * res;
 
         if (src == NULL || dst == NULL)
                 return -1;
 
-        res = instance_name_init_from(dst,
-                             src->name,
-                             src->id);
+        res = instance_name_init_from(dst, src->name, src->id);
         if (res == NULL)
                 return -1;
 
@@ -150,7 +146,7 @@ instance_name_t * instance_name_dup(const instance_name_t * src)
         if (tmp == NULL)
                 return NULL;
 
-        if (instance_name_cpy(src, tmp)) {
+        if (instance_name_cpy(tmp, src)) {
                 instance_name_destroy(tmp);
                 return NULL;
         }
@@ -160,9 +156,7 @@ instance_name_t * instance_name_dup(const instance_name_t * src)
 
 bool instance_name_is_valid(const instance_name_t * n)
 {
-        return (n != NULL &&
-          n->name != NULL &&
-          strlen(n->name));
+        return (n != NULL && n->name != NULL && strlen(n->name));
 }
 
 int instance_name_cmp(const instance_name_t * a,
@@ -205,7 +199,7 @@ char * instance_name_to_string(const instance_name_t * n)
         if (n == NULL)
                 return NULL;
 
-        size  = 0;
+        size = 0;
 
         size += (n->name != NULL ?
                  strlen(n->name) : none_len);
-- 
cgit v1.2.3