Hi,
On Tue, Nov 22, 2016 at 6:38 AM, Arne Schwabe <a...@rfc2549.org> wrote:
...


> -  for (i = 0; i < tt->options.dns_len; ++i) {
> +  /* Prefer IPv6 DNS servers,
> +   * Android will use the DNS server in the order we specify*/
> +  for (int i = 0; i < tt->options.dns6_len; i++) {
> +    management_android_control (management, "DNS6SERVER",
> +                               print_in6_addr (tt->options.dns6[i], 0,
> &gc));
> +  }
> +
> +  for (int i = 0; i < tt->options.dns_len; i++) {
>      management_android_control (management, "DNSSERVER",
>                                 print_in_addr_t(tt->options.dns[i], 0,
> &gc));
>    }


The code looks good to me, so ACK
(Note: the preferred comment closing style may be '*/' on a line by
itself?).

Selva
------------------------------------------------------------------------------
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to