diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2016-04-08 18:34:54 +0000 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2016-04-12 20:18:36 -0400 |
| commit | 88227bbe362c827208541f8bd145bd2cccfb1f59 (patch) | |
| tree | a799196215aff9570810d1b1bc5ffb5079fdb7de /lib/vrf.c | |
| parent | b8b341d7c359d9c6770cbecfb050fc992bf3ce1f (diff) | |
lib: passing in vrf string length is not needed
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>
Diffstat (limited to 'lib/vrf.c')
| -rw-r--r-- | lib/vrf.c | 12 |
1 files changed, 7 insertions, 5 deletions
@@ -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 */ |
