tools: Add O_CLOEXEC to open()

Make sure all file descriptor are closed.

service: Activate loose mode routing

If more than one service is connected at the same time,
then activate loose mode routing by setting the
/proc/sys/net/ipv4/conf/all/rp_filter to value 2
If the loose mode routing is not activated, then packets
are not routed properly if services are connected to same
subnet.

The original value of rp_filter is restored when the other
services are disconnected and only one service is connected.

For details of rp_filter setting, see Linux kernel file
Documentation/networking/ip-sysctl.txt

Fixes BMC#23606

ipconfig: Set/unset IPv4 rp_filter setting

network: Remove obsolete service lookup in disconnect

service: Check NULL pointer

session: Do not select ongoing service again

In case Session.Connect() is called several times while
the selected services is still going from the IDLE state
to ONLINE state (or error state), do not call
select_and_connect() again. This leads to an inconsistent
ref counting on the service.

Fixes BMC#23630

wispr: Reference count service properly according to contexts

wispr: wispr_start will always re-start a new context

service: wispr fallback when wpad fails

wpad: Reference count service properly

dhcp: Reference count network properly

inet: Cleanup router solicitation callback properly after error

The cleanup routine must be called after an error and after
calling RS callback, otherwise the timeout is not removed
in right time.

notifier: Don't unnecessarily report "online" state on disconnect

When a service disconnects, don't send an extra 'online'
net.connman.Manager.StateChanged signal if another service is still
connected.

Fixes BMC#23421

wispr: Set LoginRequired when wispr login is requested

Setting it while handling redirection is not bullet proof in case
of AP which does not implement well wispr and sends wispr content
with no redirection at first place.

Fixes BMC#23424

dnsproxy: Fix can't resolv hostname

When we resolv hostname, we will append the domain name then we may
get question section mismatch error. After we get the DNS reply
we can remove the domain name again. This works well with one host
with only a A record, it may failed when the reply with many cnames
or additional records due to message compression error. But in a
local network we only resolv hostname will not meet these issues.

Modified by Julien Massot according to Samuel's comments.

service: Update service order when changing the favorite flag

If the service order is not changed at the same time as the favorite
status, the service may stay on top of the service list despite not
being a favorite anymore.

Patch based on an initial fix from Danny Jeongseok Seo <s.seo@samsung.com>.

Makefile: Add missed test files to test_scripts

Fixes BMC#23418

service: Load ipconfig according to service identifier

We have seperate the storage file to different directories according
to service identifier, then we need to load it according to service
identifier too.

Fixes BMC#23475

service: Call __connman_service_disconnect from remove_service

Just calling __connman_network_disconnect() means missing proxy and
ipconfig clearance.

ipconfig: Show IPv4.Configuration properties for FIXED

Show IPv4.Configuration properties also for
CONNMAN_IPCONFIG_METHOD_FIXED configuration method.

Fixes BMC#23444

configure: Print errors if VPN binaries are not found

When a VPN has been enabled and its binary is not given
with '--with-<VPN>=PROGRAM', make an additional check
whether AC_PATH_PROG found the binary in question.

log: Add support for backtrace symbole resolving

build: Link session unit test with dynamic loader library

technology: remove the pending_timeout source with g_source_remove

device: Disconnect a connected network before disabling the device

When disabling a device, disconnect the device's connected network,
if there are any.

Fixes BCM#23165.

AUTHORS: Mention Danny's contributions

device: Remove connections counter

The device connection counter is only used by the post scan auto connection
logic, and the connection status is handled by the autoconnection service
routine itself (__connman_service_auto_connect()).

device: Remove redundant connection numbers check

__connman_service_auto_connect() already checks for service state, so there
is no need to look at the device being connected or not.

service: Add interface name matching in __connman_service_lookup().

Fixes BMC#23419

AUTHORS: Mention Philippe's contributions

ofono: Create the cellular network only when attached to the PS network

3G auto-connection may fail as the cellular network service may not be
ready to connect.
With this patch, the cellular network service should be exposed only after
PS network attachment is done.

wispr: Setting the relevant service property when a login is required

When a wispr redirect occurs we set login required to the service

Fixes BMC#23424

session: Set dirty flag in de/select service function

session: Remove double assignment

The dirty flag is already cleared in append_notify()

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