On Wed, Apr 05, 2023 at 10:09:46AM +0200, Claudio Jeker wrote: > Allow to use service names like 'bgp' for port definitions. > Adapted from pfctl/parse.y.
[...]
> +static int
> +getservice(char *n)
> +{
> + struct servent *s;
> +
> + s = getservbyname(n, "tcp");
> + if (s == NULL)
> + s = getservbyname(n, "udp");
> + if (s == NULL)
> + return (-1);
I'd omit the parentheses here to keep the style in this file consistent
on a per-function level.
ok
> + return s->s_port;
> }
