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

build: Use color output for unit tests

build: Add forgotten binaries to .gitignore

build: Remove wifi-scan testing utility

service: wpad only run on ipv4

service: clear ipconfig data and operations on free

Ensure ipconfig data and operations are cleared out when a service is
freed.

This is at least a partial fix and maybe a full fix for issues 21000
<http://bugs.meego.com/show_bug.cgi?id=21000> and 21285
<http://bugs.meego.com/show_bug.cgi?id=21285>.

build: Remove no longer needed build magic

build: Use BUILT_SOURCES for generating header files

build: Check for signalfd support

loopback: Recover Loopback on Setup when Up but No Address

It is possible to end up in situations in which the loopback interface
is up but has no valid address. In that case, we expect SIOCGIFADDR
will return EADDRNOTAVAIL and valid_loopback should return FALSE such
that connman can add a loopback address back to the interface.

This is a partial fix for bug 21001
<https://bugs.meego.com/show_bug.cgi?id=21001>.

loopback: Improve Error Reporting when Recovering Loopback

When fixing up the loopback address or netmask, log what is being
fixed up.

loopback: Use Existing Loopback Globals Rather than Temporaries

Use preassigned globals for the loopback address and netmask rather
than creating and assigning additional temporaries.

loopback: Return a More Descriptive Error Code

Return a more descriptive -errno rather than -1 when we fail to create
a socket.

service: Do not iterate over NULL service list

network: Remove unused function

service: Fix whitespace damage

provider: Fix ident memory leak if provider cannot be created

service: Cleanup fix

When the system shuts downn, we destroy the service_list.  The destroy
callback on the service_list sequence is service_free() which will
call the provider code. Provider wants to iterater over the
service_list we are currently freeing. This results into a crash.

#0  0x0000003edd680991 in __strlen_sse2 () from /lib64/libc.so.6
#1  0x0000003ee0a280fb in ?? () from /lib64/libdbus-1.so.3
#2  0x0000003ee0a15f19 in ?? () from /lib64/libdbus-1.so.3
#3  0x0000003ee0a19c0a in dbus_message_iter_append_basic () from /lib64/libdbus-1.so.3
#4  0x000000000043985b in append_path (value=0x6a7800, user_data=0x7fffffffe350)
    at src/service.c:121
#5  0x0000003edf65af07 in g_sequence_foreach_range () from /lib64/libglib-2.0.so.0
#6  0x0000000000439888 in __connman_service_list (iter=0x7fffffffe350, user_data=0x0)
    at src/service.c:127
#7  0x000000000045971e in connman_dbus_property_append_array (iter=0x7fffffffe440, key=
    0x472cd8 "Services", type=111, function=0x439860 <__connman_service_list>, user_data=0x0)
    at src/dbus.c:204
#8  0x000000000045990a in connman_dbus_property_changed_array (path=0x472ca0 "/", interface=
    0x472c8c "net.connman.Manager", key=0x472cd8 "Services", type=111, function=
    0x439860 <__connman_service_list>, user_data=0x0) at src/dbus.c:271
#9  0x00000000004391cd in services_changed (user_data=0x0) at src/profile.c:120
#10 0x000000000043924c in __connman_profile_changed (delayed=0) at src/profile.c:143
#11 0x000000000043f467 in service_free (user_data=0x6aab40) at src/service.c:2951
#12 0x0000003edf65aac8 in ?? () from /lib64/libglib-2.0.so.0
#13 0x0000003edf65aaa2 in ?? () from /lib64/libglib-2.0.so.0
#14 0x0000003edf65ae39 in g_sequence_free () from /lib64/libglib-2.0.so.0
#15 0x000000000044459f in __connman_service_cleanup () at src/service.c:5432
#16 0x000000000042ef5f in main (argc=1, argv=0x7fffffffe6b8) at src/main.c:412

provider: Untangle create and connect code block

provider: Remove pointless jump to error label

This fixes a compiler warning for an uninitialized variable. And this
warning is a real error.

  CC     src/provider.o
src/provider.c: In function ‘__connman_provider_create_and_connect’:
src/provider.c:476: warning: ‘provider’ may be used uninitialized in this function

network: Set network driver before creating the service

The autoconnect machine can potentially start from
__connman_service_create_from_network() so it needs a network driver.

service: disconnect state has priority over online one.

service: preventing to call ready state more than once.

timezone: Remove /etc/localtime if it's a symbolic link

And copy the timezone file over.

Fixes BMC #18887

device: Ignore dummy networking devices

When a kernel comes with dummy network devices support enabled, the
dummy interface will take the default route whenever a real interface
disconnects.
This is not what we want as the default route won't be released then.

network: Fix temp pointer memory leak

network: Let the gateway code handle the READY transition

For IPv4 (DHCP, fixed or manual), and for manual IPv6, the gateway handling
code (connection.c) is the one responsible for moving to the READY state.

ipconfig: Don't call IP bound ops when type does not match

network: Autoconf IPv6 is ready when address is set

It's not ready when the technology is connected. The ASSOC->READY transition
will happen through the IP bound ops.

service: Separate ipconfig state and service state handling

service: Use __connman_service_ipconfig_indicate_state()