summaryrefslogtreecommitdiffstats
path: root/src/vnet/policer/policer_api.c
diff options
context:
space:
mode:
authorGabriel Ganne <gabriel.ganne@enea.com>2017-10-19 15:10:46 +0200
committerChris Luke <chris_luke@comcast.com>2017-10-20 11:32:31 +0000
commit3491d7f1d05ee5e05ebf0f6129d54c5ca85e22cf (patch)
tree28f7f1a72f41e4304c26986803ffef20a06eceb2 /src/vnet/policer/policer_api.c
parent470a011511a41dcb893df12b33261030717d1e08 (diff)
null-terminate some formatted string
Any u8* variable created by format() is NOT null-terminated. Add the null terminating byte with vec_terminate_c_string(). If that variable is used by (at least) hash_get_mem(), then it needs to be null-terminated, as it will go through string_key_sum() which makes a call to strlen. Change-Id: I4e51e1b6668f557e53af3bb897cd281598eedbc0 Signed-off-by: Gabriel Ganne <gabriel.ganne@enea.com>
Diffstat (limited to 'src/vnet/policer/policer_api.c')
-rw-r--r--src/vnet/policer/policer_api.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/vnet/policer/policer_api.c b/src/vnet/policer/policer_api.c
index 3dc2cdd6a0a..686ed508a59 100644
--- a/src/vnet/policer/policer_api.c
+++ b/src/vnet/policer/policer_api.c
@@ -58,6 +58,7 @@ vl_api_policer_add_del_t_handler (vl_api_policer_add_del_t * mp)
u32 policer_index;
name = format (0, "%s", mp->name);
+ vec_terminate_c_string (name);
memset (&cfg, 0, sizeof (cfg));
cfg.rfc = mp->type;
@@ -152,6 +153,7 @@ vl_api_policer_dump_t_handler (vl_api_policer_dump_t * mp)
if (mp->match_name_valid)
{
match_name = format (0, "%s%c", mp->match_name, 0);
+ vec_terminate_c_string (match_name);
}
if (mp->match_name_valid)