session: Remove unnecessary deselect_service

Because in session_changed() we take care of this now.

session: Simplefy session_changed()

Since we are tracking the reason code now also in
the deslect_and_disconnect() function we can rely
on the correct handling there. Also no need to parse the
reason code again.

session: Add reason code to deselect_and_disconnect()

session: Add debug messages

session: Add CONNMAN_SESSION_REASON_DISCONNECT

Maybe the reason code should be renamed to state

session: Pass session_info pointer into functions

session: Remove deselect_previous_service()

The deselect_and_disconnect() function should be called
from session_changed() and not in the select_and_connect()
code path. All disconnect and connect decision are
initiate from session_changed(). This makes the code flow more
symetric.

Note that we break here a Online -> Offline transision, we are
going to fix this later again.

session: Rename test_and_disconnect

To deselect_and_disconnect in order to match more to the
select_and_connect function.

session: Track info not the session for ecall

This is a prepartion step for the next few patches.
Instead of passing the session pointer into functions
we are going to pass in only the session_info pointer.

session: Disconnect free ride session

In case where we get offline

Fixes BMC#22837

session: Refactor select_service

Split the online/offline code path with moving the corresponding
code into two new functions.

session: Fix null pointer derefencing

When an emergency session sets back the EmergencyCall
property, all other session are notified. If a session
previously was not connected (online == FALSE) no
entry (services) was selected. We should not derefency
a NULL pointer in this case.

Fixes BMC#22880

service: Don't iterate over changing sequence

__connman_service_disconnect_all() should not iterate over
the service_list directly because the ordering of the
services can change when disconnecting a service (e.g. default
service). The iterator might be invalid after a
__connman_service_disconnect() call.

It's better to iterate over a temporary list.

todo: Added details to personnal firewall task

tools: Copyright year update in iptables-test

tools: Tiny indentation fix in iptables-test

tools: Adding capability to load necessary modules into iptables-test

tools: Rule deletion capability added to iptables-test

tools: Factorization of target reference update in iptables-test

tools: Refactoring how jumps are handled in iptables-test

tools: Fixing target reference update in iptables-test

tools: Renaming add into append into iptables-test

tools: Adding capability to insert a rule in iptables-test

tools: Code factorization for rule inclusion in iptables-test

tools: Fix memory leak in iptables-test.c

tools: in, out, source and destination support in iptables-test

tools: Small indentation fix in iptables-test

tools: iptables-test delete a chain if only it is already flushed.

tools: Added helper function to remove table entry in iptables-test

tools: find_chain_head/find_chain_tail code factorization in iptables-test

main: Cleanup ipconfig after service

Since service.c depends on ipconfig.c, __connman_service_cleanup()
should be called before __connman_ipconfig_cleanup().

connmand[4787]: src/ipconfig.c:__connman_ipconfig_cleanup()
connmand[4787]: em1 {remove} index 2
connmand[4787]: wlan0 {remove} index 3
connmand[4787]: src/ipconfig.c:connman_ipconfig_unref() ipconfig 0x6b0460 refcount 1
connmand[4787]: src/ipconfig.c:connman_ipconfig_unref() ipconfig 0x6b04c0 refcount 1
connmand[4787]: hso0 {remove} index 4
connmand[4787]: src/service.c:__connman_service_cleanup()
connmand[4787]: src/service.c:service_free() service 0x6ae690
connmand[4787]: Service state machine inconsistency detected.
connmand[4787]: src/service.c:stats_stop() service 0x6ae690
connmand[4787]: src/service.c:services_changed()
connmand[4787]: src/ipconfig.c:connman_ipconfig_unref() ipconfig 0x6b0460 refcount 0
connmand[4787]: src/ipconfig.c:__connman_ipconfig_disable() ipconfig 0x6b0460

Breakpoint 1, g_log (log_domain=0x34b78adc06 "GLib", log_level=G_LOG_LEVEL_CRITICAL, format=
    0x34b78b6710 "%s: assertion `%s' failed") at gmessages.c:573
573     {
(gdb) bt
#0  g_log (log_domain=0x34b78adc06 "GLib", log_level=G_LOG_LEVEL_CRITICAL, format=
    0x34b78b6710 "%s: assertion `%s' failed") at gmessages.c:573
#1  0x00000034b7831d35 in g_hash_table_lookup (hash_table=0x0, key=0x3) at ghash.c:897
#2  0x000000000044e9a4 in __connman_ipconfig_disable (ipconfig=0x6b0460) at src/ipconfig.c:1678
#3  0x000000000044de9d in connman_ipconfig_unref (ipconfig=0x6b0460) at src/ipconfig.c:1279
#4  0x0000000000440750 in service_free (user_data=0x6ae690) at src/service.c:3416
#5  0x00000034b785aac8 in real_node_free (node=0x6a2ed0, seq=0x6a3de0) at gsequence.c:1755
#6  0x00000034b785aaa2 in real_node_free (node=0x6a2a70, seq=0x6a3de0) at gsequence.c:1751
#7  0x00000034b785ae39 in g_sequence_free (seq=0x6a3de0) at gsequence.c:283
#8  0x0000000000444b53 in __connman_service_cleanup () at src/service.c:5617
#9  0x000000000042fb75 in main (argc=1, argv=0x7fffffffe678) at src/main.c:401

ntp: Add skeleton for Simple NTP support

AUTHORS: Mention Jeff's contributions

session: Only disconnect online sevices

Fixes BMC#22827

manager: Only defer result when enabling session mode

When leaving the session mode we might be still online
so we should not wait for entering the idle state.

session: Only select new service if not in progress

session: Refactor select_and_connect()

Make this function more readable.

test-session: Add multiple session support

Add an new parameter session_name in test-session so all session
operations are based on app_path and session_name

wispr: Add magic "wispr" keyword to user agent

Without the "wispr" keyword in the user agent string, some WISPr 1.0
hotspots (like T-Mobile and Telekom) are not responding with the
proper WISPr XML properties.

wispr: Use different status requests for IPv4 and IPv6

build: Add OpenVPN script to ignore list

6to4: Use ConnMan status URL for checking connectivity

wifi: Add SSIDs and frequencies to wpa_supplicant scan for fast scan

Lot of fixes by Jukka Rissanen <jukka.rissanen@linux.intel.com>

gsupplicant: Get the number of scan ssids from supplicant

gsupplicant: Add support to append char ** entries to dbus dict

wifi: Add support to multi scan type

Allow multi scan type for fast connect.

Scanning request removed from interface_added() because it is already
called in connman_device_set_powered() so no need to call it again
immediately (fix by Jukka Rissanen <jukka.rissanen@linux.intel.com>)

service: Add frequency support to service

Get the frequncy of gsupplicant network on connman network
creation, and always save the frequency of the wifi service.

storage: Export service loading function

storage: Add services getter

bootstrap: Remove portal configuration

ofono: Remove 3G network after disabled 3G modem

Fixes BMC#22745

TODO: Removed wispr task

test: Added Username/Password input support in simple-agent

wispr: Requesting login input through the agent

agent: Added Username/Password request input logic

wispr: Handle WISPr messages and apply relevant states

wispr: Added WISPr XML content parser

wispr: Added portal web request and result handling logic

wispr: Proxy lookup and web context setup

wispr: Adding wispr logic entries in service and wpad

wispr: Adding structures and start/stop functions

location: Deleted location support and portal plugin

wifi: Device reference counting fix on scan

When starting to scan, up the reference count on struct connman_device
and release it in the scan callback.

wifi: Add NULL checks in interface create and remove

Interface create and remove callbacks can be called with wifi user data
set to NULL, add checks for this.

wifi: Implement network_changed gsupplicant hook

gsupplicant: Implement network properties changes notification

This is only used for propagatong signal changes for now.

gsupplicant: Track best signal and the corresponding BSS

device: Remove device_probe and device_remove

They're tiny and useless wrappers.

technology: Disable tethering only when no technology is tethering

When notifying that a technology is not tethering anymore, go
through the list of technologies and disable tethering only if
no other technology is tethering.

device: Set PENDING_DISABLE on device disable

Set powered pending state to PENDING_DISABLE when the device is
being disabled.

device: Simplify device_pending_reset()

The g_source for device_pending_reset() is going to be removed when
the callback function returns FALSE. Thus simplify the code with a
few lines.

technology: When disabling technology, disble tethering too

technology: Move bridege checking to set_tethering()

Move bridge checking into set_tethering(), as the bridge
information is not used elsewhere.

dnsproxy: Use listener data instead of hash table lookups

Pass around struct listener_data instead of always looking it up
via interface from hash table. There are no changes for exported
functions.

dnsproxy: Fix __connman_dnsproxy_add_listener() error handling

Do not add dnsproxy listener data to hash table if we fail to
create either a udp or tcp listener.

service: Filter Adding WPS to the Security Array

Some access points incorrectly advertise WPS even when they are
configured as open (i.e. no security), so filter adding WPS to the
security array appropriately.

timezone: Do not access already released memory

closedir() releases the filename but we were accessing it later.

main: Memory leak in option_debug

gsupplicant: Do not access IE array past end of buffer

IE list was traversed past buffer limit in the last round.

test: Fix dbus path in service-move-before script

Because of patch f8ef8b2a7f7ef1bd9e26df6cfb19a838e6b10b17 that removed
profile support, the path needs to be changed.

gresolv: Fix buffer size

Make buf just big enough and initialize it in order to get
rid of valgrind error (accessing uninitialized memory).

service: Behave properly when RequestInput is cancelled

Do the same actions on cancelling RequestInput as when
not retrying a service on ReportError.

Re-fixes BMC#22542.

service: Do not allow service to be moved before a VPN service

This is needed as the default routes are not set properly
if we allow a service to be moved before a VPN service.

stats: Move stat files to respective service identifier directory

storage: Switch to <service_id> directories

Service settings would now reside in /var/lib/connman/service_id/settings.
we fallback on /var/lib/connman/default.profile for a smooth transition.

storage: Switch to settings file

All the global settings would reside in /var/lib/connman/settings.
We also migrate global keys from /var/lib/connman/default.profile
to /var/lib/connman/settings for a smooth transition.

service: Add NULL check

network: Add NULL check

connection: Service was not ref counted properly

The reference counting problems were clearly seen with VPN service.

provider: Return existing service path if already connected

provider: Add ref counting debug

service: Set the service state to IDLE if method is off or unknown

service: Fix state combining

If we have IPv6 state as DISCONNECT and IPv4 state as
ONLINE or READY then we are not disconnected.

service: Change the debug print location to be more useful

service: Fix function name as it is static

service: Set the user connect status correctly for VPN

If the userconnect is not set, then informatation about authentication
errors cannot be reported by agent API.

network: Change the service disconnect state when necessary

If we are in idle or failure states, then we should not change to
disconnect state when the service is disconnected. This is because we
were not connected in the first place.

service: Add function to get the ipconfig state

vpn: Set authentication failure error code properly in provider

This patch is needed so that we can catch the authentication
error from vpn driver and inform the failure to user.

provider: NULL pointer check