On Sun, Oct 14, 2018 at 11:00:38AM +0200, Florian Obser wrote:
> I went through the tree and found these uneeded checks.
> They are unneeded because either:
> 
> - we are asking getaddrinfo for a specific AF and shove the result
>   into the right sized sockaddr_in{,6}
> - we are asking for AF_UNSPEC and switch on the returned ai_family
> - we stuff the result into sockaddr_storage which is guaranteed
>   to be large enough
> 
> The checks in radiusd and switchd are necessary since they either
> use a union or struct sockaddr.
I agree, diff looks OK.

Reply via email to