service: Prefer online state over ready when comparing services

service: Allow state downgrade from online to ready

service: Allow ready service to be moved before online one

service: Ignore ipconfig that is not active when moving service

Release 0.77

session: Defer connect/disconnect calls

When we call connect and disconnect a service we might
reenter the session state machine. So we should defer
these calls and finishing up the state change first.

session: Update reason code on disconnect

session: reason2string return a string for unknown

session: Fix selection of services

In case we didn't select a service (entry) reset the one
in the info (deselection). And then we have also take
a refence if the reason was CONNECT.

session: Don't change ref count if reason code is set.

If the reason code has been set the proper action has
been executed already.

session: Handle service trigger according the reason code.

Instead of trying to figure out what we are supposed to do
we just rely on the reason code.

session: Don't reset session if serivce was not selected

session: Update service state in all sessions

session: Initialize service hash correctly

service: Identation fix

service: Save ipconfig in states where ipconfig is set up

Service states unknown, idle and association do not have
valid ipconfig data configured. Do not save it in these states.

network: Re-read ipconfig from service when setting manual addresses

When a service was disconnected, its ipconfig data was cleared.
Re-read service ipconfig data before setting manual addresses.

Fixes BMC#17738

service: Factor out reading ipconfigs from create functions

Create new read functions for IPv4 and IPv6 from the code in
__connman_service_create_ip*config functions.

connection: Fix IPv6 host route setting when using VPN

connection: Fix indentation

loopback: Remove dead code

service: Propagate user proxy changes

Notify proxy_changed listeners when Proxy.Configuration is set/changed.

service: Properly disconnect network or provider on connect failure

Correctly check whether a service has a network or a VPN provider
associated with it if the service fails already in
__connman_service_connect().

task: Reset forked process signal mask

The forked process inherits connmand's signal mask and thus has TERM and
INT blocked. As a consequence it can not be killed properly if the signal
mask is not reset.

connection: Fix vpn on networks without gateway

VPN establishment fails on gateway less networks, like for example ppp
based cellular ones. To fix that, a host route to the VPN server is
established.

service: Fix wrong DNS server ip after VPN connection

We need to call default_changed function for ONLINE state after
profile changed, else get_default function will return the previous
service, this will lead to we have wrong DNS after VPN connection

connection: Set proper gateway for ppp links

The default gateway for ppp connection should be 0.0.0.0 for ipv4
and :: for ipv6. This issue fixed before but removed by the commit
84a739d0082b89efa8cfbf376abe17937e4bc843

plugin: Use weak reference for external debug statements

plugin: Use simpler version mismatch error

plugin: Add debug support for external plugins

log: Use separate function for enable logging

manager: Defer return of setting session mode if not idle

notify: Add state_idle()

session: Allways allow set session mode

If the unit test framework crashes it can happen that the session mode
is still enabled but we have a connected service. So if the unit test
restarted we wont disconnect all services and therefore block for
ever. Since it is safe to call __connman_service_disconnect_all()
again there is no need to disallow this operation.

session: Fix empty service name

(gdb) bt
#0  __strlen_sse2 () at ../sysdeps/x86_64/strlen.S:32
#1  0x0000003a5ee2810b in marshal_string (pos_after=0x7fffffffdd80, byte_order=108, value=0x0,
    insert_at=20, str=0x6bcc50) at dbus-marshal-basic.c:778
#2  _dbus_marshal_write_basic (str=0x6bcc50, insert_at=20, type=<optimized out>,
    value=<optimized out>, byte_order=108, pos_after=0x7fffffffdd80)
    at dbus-marshal-basic.c:858
#3  0x0000003a5ee15f29 in _dbus_type_writer_write_basic_no_typecode (value=<optimized out>,
    type=115, writer=0x7fffffffdd60) at dbus-marshal-recursive.c:1589
#4  _dbus_type_writer_write_basic_no_typecode (value=<optimized out>, type=115, writer=
    0x7fffffffdd60) at dbus-marshal-recursive.c:2736
#5  _dbus_type_writer_write_basic (writer=0x7fffffffdd60, type=115, value=<optimized out>)
    at dbus-marshal-recursive.c:2311
#6  0x0000003a5ee19c1a in dbus_message_iter_append_basic (value=0x7fffffffde58, type=115, iter=
    0x7fffffffdd50) at dbus-message.c:2563
#7  dbus_message_iter_append_basic (iter=0x7fffffffdd50, type=115, value=0x7fffffffde58)
    at dbus-message.c:2506
#8  0x00000000004597ed in connman_dbus_property_append_basic (iter=0x7fffffffddd0, key=
    0x479539 "Name", type=115, val=0x7fffffffde58) at src/dbus.c:123
#9  0x000000000045e11b in connman_dbus_dict_append_basic (dict=0x7fffffffdea0, key=
    0x479539 "Name", type=115, val=0x7fffffffde58) at ./include/connman/dbus.h:112
#10 0x000000000045e8ab in append_notify (dict=0x7fffffffdea0, session=0x6d4580)
    at src/session.c:375
#11 0x000000000045ecf6 in session_notify (user_data=0x6d4580) at src/session.c:498
#12 0x000000000045f885 in session_changed (session=0x6d4580, trigger=
    CONNMAN_SESSION_TRIGGER_SERVICE) at src/session.c:954
#13 0x0000000000460f66 in service_add (service=0x6b4880, name=0x0) at src/session.c:1602
#14 0x0000000000447946 in __connman_notifier_service_add (service=0x6b4880, name=0x0)
    at src/notifier.c:413
#15 0x0000000000442fa1 in __connman_service_create_from_network (network=0x6b4af0)
    at src/service.c:4921
#16 0x00000000004332e4 in network_probe (network=0x6b4af0) at src/network.c:167
#17 0x0000000000433d32 in connman_network_set_group (network=0x6b4af0, group=
    0x699ae0 "hidden_managed_psk") at src/network.c:544
#18 0x000000000041c4a5 in network_added (supplicant_network=0x69cab0) at plugins/wifi.c:856
#19 0x000000000041d39c in callback_network_added (network=0x69cab0)
    at gsupplicant/supplicant.c:396
#20 0x000000000041e878 in add_bss_to_network (bss=0x6a68c0) at gsupplicant/supplicant.c:1023
#21 0x000000000041efa1 in bss_property (key=0x0, iter=0x0, user_data=0x6a68c0)
    at gsupplicant/supplicant.c:1205
#22 0x000000000041f5f9 in interface_bss_added_with_keys (iter=0x7fffffffe300, user_data=
    0x6a70a0) at gsupplicant/supplicant.c:1354
#23 0x0000000000420689 in signal_bss_added (path=
    0x6b7818 "/fi/w1/wpa_supplicant1/Interfaces/39", iter=0x7fffffffe300)
    at gsupplicant/supplicant.c:1783
#24 0x0000000000420cf9 in g_supplicant_filter (conn=0x699770, message=0x6a6b20, data=0x0)
    at gsupplicant/supplicant.c:1998
#25 0x0000003a5ee0f926 in dbus_connection_dispatch (connection=0x699770)
    at dbus-connection.c:4647
#26 0x000000000040c4ab in message_dispatch (data=0x699770) at gdbus/mainloop.c:80
#27 0x0000003edf64410b in g_timeout_dispatch (source=<optimized out>,
    callback=<optimized out>, user_data=<optimized out>) at gmain.c:3895
#28 0x0000003edf6427ed in g_main_dispatch (context=0x698100) at gmain.c:2441
#29 g_main_context_dispatch (context=0x698100) at gmain.c:3014
#30 0x0000003edf642fc8 in g_main_context_iterate (context=0x698100, block=<optimized out>,
    dispatch=1, self=<optimized out>) at gmain.c:3092
#31 0x0000003edf64360d in g_main_loop_run (loop=0x6980c0) at gmain.c:3300
#32 0x000000000042eab6 in main (argc=1, argv=0x7fffffffe6b8) at src/main.c:381
(gdb) f 10
#10 0x000000000045e8ab in append_notify (dict=0x7fffffffdea0, session=0x6d4580)
    at src/session.c:375
375                     connman_dbus_dict_append_basic(dict, "Name",
(gdb) l
370                             ifname = info->entry->ifname;
371                             service = info->entry->service;
372                             bearer = info->entry->bearer;
373                     }
374
375                     connman_dbus_dict_append_basic(dict, "Name",
376                                                     DBUS_TYPE_STRING,
377                                                     &name);
378
379                     connman_dbus_dict_append_dict(dict, "IPv4",

session: Free ifname

connman_inet_ifname() returns strdup() string.

session: Insert service_entry into service_list

When an element is removed from the list, the element will be passed
to destroy_service_entry() which expects only elements of type struct
service_entry.

notify: Add service name parameter to service_add()

session: Fix invalid session_hash in cleanup path

session: Service changed cleanup

session: Track reason on a session level

We need to track the Connect()/Disconnect() on a global session level
and not only in the info. This is needed when we were not able to
select and connect to a matching service in the first place. It might
happen that a valid service is added in time and then we want to use
it.

session: Disconnect on destroy if needed

session: Move code

release_session(), session_disconnect(), owner_disconnect() and
destroy_session() have been moved below session_changed. This is
a preperation step for the next patch

session: Split test_and_disconnect()

Do split the testing into its own function and also reset the entry
member back before we call __connman_service_disconnect().

This is necessary because calling __connman_serivice_disconnect() we can
end up beeing called session_changed() before we have updated the entry.

session: Change is_connected to is_online

session: Handle configure state

We should ignore service state (ASSOCIATION, CONFIGURATION) updates
for connecting services (CONNMAN_SESSION_REASON_CONNECT). info->online
will still be false for those services. So don't set the info->reason
CONNMAN_SESSION_REASAON_FREE_RIDE.

service: Extend debug message for session refcounting

unit: Fix connect disconnect test

unit: Add manager state callback

unit: Add PropertyChanged watch for Manager

unit: Use D-Bus watch for testing ConnMan running

wifi: Move the wifi helper function to service.c and remove wifi.c.

service: Create hidden networks with a proper group identifier

hidden_* group identifiers do not match the probe response built ones,
eventually preventing users from being able to reconnect to it.

dhcp: Try to reuse the IP address we had last time.

This fix will cause the DHCP_REQUESTED_IP option to be sent to
the server.

Fixes BMC #21068

ipconfig: Remember last DHCP IP address.

ofono: Improve coding style

Makefile: Remove test-profile from test_scripts

service: Fix calling Agent API request input function

Requesting passphrases from the user has been done only when the
ConnectService D-Bus method has been called. As user input may be
needed every time a service is connected, input requesting moved to
__connman_service_connect() instead. Also to note is that the same
service can be retried after a failed connection attempt, for those
cases an already associated service->network needs to be disconnected
before an new connection is attempted.

service: When indicating error, also indicate IPv6 state as failed

Set IPv6 ipconfig state to failure in __connman_service_indicate_error().
The return value of the function isn't used for any practical purposes,
therefore always return 0.

service: Correctly set 8021X, WEP and PSK passphrases

When returning a passphrase from the Agent API RequestInput method, check
the service security type and set the passphrase accordingly. For WEP and
PSK use __connman_service_set_passphrase() and for 8021X use
__connman_service_set_agent_passphrase(). Print a debug message if the
service security type is none of the three above.

TODO: EAP-GTC support done, EAP-FAST not making progress

test: Remove test-profile

The profile API being gone for a long time now, test-profile is
irrelevant.

wifi: network_remove: Clear network pointer in wifi

When a network is destroyed we also have to remove the pointer
from the wifi data structure.

connmand[25303]: plugins/wifi.c:network_remove() network 0x69c350
connmand[25303]: src/network.c:connman_network_unref() network 0x69c350 name bmwnet02 refcount 1
connmand[25303]: src/network.c:connman_network_unref() network 0x69c350 name bmwnet02 refcount 0
connmand[25303]: src/network.c:network_destruct() network 0x69c350 name XXXXX

[...]

connmand[25303]: plugins/wifi.c:interface_state() wifi 0x699d60 interface state 1
connmand[25303]: plugins/wifi.c:is_idle() state 9
connmand[25303]: src/network.c:connman_network_set_associating() network 0x69c350 associating 0
connmand[25303]: src/network.c:connman_network_set_connected() network 0x69c350 connected 0

Fixes BMC#22307

service: Remove __connman_service_reset_from_networks()

This is not needed because there is only 1 network per
service.

Fixes BMC#22078

ofono: Destroy network_hash when disconnecting

ofono: Duplicate network path value

Path value will be freed after the function call, this may lead to
can't connect/reconnect to a 3G service

device: Free network after resetting service

After g_hash_table_remove(device->networks, identifier); the previously
cached service pointer can be invalid.

Bug reported by: Daniel Wagner <daniel.wagner@bmw-carit.de>

build: Remove g_dhcp_* from list of exported symbols

wifi: Use agent provided credentials as a fallback

service: Store agent provided credentials separately

Add new fields for agent provided identity/passphrase in service struct
(agent_identity/agent_passphrase) and network ("WiFi.AgentIdentity" and
"WiFi.AgentPassphrase").

Preparing these values in prepare_8021x() is left out because the same
thing is done in request_input_cb(), which is where the values always
come from.

network: Remove connman_network_set_pac()

This routine is not used.

service: Propagate changes of proxy auto-config url over DBus

service: Enable clearing of proxy auto-config url

This could introduce a need to null-check values where this function
is used (src/dhcp.c, src/network.c, src/provider.c). However, src/dhcp.c
requires that URL can be cleared.

AUTHORS: Mention Bertrand's contributions

ofono: Network name is empty when operator is not available

When the operator is not available and the network name is left to NULL,
the service is built with the hidden flag set.
The operator name will be updated whenever oFono passes it to ConnMan.

network: Set driver pointer to NULL back if not ready

The network_probe() function will be called several times until it
creates successfuly the service object.

In the case of the oFono plugin, the connman_device_add_network() is
called after the driver has been registered to the network. Therefore
the network->driver has been set but no device yet.
__connman_service_create_from_network() returns -EINVAL because
the device is not yet set.

The next call on network_probe() will bail out on 'if (network->driver
!= NULL) return -EALREADY;' so we have to reset it in the first round.

Fixes BMC #21705

service : Fix put data on null pointer

when using ConnectService of Manager interface a segfault arises due
to a null pointer.

in __connman_service_create_and_connect:

[...]
service = lookup_by_identifier(name);

if (service != NULL)
goto done;

network = create_hidden_wifi(device, ssid, mode, security);
if (network != NULL) {
connman_network_set_group(network, group);
service->network_created = TRUE;
}
[...]
in this part "service" is null

network: Fix double free in network_destruct()

ofono: Fix network owner ship

network: Do not unref network

wifi: Fix network owner ship

bluetooth: Fix remove device/network

device: Reorder cleanup

If free_network is called before the driver->disable() callback
the device has been removed from the networking structure.

bluetooth: Unref network objects

The bluetooth plugin creates the network object therefore
it should also unref (destroy) it.

device: Change signature of connman_device_remove_network()

ethernet: Take ownership of network objects

Therefore destroy network objects when the carrier goes offline.

device: Remove needless call to __connman_network_set_device()

network: Remove connman_network_register/unregister()

Since the service object doesn't take a reference the
intermediate step of having a connman_network_register/unregister
is not needed. This function are only there to have an additional
step where probe()/remove() will be called. This is the job of
connman_device_add_network()/connman_device_remove_network().

service: Don't ref/unref network objects

which were created by the plugins. The hidden wifi
networks will be created by service and therefore
service is the owner of the network objects and
does the unref when the service is destroyed.

service: Update network pointer in service

device: Fix a inbalance of ref/unref of network objects

The destroy callback of the hash table (device->networks)
will unref and unregister the network object.

service: Fix memory leak

valgrind reports that __connman_service_nameservers_remove leaks
memory.

g_strv_length already returns the right size of the new array.

dhcp: Fix memory leak

Reported by valgrind.

dhcp: Fix cleanup

__connman_dhcp_cleanup() is called early in the shutdown process
(before device or network).

network: Stop DHCP

If DHCP has been started for the IPv4 IP configuration we should also
stop it later.

The only place where it has been done so far was in
__connman_network_clear_ipconfig() which is called from
service.c:set_ipconfig(). But this will only be called if the user
changes service settings.

gweb: Fix gnutls_priority_set_direct() for various GnuTLS versions

dnsproxy: Add error print when DNS response send failed.

gsupplicant: set the cipher even if security is WEP

This patch fix an issue with WEP40, i tested it with WEP40 WEP104 and open AP.

It set the group cipher even if security is WEP otherwise connman won't connect to WEP40 service.

iptables: Work around API breakage introduced in iptables 1.4.11

The xtables_merge_options() paramter list got extended without version bump.
These clumsy #ifdef's make it possible to work with newer and older iptables
libraries. It got inspired from an iproute2 patch:
https://bugs.gentoo.org/attachment.cgi?id=276401

Once all distributions start to carry 1.4.11 it can be considered to bump the
minimum version for connman and remove these ifdef's.

Release 0.76