stat_client: Fix the type casting 36/16336/7
authorMohsin Kazmi <sykazmi@cisco.com>
Tue, 4 Dec 2018 14:31:15 +0000 (14:31 +0000)
committerOle Trøan <otroan@employees.org>
Thu, 6 Dec 2018 21:30:53 +0000 (21:30 +0000)
Change-Id: I9726ac5cc7292a492928b213bc85ccfa8a26c1cb
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
src/vpp-api/client/stat_client.c
src/vpp-api/client/stat_client.h

index bd9d8cc..bb7f240 100644 (file)
@@ -321,7 +321,7 @@ stat_segment_ls_r (uint8_t ** patterns, stat_client_main_t * sm)
   int i, j;
   for (i = 0; i < vec_len (patterns); i++)
     {
-      int rv = regcomp (&regex[i], (char *) patterns[i], 0);
+      int rv = regcomp (&regex[i], (const char *) patterns[i], 0);
       if (rv)
        {
          fprintf (stderr, "Could not compile regex %s\n", patterns[i]);
@@ -419,12 +419,14 @@ stat_segment_vec_free (void *vec)
 }
 
 /* Create a vector from a string (or add to existing) */
-u8 **
-stat_segment_string_vector (u8 ** string_vector, char *string)
+uint8_t **
+stat_segment_string_vector (uint8_t ** string_vector, const char *string)
 {
-  u8 *name = 0;
-  name = vec_dup ((u8 *) string);
-  vec_add1 (string_vector, (u8 *) name);
+  uint8_t *name = 0;
+  size_t len = strlen (string);
+
+  vec_validate_init_c_string (name, string, len);
+  vec_add1 (string_vector, name);
   return string_vector;
 }
 
index 52aaa24..89f6fe5 100644 (file)
@@ -54,7 +54,8 @@ int stat_segment_connect_r (const char *socket_name, stat_client_main_t * sm);
 int stat_segment_connect (const char *socket_name);
 void stat_segment_disconnect_r (stat_client_main_t * sm);
 void stat_segment_disconnect (void);
-uint8_t **stat_segment_string_vector (uint8_t ** string_vector, char *string);
+uint8_t **stat_segment_string_vector (uint8_t ** string_vector,
+                                     const char *string);
 int stat_segment_vec_len (void *vec);
 void stat_segment_vec_free (void *vec);
 uint32_t *stat_segment_ls_r (uint8_t ** patterns, stat_client_main_t * sm);