dnsproxy: Fix wrong domain len causing crash in memmove

If host_len is 0, strlen(*ptr) will return 0 as well and domain_len will
have the wrong value (-1). This causes connman to crash with the
following backtrace:

connmand[13361]: ++++++++ backtrace ++++++++
connmand[13361]: #0  0x7f33a5e49140 in /lib/libpthread.so.0
connmand[13361]: #1  0x7f33a509ed80 in /lib/libc.so.6
connmand[13361]: #2  0x4671a8 in forward_dns_reply() at src/dnsproxy.c:1568
connmand[13361]: #3  0x46725a in udp_server_event() at src/dnsproxy.c:1706
connmand[13361]: #4  0x7f33a62e14a5 in /usr/lib/libglib-2.0.so.0
connmand[13361]: #5  0x7f33a62e17d8 in /usr/lib/libglib-2.0.so.0
connmand[13361]: #6  0x7f33a62e1bd2 in /usr/lib/libglib-2.0.so.0
connmand[13361]: #7  0x40dfa1 in main() at src/main.c:510
connmand[13361]: #8  0x7f33a4f8e455 in /lib/libc.so.6
connmand[13361]: +++++++++++++++++++++++++++

To reproduce this issue, we need to have a domain name set and make a
query for "". The easiest way to do so is to use the (wrong) git command
below:

git push :your.host.name

Make sure we calculate domain_len the right way. By guaranteeing
host_len and domain_len are >= we can also use memcpy() instead of
memmove().

dnsproxy: Avoid useless memset

dnsproxy: Fix constiness of variables

There's no point in casting 'domain' and 'query' to char* to remove the
const since we are not modifying them. Also we can avoid another var
entirely by updating the pointers only.

storage: Remove obsolete functions that nobody calls

Config load and save functions are not called so they
are not needed.

config: Remove unused function

agent: Update __connman_agent_report_error to return -EINPROGRESS

Handle the change from -EIO to -EINPROGRESS return value in
service.c.

agent: Update functions used by wispr to send -EINPROGRESS

Handle the change from -EIO to -EINPROGRESS return value in
wispr.c.

agent: Return -EINPROGRESS when request passphrase calls an agent

As a result of returning -EINPROGRESS, the code in service.c can be
simplified.

service: Remove unnecessary setting of ipconfing to failure

With this change service Connect will not report failure but
instead wait for the Agent API to complete. The remaining
__connman_network_disconnect() will reset ipconfig to IDLE
state. If a service were set to failure state, the service
would be ignored in later autoconnects.

wifi: Stop autoscan in the relevant place.

If not stopped before checking the scanning status of the device, we might
not raise a user initiated scan just because an autoscan is going one which
is bogus.

wifi: Check device is not already scanning before a new scan.

gsupplicant's interface scanning value is slow to be updated since it's wpa_s event
which updates it, thus leading to a time gap where it is possible to request another scan.
It will therefore reference once again device, which will lead to too many references hold
by wifi plugin

wifi: If scanning, unref the device when removing wifi driver.

wifi: Scanning is set to false in case of effective autoscan stop.

device: Make scanning getter method public.

storage: migrate 0.76-era device enabled / powered state

This patch ensures that [device_<type>] Powered=<true|false> device enabled
/ powered state migrates from 0.76-era versions of default.profile.

storage: Remove default.profile when migration has been done

Unconditionally remove default.profile without any migration if
the currently used global settings save file exists. The current
settings formats for global and service settings were introduced
in version 0.78.

storage: Migrate services from obsolete default.profile

In order to remove the obsolete default.profile, migrate
services into the new setup with per-service directories.
If the service exists in the new format, it is not migrated.

storage: Remove support for loading services from default.profile

storage: Return any errors when saving services and global config

agent: Increase default timeout to 120 seconds

The D-Bus default timeout is 25 seconds and that is clearly not enough time
to enter a passphrase. Increasing this to 120 seconds now.

AUTHORS: Mention Elena's contributions

ipconfig: Show FIXED configuration IP address info

Show FIXED configuration IP address info in service IPv4 and
IPv6 properties. If IP address is not available via
ipconfig->system, use the provider configured ones instead.

Fixes BMC#25084

resolver: Send RS before RDNSS lifetime expires

Implemented feature from RFC 6106 section
'5.1. Recursive DNS Server Option':
"Lifetime      32-bit unsigned integer.
...
Hosts MAY send a Router Solicitation to ensure
the RDNSS information is fresh before the interval expires."

Host will send RS when a certain threshold of RDNSS lifetime is reached.
Values which can be adjusted:
- lifetime threshold - set to 80% from lifetime
- number of retries in case RA is not received - set to 0
- time between retries, in case RA is not received - set to 3 seconds

resolver: Fixed cleanup of Resolver timer

When connman is restarted, DNS servers are removed and added again
with function __connman_resolver_redo_servers(). While removing a
DNS entry, one should also remove the resolver timer registered for it.
Not doing so, causes the timeout handler resolver_expire_cb to fire
without being explicitly set.

doc: Update service Domains property description

service: Fix Domains property

The service Domains property shows Domains.Configuration if set,
otherwise the domain name given by DHCP or provider.

Fixes BMC#25122

wispr: If redirected and TLS is not supported we fallback on browser request

gweb: Adding a function to know if TLS is supported or not

agent: Verify that the reply contains a dictionary

gdbus: Fix incorrectly discarded signals

Signals with no arguments were incorrectly filtered out due to the NULL
inequality check.

manager: Use simpler argument names in ServicesChanged signal

manager: Fix wrong signature for SetProperty method

README: Add description for --disable-wispr option

gweb: Add function to check for TLS support

build: Add option for disabling WISPr support

gdbus: do not check signature twice

Message signature is already checked in generic_message(), so there's no
need to check again in the callback.

gdbus: add Method.NoReply annotation in introspection

gdbus: add Deprecated annotation in introspection

gdbus: remove signature and reply from tables

Do not set signature and reply in GDBus tables

Use GDBUS_* macros, so signature and reply fields are not set in each
method/signal.

gdbus: loop over args to check message signature

gdbus: use GDBusArgInfo to generate introspection

By using GDBusArgInfo in methods and signals, the introspection
generation is much simpler and we can add each argument name.

Convert GDBus methods and signals to use macro helpers

With these macro helpers we can separate in/out arguments and use their
own vector.

gdbus: add and use helpers for table declarations

gdbus: add argument info to methods and signals

Constify GDBus signal tables

Constify GDBus method tables

connection: Refactor VPN route setup

gdbus: Constify introspection method table

gdbus: do not call memset for terminating NUL

gdbus: return if method signature is malformed

doc: Clarify hidden WiFi network relations with Name+Type properties

technology: Set deprecated flag for GetProperties method

service: Set deprecated flag for GetProperties method

doc: Mention deprecation of GetProperties methods

gsupplicant: Coding style clean-up

This resolves several post-commit review coding style comments for
raw PSK support.

gsupplicant: Avoid critical errors

wpa_supplicant sends error if wlan interface cannot be created.
Do not treat this error as critical because after calling
g_critical() the program logging will be disabled.

doc: Update configuration format documentation

Clarify that 'Name' is used only as a string representation of an
SSID and ignored if the entry contains an 'SSID' entry. Also clarify
that the identifier part is unique only within the .config file.
Add indentation.

config: Reject invalid SSID hexadecimal entries

Reject provisioning .config file SSID entries that are not in
hexadecimal.

rtnl: Domain search list was not handled according to RFC 6106

Domain search list option can exist in options without RDNSS option.
Note that the DNSSL netlink support is not yet in linux kernel
3.4-rc7, thus we will not yet get a DNSSL option in netlink RA
options.

resolver: Do not try to update service if server is not set

resolver: Allow update to domain if server is not set

resolver: Follow connman coding style

gsupplicant: fix "raw" PSK support

This fixes "raw" PSK support by first converting the "raw" PSK from
a 64-character hexadecimal string to 32-byte array before appending
it to the network dictionary.

doc: SSID field is hexadecimal only, also document Name field

README: Add comment on OpenVPN version to use

README: Move the VPN entry down

So it is under the "Configuration and options" section.

gdhcp: Make sure we have the message before accessing it

gdhcp: Message type is required in DHCPv4

AUTHORS: Mention Lucas De Marchi's contributions

dhcpv6: Check NULL pointer dereference

AUTHORS: Mention Henrique Dante de Almeida's contributions

dhcp: Use g_list_length() to calculate the list length

dhcpv6: Check NULL pointer dereference

dhcp: Check NULL pointer dereference

wispr: Remove dead assignment

tools: Remove dead assignments in supplicant

tools: Remove dead assignment in supplicant

tools: Remove dead assignment in iptables-test

wispr: Remove dead assignment

rtnl: Remove dead assignment

iptables: Remove dead assignment

dnsproxy: Remove dead assignment

dnsproxy: Return error if initial send fails

vpn: Fix null pointer dereference

vpn: Fix null pointer dereference

ofono: Fix uninitialized variable

gsupplicant: Remove dead assignment

gdhcp: Remove dead assignment

gdhcp: Remove assignment that's never used

ofono: Fix uninitialized value

wifi: Fix NULL ssid being passed to memcmp

oFono: Initialize prefix_length

We could end up calling connman_ipaddress_set_ipv6() withouth
prefix_lenght initialized.

Reported by hdante.

Release 1.0

README: Fix minor spelling mistake

connection: Fix possible NULL dereference

connection: Fix host route to VPN server

The kernel will ignore host routes which are bogus.
In case of a PtP connection such as PPP for cellular the
gateway address should not be used.

provider: Fix network/netmask parsing and potential crash

gsupplicant: Use the right dbus helper when adding a fixed array in a dict

Fixes BMC#25120

ofono: Check for CM attached before registering network

We need to check the ConnectionManager attached signal before
we are allowed to connect to the gprs context:

connmand[592]: plugins/ofono.c:cm_update_attached() /zte_0 Attached 0

[...]

connmand[592]: plugins/ofono.c:context_set_active() /zte_0 active 1
connmand[592]: plugins/ofono.c:set_property() /zte_0 path /zte_0/context1
org.ofono.ConnectionContext.Active

[...]

connmand[592]: plugins/ofono.c:cm_update_attached() /zte_0 Attached 1

Fixes BMC#25112