Ticket #89 did it wrong

[guest]

The changes to fix ticket #89  http://dev.kryo.se/iodine/changeset/965 are wrong:

1. users_get_first_ip() uses inet_ntoa() and returns

2. tun_setip() gets passed the result as remoteip

3. tun_setip() uses inet_ntoa() and therefore remoteip gets overwritten with inet_ntoa(net).

One solution would be to change

if (tun_setip(argv[0], users_get_first_ip(), netmask) != 0

tun_setmtu(mtu) != 0) {

to

if (tun_setip(argv[0], strdup( users_get_first_ip() ), netmask) != 0

tun_setmtu(mtu) != 0) {

or even let users_get_first_ip() return with a strdup'ed value.

[yarrick]

Commited to trunk in [974].