hfp: Notify core of HF indicators supported by AG

This information was obtained from SLC negotiation

hfp: Track supported / active indicators

This information is obtained during SLC establishment

handsfree: Implement new HF indicators API

include: Add HF Indicators related API

hfp: Add implementation of +BIEV

emulator: Refactor data types

hfp: Add HF Indicators BRSF feature for 1.7

If HFP 1.7 client is used, enable the right BRSF feature

handsfree: Skeleton implementation of DDR

Distracted Driving Reduction or Enhanced Safety is implemented using HF
indicator 0x0001

handsfree: Add new driver method for HF indicators

hfp: Add initial SLC negotiation for HFP 1.7

doc: Add Distracted Driving Reduction to Handsfree

handsfree: Implement new Features property value

doc: Add 'hf-indicators' feature to Handsfree API

emulator: Enable HF Indicator feature

emulator: Extend BRSF bitmap

The current check of 8 bits is not enough with HFP 1.7

emulator: Add +BIEV implementation

emulator: Add +BIND implementation

emulator: Fix CHLD=? not treated as part of SLC

emulator: Improve SLC establishment logic

hfp: Add enum for known HF Indicators

hfp: Update for new HFP 1.7 values

doc: Fixup Handsfree API

handsfree-audio: Fix broken coding style in switch statement

gdbus: Fix match rule for NameOwnerChanged

When subscribing to the D-Bus signal NameOwnerChanged from the bus driver,
specify the object path and the sender in the match rule. Otherwise, random
connections on the bus could impersonate the bus driver.

gdbus: Fix crash when watch is toggled or disconnected

This partially reverts 510b32b7156625b9df737c916b7a7a5c6fb285b9 since it
still necessary to take a reference before calling dbus_watch_handle
since internally it can call watch_info_free as in the following trace:

 Invalid read of size 8
   at 0x121085: watch_func (mainloop.c:105)
   by 0x4C72694: g_main_context_dispatch (gmain.c:2539)
   by 0x4C729C7: g_main_context_iterate.isra.23 (gmain.c:3146)
   by 0x4C72DC1: g_main_loop_run (gmain.c:3340)
   by 0x120541: main (main.c:551)
 Address 0x5bbcd90 is 16 bytes inside a block of size 24 free'd
   at 0x4A079AE: free (vg_replace_malloc.c:427)
   by 0x4C7837E: g_free (gmem.c:252)
   by 0x4F708BF: dbus_watch_set_data (dbus-watch.c:614)
   by 0x4F70938: _dbus_watch_unref (dbus-watch.c:132)
   by 0x4F6E9A7: _dbus_transport_handle_watch (dbus-transport.c:884)
   by 0x4F59AFB: _dbus_connection_handle_watch (dbus-connection.c:1497)
   by 0x4F70AF9: dbus_watch_handle (dbus-watch.c:683)
   by 0x121084: watch_func (mainloop.c:103)
   by 0x4C72694: g_main_context_dispatch (gmain.c:2539)
   by 0x4C729C7: g_main_context_iterate.isra.23 (gmain.c:3146)
   by 0x4C72DC1: g_main_loop_run (gmain.c:3340)
   by 0x120541: main (main.c:551)

gdbus: Fix crash when calling g_dbus_add_service_watch

If g_dbus_add_service_watch is called for service which bus name is
already known the following crash can happen:

invalid read of size 1
  at 0x4C2A2F2: strlen (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
  by 0x4E97722: g_strdup (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x405B0C: update_name_cache (watch.c:435)
  by 0x405C37: update_service (watch.c:593)
  by 0x4E7A2A5: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x4E7A627: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x4E7AA39: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x4038EA: client_ready (test-gdbus-client.c:1014)
  by 0x4E9E5E0: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x4E9E7A5: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x4E9EB1A: g_test_run_suite (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x403614: main (test-gdbus-client.c:1058)
Address 0x5dbe5d0 is 0 bytes inside a block of size 7 free'd
  at 0x4C28577: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
  by 0x4E7FF7E: g_free (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x405B04: update_name_cache (watch.c:434)
  by 0x405C37: update_service (watch.c:593)
  by 0x4E7A2A5: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x4E7A627: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x4E7AA39: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x4038EA: client_ready (test-gdbus-client.c:1014)
  by 0x4E9E5E0: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x4E9E7A5: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x4E9EB1A: g_test_run_suite (in /usr/lib64/libglib-2.0.so.0.3800.2)
  by 0x403614: main (test-gdbus-client.c:1058)

Revert "gdbus: Don't include just added interfaces in GetManagedObjects"

This actually creates a problem since the code now does flush any pending
message on the queue any signal will be send right away before the
actual reply to GetManagedObjects which will not contain those interfaces.

gdbus: Fix use after free

Refactor filter_data_remove_callback so that we do not iterate over
freed pointer.

gdbus: Avoid reporting GDBusClient disconnect twice

No matter if disconnection was reported previously, g_dbus_client_unref()
was always calling service disconnect callback. This patch fix the
following scenario:
1) service disconnects from the bus
2) disconnect callback gets called
3) client calls g_dbus_client_unref(), disconnect callback is called
   again.

build: Avoid warnings about _FORTIFY_SOURCE

Gentoo (and others) define _FORTIFY_SOURCE=2 by default.  We redefine
it, resulting in an error / configure failure.

ublox: Minor style cleanup

ublox: Remove unneeded VENDOR flags

devinfo and netreg atoms don't use vendor specific behavior

plugins: new driver for u-blox SARA-U270 modems

This driver may also work (perhaps with more or less trivial changes)
with other u-blox modems (SARA, LISA, LEON) but this hasn't been tested.

atmodem: set the auth method for u-blox modems

sim: query u-blox PIN retries with AT+UPINCNT

udevng: add detection logic for u-blox modems

quectel: Minor style cleanup

quectel: Remove unneeded VENDOR flags

Only the sim atom has QUECTEL specific code

plugins: add a new driver for Quectel UC15 modems

gprs: add support for u-blox +UREG URCs

atmodem: add vendor u-blox

atmodem: Minor style fixups

sim: query Quectel UC15 PIN retries with AT+QPINC?

udevng: add detection logic for Quectel modems

atmodem: add vendor Quectel

hfpmodem: Fix crash with more than two calls

A periodic CLCC polling is started when there is an ongoing multiparty
call and a new call appears in the system. A simple way to reproduce
the crashing scenario is:

1. Place a call.
2. Place a second call.
3. Create a multiparty call with both calls.
4. Place a third call (incoming or outgoing does not matter).
5. Disconnect HFP from the modem.

Within the function ciev_callheld_notify, the AT+CLCC command is also
invoked, thus a new cyclic CLCC polling is started, and it overwrites
the timer resource identifier stored in voicecall_data.clcc_source.
This means that there are several timers doing the CLCC polling, but
only one of those is under control, i.e. it can be removed through its
source identifier, hence a timer source leak.

This has a fatal consequence when the HFP modem is disconnected. The
function hfp_voicecall_remove stops the timer that is under control
before freeing the voicecall_data struct. However there are other timers
that are still active and will execute its handler poll_clcc afterwards.
Inside poll_clcc the driver_data is accessed, which is already NULL.

A solution for this is to avoid starting a CLCC polling if there is
already one active, i.e. clcc_source is not 0. By doing this the
uncontrolled timers will not cycle forever.

Release 1.15

gprs: provision the authentication method

mbpi: add support for provisioning the auth method

Use the authentication method from the mobile-broadband-provider-info
database if it is specified and supported (we support CHAP and PAP).
Default to CHAP if the database does not specify a method (i.e.: the
previous behaviour).

include: add auth method to the provisioning API

AUTHORS: Mention Philip's contributions

atmodem: set the PPP authentication method to use

gprs: Make the code a bit easier to read

gprs: make PPP authentication method configurable

Add a new "AuthenticationMethod" property to select between "pap"
and "chap" PPP authentication, defaulting to "chap" (i.e.: previous
behaviour).

include: add auth method to the gprs context API

doc: describe the AuthenticationMethod property

gatchat: Fix up minor coding style issues

gatchat: implement PAP authentication

Make the authentication method configurable, CHAP or PAP, defaulting to
CHAP (i.e.: previous behaviour).

Implementation details:

 o If PAP is configured, we NAK the CHAP authentication protocol option
   in LCP configuration requests and suggest PAP instead.  This works
   around the amusing requirement of 3GPP TS 29.061 that modems must
   send a forced positive acknowledgement of the authentication method
   tried (i.e.: the modem will successfully accept any CHAP handshake,
   but if the network only supports PAP, the modem will hang up
   when it tries and fails to activate the PDP context)

 o The PAP Authenticate-Request is resent a hard-coded three times at
   ten-second intervals.  This may be a bit too persistent.  Chances
   are if it doesn't work the first time, it'll never work, but the
   RFC insists that we MUST retry.

AUTHORS: Mention Sergio's contributions

hfpmodem: Split AT+VTS command into multiple +VTS

According to the standard "3GPP 27.007 v6.8.0" Appendix C.2.11,
when sending multiple DTMF characters, these must go in individual
+VTS commands for each tone. This adopts the AT modem approach.

Before: AT+VTS=1234\r
After:  AT+VTS=1;+VTS=2;+VTS=3;+VTS=4\r

test: Adapt data test scripts to multi-modem

Some tests scripts were not ready to handle situations with more than
one modem present. This change fixes the data context scripts.

test: Adapt voice call test scripts to multi-modem

Some tests scripts were not ready to handle situations with more than
one modem present. This change fixes the scripts that handle voice
calls.

hfp: Fix case where RING never arrives

hfp: Use proper prefix

Otherwise the BCS command can consume unsolicited notifications.

ofonod[3270]: < \r\n+BCS: 2\r\n
ofonod[3270]: > AT+BCS=2\r
ofonod[3270]: < \r\n+CIEV: 1,1\r\n
ofonod[3270]: < \r\n+CIEV: 2,0\r\n
ofonod[3270]: < \r\nOK\r\n

handsfree: correct crash on connect of hfp

handsfree: Rename driver function

To be more consistent with others in the same file

handsfree: Simplify logic

hfp: Add CNUM query to handsfree interface for subscriber number

handsfree: Rename variable

handsfree: Reflow logic

handsfree: Remove function

No need for one line functions that are only used once

handsfree: Rename function

No need for the ofono_ prefix here

handsfree: Remove unneeded variable

handsfree: Remove unneeded variable i

handsfree: style fixes for previous commit

hfp: Add subscriber number to handsfree properties

include: Add subscriber number to handsfree API

doc: Add subscriber number to handsfree-api

AUTHORS: Mention Jussi's contributions

phonesim: Fixup whitespace

phonesim: Add conf file override

OFONO_PHONESIM_CONFIG allows the configuration file location to be
overriden from the default $CONFIGDIR/phonesim.conf (typically /etc)

gdbus: Add g_dbus_proxy_set_property_array

This patch adds a new gdbus utility function to allow setting a property
of fixed, and non-fixed values array.

he910: Fix unused variable warning

plugins/he910.c: In function ‘qss_query_cb’:
plugins/he910.c:174:21: error: unused variable ‘data’
[-Werror=unused-variable]

he910: Switch to permissive parser

Seems the CCFC implementation on Telit firmware is not v.250 compliant:
ofonod[445]: Aux: > AT+CCFC=0,2\r
ofonod[445]: Aux: < \r\n\r\n+CCFC:0,7\r\n\r\nOK\r\n

he910: Fixup SIM detection

In certain cases the #QSS notification is never sent, or sent too early.
Make sure that we enable #QSS notification only after the modem has been
set as powered.  Also make sure that we explicitly query #QSS in case we
missed the earlier #QSS notification.

hfp: Use enum from hfp.h

hfp: Fixup variable names

hfp: Add CHLD features to drver data

handsfree: Fixup variable names

hfp: extend Features to three way calling and CHLD

include: Declaration function to set CHLD features

hfp: Define hfp_ag_chld_feature

doc: Extend Features property

AUTHORS: Mention Alfonso's contributions

unit: Fix test to use valid EF_PNN

test-simutil was not using EF_PNN files with a valid format.

simutil: Fix EF_PNN access

EF_PNN was not being read properly (see TS 24.008, section 10.5.3.5a,
for network names format), which affected the displayed PLMN name for
some MVNOs. Some modems already read the file and return the right
string: these do not show the problem.

stktest: Remove some unneeded output

test: Make sure test-stk-menu works with python3

AUTHORS: Mention Martin's contributions

test: Convert to Python 3

Change raw_input() to input() and unicode to str. This is *not*
compatible with Python 2. Update the hashbangs to run with Python 3.