On Fri, Sep 02, 2022 at 03:32:56PM +0200, Cezary Rojewski wrote: > Make use of global integer-array parsing helper instead of the internal > one as both serve same purpose. With that, both strsplit_u32() and > tokenize_input() become unused so remove them. More minor comments... ... > + ret = parse_int_array_user(from, count, (int **)&array); > if (ret < 0) > return ret; > + num_elems = *array; > + bytes = sizeof(*array) * num_elems; > + if (!num_elems || (bytes % sizeof(*desc))) { !num_elems is a dup since previous already does this check. ... > + ret = parse_int_array_user(from, count, (int **)&array); > if (ret < 0) > return ret; > + > + num_elems = *array; > + if (!num_elems) { > ret = -EINVAL; > goto exit; > } Ditto. -- With Best Regards, Andy Shevchenko