On Sun, 19 Feb 2023 19:04:28 +0100
Daniel Gröber <d...@darkboxed.org> wrote:
> +static inline bool parse_address_family(int *family, const char *value)
> +{
> + if (strcmp(value, "inet") == 0)
> + *family = AF_INET;
> + else if (strcmp(value, "inet6") == 0)
> + *family = AF_INET6;
Wouldn't the first condition match "inet6" as well, not ever checking the
second condition?
> + else if (strcmp(value, "unspec") == 0)
> + *family = AF_UNSPEC;
> + else
> + return false;
> +
> + return true;
> +}
--
With respect,
Roman
