Klemens Nanni <k...@openbsd.org> wrote: > Thus I'd like to remove error checks that are never hit and use > sizeof(source) instead of hardcoded IFNAMSIZ while there.
Never use sizeof(source). Always use sizeof(dest). Which it appears you did. I'm not sure I see the point of the (void) casts. In my experience this is a useless syntax which fails to indicate if someone recently reviewed it for correctness.