Hi Gert, Thank you for your support. Please find attached patch with diff -uw.
Thanks,
Kazuyoshi
2010/4/29 Gert Doering <g...@greenie.muc.de>:
> Hi again,
>
> On Thu, Apr 29, 2010 at 01:02:37PM +0900, Kazuyoshi Aizawa wrote:
>> I've attached a patch for tun.c which includes support for tap
>> interface for solaris/opensolaris.
>
> I've glanced through the patch, and there's also changes in the Win32
> part of the tun.c, for example this chunk:
>
>
> ----------------------- quote ----------------------
> @@ -3333,19 +3409,19 @@
> bool ret = false;
> const IP_ADAPTER_INDEX_MAP *inter = get_interface_info (adapter_index, &gc);
>
> - if (inter)
> - {
> - DWORD status = IpRenewAddress ((IP_ADAPTER_INDEX_MAP *)inter);
> - if (status == NO_ERROR)
> + if (inter)
> {
> - msg (D_TUNTAP_INFO, "TAP: DHCP address renewal succeeded");
> - ret = true;
> + DWORD status = IpRenewAddress ((IP_ADAPTER_INDEX_MAP *)inter);
> + if (status == NO_ERROR)
> + {
> + msg (D_TUNTAP_INFO, "TAP: DHCP address renewal succeeded");
> + ret = true;
> + }
> + else
> + msg (M_WARN, "WARNING: Failed to renew DHCP IP address lease on
> TAP-Win32 adapter: %s (code=%u)",
> + strerror_win32 (status, &gc),
> + (unsigned int)status);
> }
> - else
> - msg (M_WARN, "WARNING: Failed to renew DHCP IP address lease on
> TAP-Win32 adapter: %s (code=%u)",
> - strerror_win32 (status, &gc),
> - (unsigned int)status);
> - }
> gc_free (&gc);
> return ret;
> }
> ----------------------- quote ----------------------
>
>
> From a first glance, this looks like "whitespace changes only".
>
> For easier review: would you be so kind and re-send your patch with
> "diff -uw" (-w = ignore white space changes)?
>
> thanks,
>
> gert
> --
> USENET is *not* the non-clickable part of WWW!
> //www.muc.de/~gert/
> Gert Doering - Munich, Germany g...@greenie.muc.de
> fax: +49-89-35655025 g...@net.informatik.tu-muenchen.de
>
>
tun.c_tap_solaris.patch
Description: Binary data
