09/10/2018 15:34, Thomas Monjalon: > +int __rte_experimental > +rte_eth_iterator_init(struct rte_dev_iterator *iter, const char *devargs_str) > +{ > + int ret; > + struct rte_devargs devargs = {.args = NULL}; > + const char *bus_param_key; > + char *bus_str = NULL; > + char *cls_str = NULL; > + size_t str_size;
Because of the new test below, the type must be fixed to int. [...] > + ret = snprintf(cls_str, str_size, "+%s", devargs.args); > + if (ret != str_size - 1) { > + ret = -EINVAL; > + goto error; > + }