session: Handle bearers update correctly

We cannot create a new service_list in populate_service_list
while we still hold pointers to entries (session->entry).
deselect_and_disconnect() would then operate on bogus pointers
leading to memory corruption

Fixes BMC#23325

session: Rename update_allowed_bearers()

session: Move update_allowed_bearers() up

technology: Register a rfkilled technology in technology_add_rfkill

If a technology was rfkilled before connman ran, we need to register
it. If such tech was enabled, technology->device_list will be empty.
We save the state, rfkill unblock the type and exit. When the devices
get unblocked, they are automatically enabled since we saved the enable
state.

tools: Fixed double-free error possibility into iptables-test

iptables: Copyright year update

iptables: Tiny indentation fix

iptables: Adding capability to load necessary modules

iptables: Rule deletion capability added

iptables: Factorization of target reference update

iptables: Refactoring how jumps are handled

iptables: Fixing target reference update

iptables: Renaming add into append

iptables: Adding capability to insert a rule

iptables: Code factorization for rule inclusion

iptables: Fix memory leak

iptables: Small indentation fix

iptables: Adding capability to delete user-defined chains

iptables: Added helper function to remove table entry

iptables: find_chain_head/find_chain_tail code factorization

provider: Remove provider if there is an error when connecting

If vpn service cannot be started for some reason, we must
remove the provider so that user can reconnect to it later.

provider: Fix memory leak

Provider name is already allocated in connman_provider_get() so we
must deallocate it before setting it again.

Use 'cellular' instead of '3g' as bearer name

Fixes BMC#23324

session: Move info->online update code to session_changed()

service_state_changed() should only update the service
states. session_changed() takes care of all session settings.

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.