On Sun, 16 Mar 2025 16:39:17 +0100 chia-yu.chang@xxxxxxxxxxxxxxxxxxx wrote: > +static int get_packets(uint32_t *val, const char *arg) > +{ > + unsigned long res; > + char *ptr; > + > + if (!arg || !*arg) > + return -1; > + res = strtoul(arg, &ptr, 10); > + if (!ptr || ptr == arg || > + (strcmp(ptr, "p") && strcmp(ptr, "pkt") && strcmp(ptr, "pkts") && > + strcmp(ptr, "packet") && strcmp(ptr, "packets"))) > + return -1; No shortcuts please. We ran into this with matches() and arg conflicts already.