gatchat: Introduce g_at_chat_get_userdata

handsfree-audio: Add additional debugs

gdbus: Close private connection if setup fails

Private connection should be properly closed with dbus_connection_close
otherwise libdbus exits with the following error:

  'The last reference on a connection was dropped without closing the
   connection. This is a bug in an application. See
   dbus_connection_unref() documentation for details. Most likely, the
   application was supposed to call dbus_connection_close(), since this
   is a private connection.'

gdbus: Fix crash in g_dbus_create_error_valist

Passing NULL format parameter to vsnprintf results in invalid argument
error on glibc. But with some other libc libraries (musl and uClibc)
this results in dereferencing NULL pointer and crash due to
segmentation fault.

gdbus: Use g_dbus_create_error_valist internally

There is no need to duplicate code in g_dbus_send_error_valist.

gdbus: Add g_dbus_get_flags function

The g_dbus_get_flags function enables detection of when the
G_DBUS_FLAG_ENABLE_EXPERIMENTAL is set.

gdbus: Make GDBusClient work without ObjectManager

This makes GDBusClient work normally without ObjectManager.

gdbus/client: Allow specifying ObjectManager path

GDBusClient currently hard-codes "/" as the remote ObjectManager path.
This is generally incorrect, as an application can choose to expose an
ObjectManager at any well-known path. This patch fixes this by allowing
the user to pass in the ObjectManager path by introducing a new
conctructor "g_dbus_client_new_full".

gdbus/client: Don't GetManagedObjects w/o handlers

The client code currently issues GetManagedObjects if new handlers are
set via g_dbus_client_set_proxy_handlers. An application may set these
to NULL before unref'ing a client or to simply prevent further events.
Hence, there is no need to refresh objects or properties if all handlers
are NULL.

hfpmodem: slc.c: make sure to use none_prefix

hfpmodem: hfpmodem.c make sure to use none_prefix

hfp_ag_bluez5: use none prefix for AT+BCC.

iPhone 5s with iOS8.2 sometimes failes to acknowledge AT+BCC
with OK. This means +CIEV events get consumed by this command
and call ended notification is not parsed.

Nov 30 00:00:19 ofonod[938]: > AT+BCC\r
Nov 30 00:00:28 ofonod[938]: < \r\n+CIEV: 2,0\r\n

gdbus: Fix not calling disconnect function

If daemon gets disconnected from D-Bus sender is NULL. Watches that
was explicitly added with NULL sender (ie disconnected_signal in
g_dbus_set_disconnect_function) should be called anyway.

gdbus: Don't refresh objects/props if disconnected

If g_dbus_client_set_proxy_handlers gets called from within a
proxy_removed callback, the code may end up refreshing the proxy's
properties and incorrectly access the client's proxy_list as it gets
freed. This patch fixes this, so that get_managed_objects does nothing
if it gets called during a service disconnect.

hfpmodem: Make sure to use none_prefix

ofonod[253]: > AT+CCWA=1\r
ofonod[253]: < \r\n+BCS:2\r\n
ofonod[253]: < \r\nOK\r\n

hfpmodem: Make sure to set the prefix properly

ofonod[1239]: > AT+COPS=3,0\r
ofonod[1239]: < \r\n+BCS:2\r\n
ofonod[1239]: < \r\nOK\r\n

smsutil: Add additional sanity check

We make sure that after performing the UTF8 -> GSM conversion, the
number of GSM bytes is not greater than 11, which is the maximum
payload.

unit: Add test to encode / decode 11 char TP-OA

sms: Fix alphanumeric TP-OA handling

TP-OA max length comparisons were incorrect because TP-OA's 7-bit
coded octets transport eleven 8-bit chars.  The current code assumed
only 10 chars were possible.

The patch
- increases the array size to 23, (maximum of 22 bytes for UTF8
  encoding + null terminator)
- Updates the sanity check to account for the correct maximum
- For encoding, checks the maximum length in UTF8 characters instead of
  bytes

AUTHORS: Mention Tommi's contributions

handsfree-audio: Refactor manager init / cleanup

main: Remove handsfree_audio_manager init/cleanup

Init allocates a SCO audio socket always. oFono should do that
with bluez5 but not with bluez4.  This patch starts the refactoring of
the handsfree_audio_manager init/cleanup functionality.

atmodem: fix retries reporting from AT+CPINR

The retries array was not correctly filled in.

atmodem: Sierra modems should be polled after CPIN

Sierra modem will return "CME ERROR: 14" when polled right after pin has
been entered. Use the existing vendor quirk to handle this.

sierra: add sim state polling after CFUN enable

When pin is queried shortly after a Siera dongle is plugged in,
"AT+CPIN?" responds with "CME ERROR 14: SIM".
Poll the sim, as already done by several other vendor plugins.

Release 1.16

radio-settings: Show all available technologies

Not all possible futures technologies were being showed on the DBus
interface.

phonesim: Instantiate radio-settings in post_sim

phonesim: Add minimal radio-settings functionality

phonesim: Add skeleton of radio settings driver

radio-settings: Always query available rats

If fast dormancy query is not implemented the available rats are not
queried.

radio-settings: Add additional sanity check

radio-settings: rework AvailableTechnologies logic

Take advantage of the new enum structure to future proof and simplify
the logic of generating the AvailableTechnologies property

include: No need for OFONO_FLAG* defines

test: Add AvailableTechnologies to list-modems

src: Implement RAT list property

include: Add method to list RATs to radio-settings

doc: Add AvailableTechnologies property

This new property in org.ofono.RadioSettings interface will hold the
possible values for radio access technology for the modem.

sim: Fix pin retries bogus values on some arch-es

On some architectures the SimManager.Retries property was getting bogus
values.  This is because we were sending an array which pointed to int
values instead of the expected unsigned char values.

This fix allocates a temporary array of unsigned chars to hold the
actual D-Bus values being sent.  Additionally, the dictionary array is
changed to point to the temporary unsigned char based values instead of
the raw 'int' based retry values.

test: Support receiving SMS with non-ASCII chars

Without this I get the following Python traceback, for an SMS that
contains the UK pound sign.

ERROR:dbus.connection:Exception in handler for D-Bus signal:
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/dbus/connection.py", line 230, in maybe_handle_message
    self._handler(*args, **kwargs)
  File "./receive-sms", line 9, in incoming_message
    print("%s" % (message))
UnicodeEncodeError: 'ascii' codec can't encode character u'\xa3' in position 51: ordinal not in range(128)

he910: Update copyright

he910: enable location raporting atom

udev: add GPS port definition for HE910

telitmodem: Fix a few coding style violations

AUTHORS: Mention Kuba's contributions

telitmodem: location reporting driver

doc: new file doc/telit-modem.txt

emulator: Refine +BIND support

hfp_ag_bluez5: Bump HFP version to 1.7

emulator_fuzz: Add test plugin

This plugin creates an org.ofono.test.EmulatorFuzz interface on path
/test.  Currently only one method is implemented, SetIndicatorActive.

SetIndicatorActive("DistractedDrivingReduction", True|False) allows to
activate / deactivate the HF indicator via +BIND unsolicited
notification.

emulator: Implement new API

include: Add API to (de)activate HF indicators

test: Add test/set-ddr

Script to set DistractedDrivingReduction property

handsfree: Be extra pedantic with booleans

hfp_hf_bluez5: Use latest HFP version

handsfree: Don't try to send a method return

When we are simply updating the AG of HF indicator changes, do not try
to send a method return message.

slc: Make sure to initialize variables

hfp: Implement +BIND notifications

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