Hi,

Thank you for the review, Dmitry, Mike! I'll send out a patch soon.
 

> +     memset(&msg, 0, sizeof(msg));
> +     memset(&sa, 0, sizeof(sa));
> +     memset(&nlh, 0, sizeof(nlh));
> +     memset(&conn_req, 0, sizeof(conn_req));

do you need to zero these out when you initialize their content below ?

Indeed, the calls do seem to fail when they are not initialised.


Thank you!
Zubin