test-iptables: Add fallthrough unit test

A fallthrough rule is a one which does not have a verdict, e.g.
ACCEPT, DROP etc.

iptables: Remove unused __connman_iptables_command()

nat: Use new iptables API

test-nat: Move tests to test-iptables

These are iptables related tests. So let's move them over.

test-iptables: Add unit test for the new API

iptables: Add split out iptables commands

Instead of having a pure string based API, we add two new
main functions, __connman_iptables_append() and
__connman_iptables_remove(). The missing commands will be added later.

To simplify the whole code, the __connman_iptables_command() code
is refactored into smaller pieces: parse_rule_spec() calls a few
functions such as clear_tables_flags() and parse_xt_modules()
which should make the reading of the main parser loop simpler.

Also added a few comments on the parser which is really tricky.

nat: No need to 'nat' table anymore

This is done in iptables.c directly.

iptables: Flush 'filter' 'mangle' and 'nat' table

The implementation is ugly but there is not much we can do about it, the
iptables API is being just plain stupid here.

iptables: Add CONNMAN_IPTABLES_DEBUG environment variable

If CONNMAN_IPTABLES_DEBUG is set, then print the table when it is loaded
and print it the table which will be written.

Also use DBG() instead of connman_info().

iptables: Remove dependency on table in iterator_entries_cb_t

With removing the table argument the callback can now either
iterate over buffer we get from the kernel or the one we write
to the kernel.

iptables: Do not pass table into dump_match() and dump_target()

iptables: Get rid of the iterator macro

Instead implement the iterator loop directly. Since both dump_entry()
and add_entry() have calculated 'builtin' and 'offset' let's pass in
them in as well.

In the next step we are able to remove also the table argument
which will allow us to unify the parsing the table we get
from IPT_SO_GET_ENTRIES and the table we will pass in to the kernel
via IPT_SO_SET_REPLACE.

iptables: Clear global parser flags

That allows the parser to be reentrantable.

iptables: Make string arguments const

iptables: Add some documentation

These are some random notes but should give the next person to debug
iptables some introduction.

test-iptables: Add unit test for iptables

gdbus: Don't call property changed callback during client init

When the client uses ObjectManager to init properties, do not call
property changed callbacks. They should only be called once the proxy
added has been successfully signaled since the proxy itself provides
a full copy of available properties.

gdhcp: DHCP server IP address is stored in host order

This change affects unicasted DHCP renew and release messages.
A DHCP server receiving packets via a raw socket will get all
packets destined to port 67. If the DHCP server checks the
intended server IP address, the request will fail. If the server
does not care about the IP address being used, it will send a
reply. The reply is not recognized by ConnMan since it is coming
from another IP address than what it was sent to. ConnMan will
retry quite a few times, but eventually settle down and use the
address even without a proper response from the server.

ipconfig: Use proper address structure when appending IPv4 info

service: Validate the IP addresses before use

When setting IP address information via D-Bus, first create a new ipconfig
structure to hold the values. If the values were validated correctly, update
the service ipconfig structure with the new values. In order to achieve
this, refactor the existing code.

Fixes BMC#25930

service: Factor out ipconfig creation

The caller has to set the created ipconfig into its data structures.

ipconfig: Add checks for invalid IPv4/IPv6 addresses

Also fine tune returned errors.

test: Fix simple-agent so it can run even if service files are missing

Fixes BMC#25964

ipconfig: Do not overwrite gateway address

If we have already received gateway address via newroute rtnl
message, then do not overwrite it when receiving newaddr message.

Fixes BMC#25931

gdbus: Fix unpublished interface signal emission

If we haven't published an interface yet (i.e. it's in the data->added
list), we should just ignore any property changed indications as the
values for the properties will anyway be part of the InterfacesAdded
signal.

Release 1.11

dundee: Set network available is needed only when scanning

ofono: Set network available is needed only when scanning

ethernet: Set network available is needed only when scanning

README: Add kernel options needed for tethering

TODO: Update TODO with Bluez 4.x removal task

bluetooth: Implement tethering for the Bluez 5 bluetooth plugin

technology: Support multiple technology drivers

As tethering is enabled on the technology level, both the bluetooth and
the bluetooth_legacy plugin need to register technology drivers for
CONNMAN_SERVICE_TYPE_BLUETOOTH. Modify the technology code to create
a list of registered technology drivers instead of a single technology
driver pointer.

bluetooth: Connect and disconnect a PAN NAP network

Implement connect and disconnect functionality for a ConnMan network.
On network creation check the connection status of the Bluez 5 Network1
object and set the network to connected when needed.

service: Must not return NULL with a synchronous GDBus method call

If NULL is returned, GDBus will call the registered function repeatedly.

bluetooth: Expose Bluez 5 org.bluez.Network1 objects as networks

Create ConnMan networks for those Bluez 5 org.bluez.Network1/Device1
objects that support PAN NAP. A network is created or removed in response
to the Network1/Device1 being created or when the Device1 object's UUID
changed.

The ConnMan network struct is added to and removed from the ConnMan
device when the device is created, enabled or disabled.

network: Allow more than one network driver register the same type

Allowing more than one network driver to register the same network type
requires the network driver to properly identify which networks belong
to it in the network driver probe function.

bluetooth_legacy: Check network in probe function

The network probed can belong to the new Bluez 5 bluetooth plugin or to
this plugin. Check if this plugin created the network and proceed
accordingly.

bluetooth: Keep track of Bluez 5 org.bluez.Network1 objects

Set up functions monitoring Bluez 5 org.bluez.Network1 objects being
added and removed. Add a GDBusProxy for the org.bluez.Device1 object
that corresponds to the Network1 object. Set up functions monitoring
Network1 'Connected' and Device1 'UUID' properties.

bluetooth: Expose Bluez 5 adapters as ConnMan devices

Set up functions monitoring Bluez 5 objects being added and removed using
GDBusProxy. When an adapter appears, create a new device for it and set up
a watch function for the adapter's 'Powered' property.

Keep the adapter 'Powered' property in sync with the device state and if
they differ reset the adapter to the device state. Also update the Bluez 5
adapter 'Powered' property when the device is enabled or disabled.

technology: Enable/disable individual devices also with rfkill

In order to keep ConnMan devices in sync with Bluz 5 adapters, the
individual devices need to be enabled/disabled also when
unblocking/blocking them with rfkill. Thus enable devices after
unblocking and disable devices before blocking with rfkill.

bluetooth: Register technology driver

bluetooth: Add new bluetooth plugin

bluetooth: Rename bluetooth plugin to bluetooth_legacy

Set the plugin priority to CONNMAN_PLUGIN_PRIORITY_LOW and the technology
priority to -10. Update strings to "bluetooth_legacy" and use ordinary
string instead of ident.

technology: Check technology driver probe function before calling

technology: Remove technology driver by pointer

Compare the technology driver pointer to the driver being unregistered
as the function is supposed to remove only the given driver.

Also check if the driver has a remove function before calling it.

technology: Remove unnecessary comparison and goto

ofono: Use ordinary string instead of ident

The string is only used for debugging purposes.

dundee: Use ordinary string instead of ident

The string is only used for debugging purposes.

network: Unset driver on failed match

gresolv: Destroy query at the relevant place when parsing the response

parse_response() will eventually call sort_and_return_results() which in turn
will call the result function. But the result function might cancel the gresolv.
At that point all queries belonging to this gresolv are destroyed. Returning back
to parse_response(), it calls again destroy_query() on an already destroyed one.
Thus leading to a crash.

Same issue with query_timeout()

Reported by Daniel Wagner

service: Notify leaving online state when downgrading it to ready

Fixes BMC#25862

ipconfig: IPv6 was enabled too early

When enabling IPv6 ipconfig, we enable kernel IPv6 support too early.
The ipconfig might get unreffed which will disable ipconfig and
thus disable kernel IPv6 support. By moving kernel IPv6 enabling
after the ipconfig unref call, we make sure that IPv6 kernel support
is properly enabled.
This is related to fix in commit d479904ecaa2bd9

gdbus: Check for valid path before handling properties

dnsproxy: DNS response answer count was incorrectly set

We set the answer count in host byte order instead of network
byte order when sending cached AAAA record. This problem is only
seen in special case when an AAAA record is generated by us when
there is IPv4 address in cache but no IPv6 address for the host.

network: Do not report any error when removing a connecting network

When connecting a network, if it's related technology is disabled, this network
will raise an error through the agent. Leading to a possible retry request from
the user, then to a crash if the user does so.
Thus, we prevent any error to be raised in such situation and silently disconnect
the connecting network instead.

network: Move up code about set_connected() and set_disconnected

service: Trigger autoconnect when AutoConnect flag is enabled

ipconfig: Do not disable IPv6 in lower up state

Disabling IPv6 in lower up causes problems in IPv6 connectivity.
The interface IPv6 status can stay in disabled state because of
races. So this patch reverts the commit 4ce90440a70abce7de537777
and lets service IPv6 state go to READY when we get an auto
configured address for the interface.

Fixes BMC#25929

service: Update notifier state when relevant while getting ready

When getting 1 service to online, and disonnecting it: Manager state
stays at 'ready'. This is due to service updating the notifier about
its state 'ready' 2 times. Once when connecting: idle -> ready, and
once when disconnecting: online -> ready.

agent: Keep track of the driver to unref user context relevantly

Fixes BMC#25884

gweb: Make sure to destroy the lookup before calling any result function

Program received signal SIGSEGV, Segmentation fault.
Backtrace:
0  0x00000000004219fe in _debug (resolv=0x75f9a0, file=0x497267 "gweb/gresolv.c",
    caller=0x497598 "destroy_lookup",
    format=0x497278 "lookup %p id %d ipv4 %p ipv6 %p") at gweb/gresolv.c:136
1  0x0000000000421ac5 in destroy_lookup (lookup=0x760e40) at gweb/gresolv.c:154
2  0x00000000004224ce in sort_and_return_results (lookup=0x760e40) at gweb/gresolv.c:520
3  0x0000000000422597 in query_timeout (user_data=0x760510) at gweb/gresolv.c:542
4  0x00007ffff7b1b26b in g_timeout_dispatch (source=0x760ea0, callback=<optimized out>,
    user_data=<optimized out>) at gmain.c:4095
5  0x00007ffff7b1a643 in g_main_dispatch (context=0x6f5110) at gmain.c:2784
6  g_main_context_dispatch (context=0x6f5110) at gmain.c:3288
7  0x00007ffff7b1a988 in g_main_context_iterate (dispatch=1, block=<optimized out>,
    context=0x6f5110, self=<optimized out>) at gmain.c:3359
8  g_main_context_iterate (context=0x6f5110, block=<optimized out>, dispatch=1,
    self=<optimized out>) at gmain.c:3296
9  0x00007ffff7b1ade5 in g_main_loop_run (loop=0x6f4fe0) at gmain.c:3553
10 0x0000000000442a5f in main (argc=1, argv=0x7fffffffdea8) at src/main.c:705

Reported by Daniel Wagner

gsupplicant: Return max SSIDs to scan given by wpa_supplicant

Although it seems wpa_supplicant can handle 4 SSIDs in version 0.7.x
and 16 starting with 0.8.x, in practise this does not work. Return the
value reported without relying on wpa_supplicant internals to handle
the situation. In order for hidden scans to work properly, the minimum
value needs to be 1.

build: Do not use deprecated AM_CONFIG_HEADER

The long-obsoleted AM_CONFIG_HEADER macro was removed in automake 1.13.
Use AC_CONFIG_HEADERS instead.

inet: Fix error handling when bridging interfaces

Functions that add and remove interfaces to and from bridges now return
an appropriate error code.

inet: Fix error handling when adding/removing routes

Functions that add and remove routes now return an appropriate error
code.  EEXIST and ESRCH are ignored when adding and removing routes
(respectively) to reduce spurious error messages in the logs.

vpn-provider: Check if providers list is NULL before using it

gdbus: Fix memory leak on properties_set()

The pointer returned by dbus_message_iter_get_signature() must be freed
with dbus_free().

Fixes this memory leak:

==1857== 16 bytes in 1 blocks are definitely lost in loss record 104 of
251
==1857==    at 0x402BF52: realloc (in
/usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==1857==    by 0x415E286: dbus_realloc (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x415E70B: ??? (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x415F17B: ??? (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x414CB33: dbus_message_iter_get_signature (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x8053239: properties_set (object.c:899)
==1857==    by 0x5FFFFF: ???
==1857==

gdbus: Check signature of property value before calling setter

TODO: Remove the storage migration task

storage: Remove storage migration code

TODO: Remove WiMAX removal task

doc: Remove WiMAX info from documentation

core: Remove WiMAX definitions from code

plugins: Remove WiMAX plugin files

build: Remove WiMAX from configuration options

gdbus: Don't include just added interfaces in GetManagedObjects

If we received a call to ObjectManager.GetManagedObject we should not
include in the response the interfaces in data->added. This is because
it's not guaranteed that those interfaces will trigger an
InterfacesAdded signal, which is the case if the interface is removed in
the same mainloop iteration.

gdbus: Simplify generated introspection

The generated introspection is not supposed to be read as is by human,
so there's no point in printing the indentation or writing more code to
use auto-close tags.

If it's desired to read the raw xml file, user can always use other
tools to transform the output such as "xmllint --format".

This also fixes a missing </property> when property is deprecated.

test: Fix simple-agent so it registers the right path

TODO: Add entry for BlueZ 5.x task

gdbus: Hold client reference during get name owner reply

gdbus: Call check_signals when sending signals with g_dbus_send_message

If message passed to g_dbus_send_message is a signal verify if it is a
valid and there really exists an interface with respective signal name.

gdbus: Check if the interface being registered is valid

This prevent registering interfaces that are empty or have all members
marked as experiemental.

gdbus: Introduce G_DBUS_PROPERTY_FLAG_EXPERIMENTAL

This flag can be used to mark properties as experimental, marked
properties are disabled by default and can be enabled by setting
G_DBUS_FLAG_ENABLE_EXPERIMENTAL using g_dbus_set_flags.

gdbus: Introduce G_DBUS_SIGNAL_FLAG_EXPERIMENTAL

This flag can be used to mark signals as experimental, marked
signals are disabled by default and can be enabled by setting
G_DBUS_FLAG_ENABLE_EXPERIMENTAL using g_dbus_set_flags.

gdbus: Introduce G_DBUS_METHOD_FLAG_EXPERIMENTAL

This flag can be used to mark methods as experimental, marked
methods are disable by default and can be enabled by setting
G_DBUS_FLAG_ENABLE_EXPERIMENTAL using g_dbus_set_flags.

gdbus: Include changed properties only once per signal

gdbus: Update properties on D-Bus client re-connections

gdbus: Add function to manually refresh properties

gdbus: Add support for proxy property change notifications

gdbus: Fix handling of client connect/disconnect signals

gdbus: Protect standalone proxy creation with client reference

gdbus: Use object manager only if callback functions are set

gdbus: Use client service name and not hardcoded org.bluez

gdbus: Add support for creating D-Bus proxies without object manager

Release 1.10

build: Fix Daniel's commits without name

build: Fix Flavio's name mapping to email address

build: Add vpn/net.connman.vpn.service to .gitignore

gdbus: Avoid using g_ptr_array_new_full convenience function

The g_ptr_array_new_full function only got introduced with GLib 2.30
and to avoid requiring a newer GLib use g_ptr_array_set_free_func instead.