AUTHORS: Mention Ross' contributions

doc: Add coding-style.txt

This is a copy of the oFono coding-style document, with oFono instances
replaced with ConnMan ones.
It's also in sync with the neard one.

storage: check that the string isn't empty before splitting

If the string was non-NULL but empty (str="\0"), the following \0 assignment
would write to str[-1] and thus cause memory corruption.

On PPC and MIPS, this was causing crashes in glibc.

device: Do not set regdom when device is not powered

Fixes crash bug:
connmand[6761]: plugins/ofono.c:get_properties_reply() /huawei_0 path /huawei_0 org.ofono.NetworkRegistration
connmand[6761]: plugins/ofono.c:netreg_properties_reply() /huawei_0
connmand[6761]: plugins/ofono.c:netreg_update_regdom() /huawei_0 MobileContryCode 460
connmand[6761]: src/device.c:connman_device_ref_debug() 0x8686ac0 ref 3 by plugins/wifi.c:991:wifi_set_regdom()
connmand[6761]: Aborting (signal 11) [src/connmand]
connmand[6761]: ++++++++ backtrace ++++++++
connmand[6761]: #0  0xb7749400 in
connmand[6761]: #1  0x806aa0b in g_supplicant_interface_set_country() at nat.c:0
connmand[6761]: #2  0x8062297 in wifi_set_regdom() at wifi.c:0
connmand[6761]: #3  0x80a607d in connman_technology_set_regdom() at ??:0
connmand[6761]: #4  0x806fefb in netreg_update_regdom() at ofono.c:0
connmand[6761]: #5  0x8071f3b in netreg_properties_reply() at ofono.c:0
connmand[6761]: #6  0x806f747 in get_properties_reply() at ofono.c:0
connmand[6761]: #7  0x4a374b62 in /usr/lib/libdbus-1.so.3
connmand[6761]: #8  0x4a3602b7 in /usr/lib/libdbus-1.so.3
connmand[6761]: #9  0x4a3639f5 in /usr/lib/libdbus-1.so.3
connmand[6761]: #10 0x8054250 in message_dispatch() at mainloop.c:0
connmand[6761]: #11 0x49f3e9d7 in /lib/libglib-2.0.so.0
connmand[6761]: #12 0x49f3de38 in /lib/libglib-2.0.so.0
connmand[6761]: #13 0x49f3e13e in /lib/libglib-2.0.so.0
connmand[6761]: #14 0x49f3e5a6 in /lib/libglib-2.0.so.0
connmand[6761]: #15 0x8053ab6 in main() at nat.c:0
connmand[6761]: #16 0x49cfc4c4 in /lib/libc.so.6
connmand[6761]: +++++++++++++++++++++++++++

gdhcp: Set client state to INIT_SELECTING on restart

With the previous behavior, we would end up sending a discover message
and then ignoring the reply due to the client state mismatch, so a
restart would never result in successful acquisition of a lease.

RFC 2131 specifies that the client should return to INIT state in
various restart scenarios.  However, also note that it would not make
sense in any case to send a discover message and ignore the reply.

rtnl: Support PPP connections

The ARP header type for PPP connections is ARPHRD_PPP (512).

gresolv: Use predefined mnemonics rather than magic numbers

Use predefined mnemonics from arpa/nameserv.h for the return value from
ns_msg_getflag rather than magic numbers.

gresolv: Do not update successful status with unsuccessful one

When performing a resolver lookup from timeserver or wpad, both perform
queries with an unspecified address family. This means that both A and
AAAA record queries are issued. In cases where a valid, successful A
response comes back but where the AAAA query results in a timeout, do
not smash the successful A status with the time out AAAA status;
otherwise, the timeserver or wpad will appear to fail to them when, in
fact, the A query was successful and more than satisfies its unspecified
address family requirement.

Partial fix for BMC#25486.

gresolve: Fix a typo in conditional check for returning results

In both parse_response and query_timeout there exists logic that checks
to ensure that both an A and AAAA lookups have either been responded to
or timed out before processing and returning results to the caller. In
query_timeout, there was a typo in the condition check such that it did
not match those conditions tested in parse_response.

ntp: Add debug message when starting NTP against resolved server

This patch adds an analagous debug message when starting NTP against a
resolved server as when starting against a numbered server.

ntp: Fixed a debug message typo

ntp: Retry a given server address multiple times before falling back

This patch causes the internal NTP client to retry a given server
address up to three times (four total attempts) at a two second interval
before falling back to another server name or server address.

ntp: Use mnemonic when specifying the request precision

This patch uses a mnemonic to specify the request precision and also
adds a comment how a future implementation might pass along the actual
system wall clock precision.

ntp: Add mnemonics for common clock precisions

ntp: Use mnemonics when checking reply flags

Use mnemonics when checking reply flags and print out information in
debug mode about why the reply flags were rejected.

ntp: Specify leap-not-in-sync in flags

This patch sets the non-in-sync flag of the leap subfield of the packet
flags field. This reflects behavior implemented by ntpdate or ntpd in
one-shot mode.

ntp: Send a transmit time value that reflects current system time

This sets the ntp request packet transmit time to the current system
time to reflect behavior implemented by ntpdate and ntpd in one-shot
mode.

ntp: Use mnemonics rather than magic numbers

Convert a magic number into equivalent mnemonics for the flags (leap,
version, mode) packet field.

ntp: Add mnemonics for the flags (leap, version, mode) packet field

timeserver: Try all resolved timeserver IP addresses

If we get more than one IP address, then add those
server addresses to the time server list so that they
can be tried if the first entry fails.

ntp: If the server cannot be contacted, try next one

This could happen if the network routes are not yet setup
so the sendto() could fail. In this case try next address and
hope for the best. This is a rare issue but I have seen it
couple of times.

Revert "build: Use -Wshadow compiler option"

build: Use -Wshadow compiler option

dnsproxy: Avoid multiple shadowing variables

tethering: Avoid shadowing connection variable

session: Avoid shadowing hash table iterator

session: Avoid shadowing session info variable

config: Avoid shadowing config structure variable

counter: Avoid shadowing connection variable

notifier: Avoid shadowing connected variable

agent: Avoid shadowing connection variable

service: Avoid shadowing string variable

task: Avoid shadowing connection variable

plugins: Avoid shadowing connection variable

gsupplicant: Avoid shadowing key variable

unit: Avoid shadowing session info variable

tools: Avoid shadowing buffer variable

gdhcp: Avoid shadowing address string variable

service: Gateway IP address property is not sent

The address of the IP gateway is not sent when the IP
property changes. This was because of timing issues i.e.,
the gateway was not yet set when the other IP properties
were sent.

Fixes BMC#25471

wifi: Debug print fixed

Convert a warn message to debug one because it is printed
too often to be usefull as a warning message.

wifi: Scan all hidden and provisioned AP

config: Get configurations that are provisioned

We need the list of provisioned services so that
all the hidden ones can be scanned.

config: Read hidden AP information from config file

doc: Hidden AP provisioning information added

TODO: Remove task about wpspin as PreviousPassphrase type

doc: Add necessary documentation about wpspin as a PreviousPassphrase type

agent: Handle wpspin type as a PreviousPassphrase

agent: Rewrite how PreviousPassphrase field is handled

This is a necessary rewrite to make it clearer and simpler, before implementing
wpspin type support.

device: Turn off all running interfaces at startup

Turning off running interfaces that have IP address,
causes all routes related to those interfaces to be
removed by kernel. This way connman will get a fresh
start without any extra and obsolete routes around.

inet: Return all running interfaces

Add a function that returns all (except loopback)
interfaces that are up and running (have IP address).
This information is used in following patch to clean
up the routes that are left hanging around.

service: Make sure the pending reply is not lost

The user data (pending reply) can be null if agent returns
an error. In this case do not overwrite the real pending reply.

dhcp: warn on invalid netmask

main: Dynamically allocate default blacklist array, free on exit

main: Move default string arrays top of the file

technology: check validity of given properties for tethering

A WPA2 passphrase is between 8 and 63 characters,
a SSID name is between 1 and 32 characters.

technology: return already enabled when tethering is enabled

TODO: Remove BackgroundScanning task

wifi: Add support for autoscan request

gsupplicant: Add a function to handle autoscan method

TODO: Add what wifi legacy support should be removed about set_regdom

wifi: Add capability to set regulatory domain through device's interface

Fixes BMC#25244

gsupplicant: Add a method to set interface's country property

gsupplicant: Adapt set country callback to return a result

This is necessary for the next coming patches, to fix country setting by interface

technology: Set regulatory domain by device or fallback to technology based

device: Add capability to notify on regulatory domain setting

technology: Notify the regulatory domain setting by device

device: Add driver capability to set regulatory domain on device basis

doc: Fix small typo

Release 1.3

rtnl: Fixed log for the type RTM_GETADDR

The connman prints an unknown type when sending a request RTM_GETADDR

tethering: Fix incorrect index value for private network

In __connman_private_network_request, pn->fd is used as the index value
and passed to __connman_ippool_create, which is not correct.
It result in the private network is block used externally.

unit: Watch for signals only on CONNMAN_SERVICE

ofono: Watch for signals only on OFONO_SERVICE

dundee: Watch for signals only on DUNDEE_SERVICE

bluetooth: Watch for signals only on BLUEZ_SERVICE

gdbus: Fix removal of filter after last filter_data

If there's a signal watch that's also watching for name
(data->name_watch) currently we are trying to remove the message_filter
twice since we may have the following call chain:

filter_data_remove_callback()
  filter_data_free()
    g_dbus_remove_watch()
      filter_data_remove_callback()
filter_data_free()
        dbus_connection_remove_filter()
  dbus_connection_remove_filter()

Because of this we can't currently watch for signals passing the bus
name. After this patch we don't have this issue anymore.

We fix it by removing the filter before calling filter_data_free() if we
are the last filter_data and thus avoid calling
dbus_connection_remove_filter() twice.

ofono: Fix potential crash

If a dummy context is created, it might happen
that its index is equal to -1. In this particular case
the IP address has not been copied to context->address
during ipv4 extraction.
If we force from oFono a context activation, it will
make ConnMan crashing on set_connected().

inet: Fix usage of getaddrinfo() return value

Fixes BMC#25304

wifi: Add wifi pointer NULL checks

Always check what connman_device_get_data() returns and act
accordingly.

main: static-qualify default config values

The default configuration arrays default_auto_connect and
default_blacklist must be static-qualified. Otherwise, their
stack-allocated storage will go out of scope after parse_config
is finished and empty or commented-out DefaultAutoConnectTechnologies
or NetworkInterfaceBlacklist config entries will refer to
random stack data.

TODO: Add WPS pin code task

TODO: Remove DUN task, update WiFi scan owner

Release 1.2

gweb: Use g_try_realloc instead of g_realloc

doc: Document 'informational' value of Requirement field

agent: Fix how to provide PreviousPassphrase through D-Bus

ipconfig: Save prefixlen only if not cleared

This commit fixes a bug when using manual ipv4 addressing.
The first time it works, but once the service is reconnected it uses
an invalid netmask: 255.255.255.255.

wifi: Check we have valid wifi pointer in autoscan

The wifi pointer in device might be NULL if wpa_supplicant dies
between autoscans.

technology: Return proper error code when already disabled

Return AlreadyDisabled to the caller instead of InProgress if we
are already disabled.

technology: Return proper error code when already enabled

We do not call the generic error message function because
it maps EALREADY to EINPROGRESS. So instead we call correct
error return function which returns AlreadyEnabled to the
caller.

Fixes BMC#25306

main.conf: Comment out example default values and add documentation

main: Use 'NetworkInterfaceBlacklist' string everywhere

rtnl: Use filtered devices to determine blacklisted devices

Instead of hard-coding some selected devices to be ignored, the
blacklist check now uses the connman_device_isfiltered list.

device: Consider blacklisted interfaces from main.conf when filtering

Interfaces blacklisted in main.conf are now also taken into account
by connman_device_isfiltered. This is done in addition to what gets
passed with -i or -I on the command line.

main: Support NetworkInterfaceBlacklist from main.conf

This adds a settings variable and a function for fetching that list

main.conf: Add NetworkInterfaceBlacklist

dnsproxy: Check cache for TCP request before connecting to server

It is not necessary to create TCP sockets to server if we have
the answer already in the cache.

dnsproxy: Use DNS header in checks

We should use the DNS header struct when checking DNS
protocol values instead of using pointer so that the
TCP offset is properly taken care of.

dnsproxy: Add more debugging prints

dnsproxy: Check cache properly for TCP packets

The DNS packet starts from offset 2 in TCP payload.