]> git.puffer.fish Git - matthieu/frr.git/commitdiff
lib: passing in vrf string length is not needed
authorDonald Sharp <sharpd@cumulusnetworks.com>
Fri, 8 Apr 2016 18:34:54 +0000 (18:34 +0000)
committerDonald Sharp <sharpd@cumulusnetworks.com>
Wed, 13 Apr 2016 00:18:36 +0000 (20:18 -0400)
vrf_create does not need the string length of the vrf.
Remove it.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Reviewed-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
lib/vrf.c
lib/vrf.h

index f730c539f5a388ce747620a30ae3f744522525cc..edba3547c59f53dc5089bb75be3e9ffccf6fb656 100644 (file)
--- a/lib/vrf.c
+++ b/lib/vrf.c
@@ -72,16 +72,18 @@ vrf_list_lookup_by_name (const char *name)
 
 /* Create new vrf structure. */
 struct vrf *
-vrf_create (const char *name, size_t namelen)
+vrf_create (const char *name)
 {
   struct vrf *vrfp;
 
   vrfp = XCALLOC (MTYPE_VRF, sizeof (struct vrf));
 
   assert (name);
-  assert (namelen <= VRF_NAMSIZ);      /* Need space for '\0' at end. */
-  strncpy (vrfp->name, name, namelen);
-  vrfp->name[namelen] = '\0';
+
+  zlog_debug ("Vrf_create: %s", name);
+  strncpy (vrfp->name, name, VRF_NAMSIZ);
+  vrfp->name[VRF_NAMSIZ] = '\0';
+
   if (vrf_list_lookup_by_name (vrfp->name) == NULL)
     listnode_add_sort (vrf_list, vrfp);
   else
@@ -102,7 +104,7 @@ vrf_get_by_name (const char *name)
   struct vrf *vrfp;
 
   return ((vrfp = vrf_list_lookup_by_name (name)) != NULL) ? vrfp :
-          vrf_create (name, strlen(name));
+          vrf_create (name);
 }
 
 /* Build the table key */
index 3172b4f4c539d9812c3f83db89e5e2d3b1325d6b..79741d3b0e649a9d6ae0d58bebb6256da07aea49 100644 (file)
--- a/lib/vrf.h
+++ b/lib/vrf.h
@@ -111,7 +111,7 @@ extern struct vrf *vrf_lookup_by_name (const char *);
 extern struct vrf *vrf_list_lookup_by_name (const char *);
 extern struct vrf *vrf_get_by_name (const char *);
 extern struct vrf *vrf_get (vrf_id_t, const char *);
-extern struct vrf *vrf_create (const char *, size_t);
+extern struct vrf *vrf_create (const char *);
 extern void vrf_delete (struct vrf *);
 extern int vrf_enable (struct vrf *);
 extern vrf_id_t vrf_name_to_id (const char *);