}
M (GET_FIRST_MSG_ID, mp);
- vl_api_vec_to_api_string (name, &mp->name);
+ clib_memcpy (mp->name, name, vec_len (name));
S (mp);
W (ret);
return ret;
u32 context; /* opaque value to be returned in the reply */
i32 ctx_quota; /* requested punt context quota */
u64 input_queue; /* client's queue */
- string name [limit=64]; /* for show, find by name, whatever */
+ u8 name[64]; /* for show, find by name, whatever */
u32 api_versions[8]; /* client-server pairs use as desired */
};
define get_first_msg_id {
u32 client_index;
u32 context;
- string name [limit=64];
+ u8 name[64];
};
define get_first_msg_id_reply {
u32 major;
u32 minor;
u32 patch;
- string name [limit=64];
+ u8 name[64];
};
define api_versions {
u32 client_index;
{
u32 client_index;
u32 context;
- string plugin_name [limit=128];
+ u8 plugin_name[128];
u16 first_msg_id;
u16 last_msg_id;
};
*/
define sockclnt_create {
u32 context; /* opaque value to be returned in the reply */
- string name [limit=64]; /* for show, find by name, whatever */
+ u8 name[64]; /* for show, find by name, whatever */
};
define sockclnt_create_reply {
q = regp->vl_input_queue = (svm_queue_t *) (uword) mp->input_queue;
- regp->name = vl_api_from_api_to_vec (&mp->name);
+ regp->name = format (0, "%s", mp->name);
vec_add1 (regp->name, 0);
if (am->serialized_message_table_in_shmem == 0)
mp->_vl_msg_id = ntohs (VL_API_MEMCLNT_CREATE);
mp->ctx_quota = ctx_quota;
mp->input_queue = (uword) vl_input_queue;
- vl_api_to_api_string (strnlen_s (name, 64), name, &mp->name);
+ strncpy ((char *) mp->name, name, sizeof (mp->name) - 1);
vl_msg_api_send_shmem (shmem_hdr->vl_input_queue, (u8 *) & mp);
clib_time_t clib_time;
u16 rv = ~0;
- size_t plugin_name_len = strnlen_s (plugin_name, 128);
- if (plugin_name_len == 128)
+ if (strlen (plugin_name) + 1 > sizeof (mp->name))
return (rv);
clib_memset (&clib_time, 0, sizeof (clib_time));
clib_memset (mp, 0, sizeof (*mp));
mp->_vl_msg_id = ntohs (VL_API_GET_FIRST_MSG_ID);
mp->client_index = am->my_client_index;
- vl_api_to_api_string (plugin_name_len, plugin_name, &mp->name);
+ strncpy ((char *) mp->name, plugin_name, sizeof (mp->name) - 1);
if (vl_socket_client_write () <= 0)
goto sock_err;
clib_memset (mp, 0, sizeof (*mp));
mp->_vl_msg_id = ntohs (VL_API_GET_FIRST_MSG_ID);
mp->client_index = am->my_client_index;
- vl_api_to_api_string (plugin_name_len, plugin_name, &mp->name);
+ strncpy ((char *) mp->name, plugin_name, sizeof (mp->name) - 1);
vl_msg_api_send_shmem (am->shmem_hdr->vl_input_queue, (u8 *) & mp);
ASSERT (regp->registration_type == REGISTRATION_TYPE_SOCKET_SERVER);
- regp->name = vl_api_from_api_to_vec (&mp->name);
- vec_add1 (regp->name, 0);
+ regp->name = format (0, "%s%c", mp->name, 0);
u32 size = sizeof (*rp) + (nmsg * sizeof (vl_api_message_table_entry_t));
rp = vl_msg_api_alloc_zero (size);
mp = vl_socket_client_msg_alloc (sizeof (*mp));
mp->_vl_msg_id = htons (VL_API_SOCKCLNT_CREATE);
-
- vl_api_to_api_string (strnlen_s (client_name, 64), client_name, &mp->name);
+ strncpy ((char *) mp->name, client_name, sizeof (mp->name) - 1);
+ mp->name[sizeof (mp->name) - 1] = 0;
mp->context = 0xfeedface;
clib_time_init (&scm->clib_time);
vl_api_trace_plugin_msg_ids_t_print (vl_api_trace_plugin_msg_ids_t * a,
void *handle)
{
- u8 *plugin_name = vl_api_from_api_to_vec (&a->plugin_name);
- vl_print (handle, "vl_api_trace_plugin_msg_ids: %v first %u last %u\n",
- plugin_name,
+ vl_print (handle, "vl_api_trace_plugin_msg_ids: %s first %u last %u\n",
+ a->plugin_name,
clib_host_to_net_u16 (a->first_msg_id),
clib_host_to_net_u16 (a->last_msg_id));
- vec_free (plugin_name);
return handle;
}
uword *p;
api_main_t *am = &api_main;
vl_api_msg_range_t *rp;
+ u8 name[64];
u16 first_msg_id = ~0;
int rv = -7; /* VNET_API_ERROR_INVALID_VALUE */
if (!regp)
return;
- u8 *name = vl_api_from_api_to_vec (&mp->name);
-
if (am->msg_range_by_name == 0)
goto out;
-
+ strncpy ((char *) name, (char *) mp->name, ARRAY_LEN (name));
+ name[ARRAY_LEN (name) - 1] = '\0';
p = hash_get_mem (am->msg_range_by_name, name);
if (p == 0)
goto out;
rv = 0;
out:
- vec_free (name);
rmp = vl_msg_api_alloc (sizeof (*rmp));
rmp->_vl_msg_id = ntohs (VL_API_GET_FIRST_MSG_ID_REPLY);
rmp->context = mp->context;
rmp->api_versions[i].major = htonl (vl->major);
rmp->api_versions[i].minor = htonl (vl->minor);
rmp->api_versions[i].patch = htonl (vl->patch);
- vl_api_to_api_string (strnlen (vl->name, 64), vl->name,
- &rmp->api_versions[i].name);
+ strncpy ((char *) rmp->api_versions[i].name, vl->name,
+ ARRAY_LEN (rmp->api_versions[i].name));
+ rmp->api_versions[i].name[ARRAY_LEN (rmp->api_versions[i].name) - 1] =
+ '\0';
}
vl_api_send_msg (reg, (u8 *) rmp);
clib_memset (mp, 0, sizeof (*mp));
mp->_vl_msg_id = clib_host_to_net_u16 (VL_API_TRACE_PLUGIN_MSG_IDS);
- vl_api_to_api_string (strnlen_s ((char *) name, 64), (char *) name,
- &mp->plugin_name);
+ strncpy ((char *) mp->plugin_name, (char *) name,
+ sizeof (mp->plugin_name) - 1);
mp->first_msg_id = clib_host_to_net_u16 (first_msg_id);
mp->last_msg_id = clib_host_to_net_u16 (last_msg_id);
if (am->replay_in_progress == 0)
return;
- u8 *plugin_name = vl_api_from_api_to_vec (&mp->plugin_name);
- vec_add1 (plugin_name, 0);
-
- p = hash_get_mem (am->msg_range_by_name, plugin_name);
+ p = hash_get_mem (am->msg_range_by_name, mp->plugin_name);
if (p == 0)
{
clib_warning ("WARNING: traced plugin '%s' not in current image",
- plugin_name);
+ mp->plugin_name);
return;
}
if (rp->first_msg_id != clib_net_to_host_u16 (mp->first_msg_id))
{
clib_warning ("WARNING: traced plugin '%s' first message id %d not %d",
- plugin_name, clib_net_to_host_u16 (mp->first_msg_id),
+ mp->plugin_name, clib_net_to_host_u16 (mp->first_msg_id),
rp->first_msg_id);
}
if (rp->last_msg_id != clib_net_to_host_u16 (mp->last_msg_id))
{
clib_warning ("WARNING: traced plugin '%s' last message id %d not %d",
- plugin_name, clib_net_to_host_u16 (mp->last_msg_id),
+ mp->plugin_name, clib_net_to_host_u16 (mp->last_msg_id),
rp->last_msg_id);
}
- vec_free (plugin_name);
}
#define foreach_rpc_api_msg \
def get_c_def(self):
if self.len is not None:
- try:
- return "%s %s[%d];" % (self.type.get_c_name(), self.name, self.len)
- except:
- raise Exception("%s %s[%s];" % (self.type.get_c_name(), self.name, self.len))
+ return "%s %s[%d];" % (self.type.get_c_name(), self.name, self.len)
else:
return "%s %s;" % (self.type.get_c_name(), self.name)
else:
field_type = json_parser.lookup_type_like_id(field[0])
logger.debug("Parsing message field `%s'" % field)
- f = parse_field(field_class, fields, field, field_type)
- logger.debug("Parsed field `%s'" % f)
- fields.append(f)
+ l = len(field)
+ if any(type(n) is dict for n in field):
+ l -= 1
+ if l == 2:
+ if self.header is not None and\
+ self.header.has_field(field[1]):
+ continue
+ p = field_class(field_name=field[1],
+ field_type=field_type)
+ elif l == 3:
+ if field[2] == 0:
+ raise ParseError(
+ "While parsing message `%s': variable length "
+ "array `%s' doesn't have reference to member "
+ "containing the actual length" % (
+ name, field[1]))
+ p = field_class(
+ field_name=field[1],
+ field_type=field_type,
+ array_len=field[2])
+ elif l == 4:
+ nelem_field = None
+ for f in fields:
+ if f.name == field[3]:
+ nelem_field = f
+ if nelem_field is None:
+ raise ParseError(
+ "While parsing message `%s': couldn't find "
+ "variable length array `%s' member containing "
+ "the actual length `%s'" % (
+ name, field[1], field[3]))
+ p = field_class(
+ field_name=field[1],
+ field_type=field_type,
+ array_len=field[2],
+ nelem_field=nelem_field)
+ else:
+ raise Exception("Don't know how to parse message "
+ "definition for message `%s': `%s'" %
+ (m, m[1:]))
+ logger.debug("Parsed field `%s'" % p)
+ fields.append(p)
self.fields = fields
self.depends = [f.type for f in self.fields]
logger.debug("Parsed message: %s" % self)
self.crc)
-def parse_field(field_class, fields, field, field_type):
- l = len(field)
- if l > 2:
- if type(field[2]) is dict:
- if "limit" in field[2]:
- array_len = field[2]["limit"]
- else:
- l -= 1
- else:
- array_len = field[2]
-
- if l == 2:
- return field_class(field_name=field[1],
- field_type=field_type)
- elif l == 3:
- if field[2] == 0:
- raise ParseError("While parsing type `%s': array `%s' has "
- "variable length" % (name, field[1]))
- return field_class(field_name=field[1],
- field_type=field_type,
- array_len=array_len)
- elif l == 4:
- nelem_field = None
- for f in fields:
- if f.name == field[3]:
- nelem_field = f
- if nelem_field is None:
- raise ParseError(
- "While parsing message `%s': couldn't find "
- "variable length array `%s' member containing "
- "the actual length `%s'" % (
- name, field[1], field[3]))
- return field_class(field_name=field[1],
- field_type=field_type,
- array_len=array_len,
- nelem_field=nelem_field)
- else:
- raise ParseError(
- "Don't know how to parse field `%s' of type definition "
- "for type `%s'" % (field, t))
-
-
class StructType (Type, Struct):
def __init__(self, definition, json_parser, field_class, logger):
continue
field_type = json_parser.lookup_type_like_id(field[0])
logger.debug("Parsing type field `%s'" % field)
- f = parse_field(field_class, fields, field, field_type)
- fields.append(f)
+ if len(field) == 2:
+ p = field_class(field_name=field[1],
+ field_type=field_type)
+ elif len(field) == 3:
+ if field[2] == 0:
+ raise ParseError("While parsing type `%s': array `%s' has "
+ "variable length" % (name, field[1]))
+ p = field_class(field_name=field[1],
+ field_type=field_type,
+ array_len=field[2])
+ elif len(field) == 4:
+ nelem_field = None
+ for f in fields:
+ if f.name == field[3]:
+ nelem_field = f
+ if nelem_field is None:
+ raise ParseError(
+ "While parsing message `%s': couldn't find "
+ "variable length array `%s' member containing "
+ "the actual length `%s'" % (
+ name, field[1], field[3]))
+ p = field_class(field_name=field[1],
+ field_type=field_type,
+ array_len=field[2],
+ nelem_field=nelem_field)
+ else:
+ raise ParseError(
+ "Don't know how to parse field `%s' of type definition "
+ "for type `%s'" % (field, t))
+ fields.append(p)
Type.__init__(self, name)
Struct.__init__(self, name, fields)
(vl_api_memclnt_create_t * mp, void *handle)
{
u8 *s;
- u8 *name = vl_api_from_api_to_vec (&mp->name);
- s = format (0, "SCRIPT: memclnt_create name %v ", name);
- vec_free (name);
+ s = format (0, "SCRIPT: memclnt_create name %s ", mp->name);
FINISH;
}
(vl_api_sockclnt_create_t * mp, void *handle)
{
u8 *s;
- u8 *name = vl_api_from_api_to_vec (&mp->name);
- s = format (0, "SCRIPT: sockclnt_create name %s ", name);
+ s = format (0, "SCRIPT: sockclnt_create name %s ", mp->name);
- vec_free (name);
FINISH;
}
{
ck_assert_int_eq (VAPI_OK, rv);
ck_assert_int_eq (true, is_last);
- ck_assert_str_eq ("vpe", (char *) vl_api_from_api_string (p->program));
+ ck_assert_str_eq ("vpe", (char *) vl_api_from_api_string (&p->program));
printf
("show_version_reply: program: `%s', version: `%s', build directory: "
"`%s', build date: `%s'\n",
- vl_api_from_api_string (p->program),
- vl_api_from_api_string (p->version),
- vl_api_from_api_string (p->build_directory),
- vl_api_from_api_string (p->build_date));
+ vl_api_from_api_string (&p->program),
+ vl_api_from_api_string (&p->version),
+ vl_api_from_api_string (&p->build_directory),
+ vl_api_from_api_string (&p->build_date));
++*(int *) caller_ctx;
return VAPI_OK;
}
ck_assert_ptr_ne (NULL, msg);
vapi_msg_show_version_reply *reply = msg;
ck_assert_str_eq ("vpe",
- (char *) vl_api_from_api_string (reply->payload.program));
+ (char *) vl_api_from_api_string (&reply->
+ payload.program));
return VAPI_OK;
}
auto &p = r.get_payload ();
printf ("show_version_reply: program: `%s', version: `%s', build directory: "
"`%s', build date: `%s'\n",
- vl_api_from_api_string (p.program),
- vl_api_from_api_string (p.version),
- vl_api_from_api_string (p.build_directory),
- vl_api_from_api_string (p.build_date));
- ck_assert_str_eq ("vpe", (char *)vl_api_from_api_string (p.program));
+ vl_api_from_api_string (&p.program),
+ vl_api_from_api_string (&p.version),
+ vl_api_from_api_string (&p.build_directory),
+ vl_api_from_api_string (&p.build_date));
+ ck_assert_str_eq ("vpe", (char *)vl_api_from_api_string (&p.program));
}
Connection con;