packaging: Bump to 1.17

Change-Id: If60f26467c1d59a3ecaebec95ff9fa8b1cdf3732
Bug-Tizen: TC-2538
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>

changes: drop this change file

There is no real gain to use it, many other projects don't

Change-Id: I6978d2f682dc487712c1b0034d84c2e9b1c22a35
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>

change: fix date

Change-Id: Ie5382994562c09e1b78520f9a13c8d47dd12a3a6
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>

packaging: Update to version 1.16

Signed-off-by: Leon Anavi <leon.anavi@konsulko.com>
Change-Id: I2d38ed40dedefff9435921fd661c52a7172db063
Bug-Tizen: TC-2538
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>

packaging: gbs: Use tags from upstream

Change-Id: I8683fa093eeac6d2ad7395e8ee55c36289797625
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>

Change SmackExecLabel to SmackProcessLabel

Since systemd v219 Tizen-specific SmackExecLabel option was dropped in
favour of SmackProcessLabel option from upstream.

Change-Id: Ie0bef9c21ae163008bdea8cb5dafb305ec0f058c
Signed-off-by: Maciej Wereski <m.wereski@partner.samsung.com>

Update to version 1.15

he910: Fixup SIM status switching

SIM inserted notification may be missed, when get SIM status is ready,
make sure SIM inserted is set.

Change-Id: Iad752d39af544c2f177015b5e2530018e8947d40

Upgrade to version 1.14

Add the SmackExecLabel=User to allow rw for userspace apps

Change-Id: I35aad4dcd50d0c7025a15ec184d481540add1294
Signed-off-by: Michael Demeter <michael.demeter@intel.com>

packaging: diable bluez4 as bluez5 is integrated

Change-Id: I42fc2b2b13a2f3ab32c6049f555262b13458cf62
Signed-off-by: Sebastian Chlad <sebastianx.chlad@intel.com>

 Imported 1.13

resetting manifest requested domain to floor

Set license using %license

fix packaging and add ofono.manifest

Initial packaging

packaging: Initial packaging for Tizen on 1.12

Change-Id: I5cfd9eb836c0dc81dc70f84306b6d4df342d4ba6

Release 1.17

sms: Add trace when datagram is not delivered

smsutil: Do not check the origin port

Do not check if the origin port is reserved, as some operators use that
range of values.

push-notification: Accept push from any origin

Accept push notifications regardless of the origin port, as some
operators do not insert there the WAP connectionless session service
port, and use random values instead.

hfp: Handle extra CCWA event

When a call is waitng, CCWA event is sent and call object
in state WAITING is created. on ReleaseAndAnswer it is
promoted to INCOMING and later to ACTIVE.
iPhones send an extra CCWA event when active call is ended.
This extra event is creating a second call object in state
WAITING. It is not possible to have two WAITING calls, but
previously waiting call was already promoted to INCOMING.
For a brief time we have two calls from the same number,
one INCOMING and one WAITING. Later WAITING one is removed.
As we cannot have a waiting and incoming call at the same
time, ignore CCWA when there is already an INCOMING call.

< \r\n+CIEV: 3,3\r\n
< \r\n+CIEV: 2,1\r\n
< \r\n+CIEV: 3,0\r\n
< \r\n+CCWA: "01234567890",129,1,"Me"\r\n
< \r\n+CIEV: 3,1\r\n
> AT+CLCC\r
< \r\n+CLCC: 1,0,0,0,0,"09876543210",129,"Me"\r\n
< \r\n+CLCC: 2,1,5,0,0,"01234567890",129,"Me"\r\n
< \r\nOK\r\n
< \r\n+CIEV: 2,0\r\n
< \r\n+CCWA: "01234567890",129,1,"Me"\r\n
< \r\n+CIEV: 2,1\r\n
< \r\n+CIEV: 3,0\r\n
> AT+CLCC\r
< \r\n+CLCC: 2,1,0,0,0,"01234567890",129,"Me"\r\n
< \r\nOK\r\n

voicecall: Don't accept USSD strings in Dial()

hfp: Synchronize call state in case of +CHUP error

It is possible for the phone to accept Dial request
but not actually dial. This leaves a voicecall object
in state 'dialling' that cannot be removed.
Proposed workaround is to trigger AT+CLCC when an error
is returned for Hangup. As the call is not on the list,
this would remove this hanging object and signal CallRemoved.

Windows Phone trace with this fix:
ofonod[273]: > ATD1;\r
ofonod[273]: < \r\nOK\r\n
ofonod[273]: src/voicecall.c:dial_handle_result() Registering new call: 1
ofonod[273]: < \r\n+CIEV: 5,4\r\n
ofonod[273]: src/network.c:ofono_netreg_strength_notify() strength 80
ofonod[273]: > AT+CHUP\r
ofonod[273]: < \r\nERROR\r\n
ofonod[273]: src/voicecall.c:generic_callback() command failed with error: Unknown error type
ofonod[273]: > AT+CLCC\r
ofonod[273]: < \r\nOK\r\n
ofonod[273]: src/voicecall.c:ofono_voicecall_disconnected() Got disconnection event for id: 1, reason: 2

cdma-connman: Make static analysis tools happy

The kernel simply puts a null terminator at index 15 prior to ifr_name
processing.  So we do the same.

Original report by:
Sabas Rosales, Blanca E <blanca.e.sabas.rosales@intel.com>

 Buffer not null terminated (BUFFER_SIZE_WARNING) buffer_size_warning:
 Calling strncpy with a maximum size argument of 16 bytes on destination
 array ifr.ifr_ifrn.ifrn_name of size 16 bytes might leave the
 destination string unterminated.

  92        strncpy(ifr.ifr_name, interface, IFNAMSIZ);

ppp_net: Make static analysis tools happy

The kernel simply puts a null terminator at index 15 prior to ifr_name
processing.  So we do the same.

Original report by:
Sabas Rosales, Blanca E <blanca.e.sabas.rosales@intel.com>

 Buffer not null terminated (BUFFER_SIZE_WARNING) buffer_size_warning:
 Calling strncpy with a maximum size argument of 16 bytes on destination
 array ifr.ifr_ifrn.ifrn_name of size 16 bytes might leave the
 destination string unterminated.

  67        strncpy(ifr.ifr_name, net->if_name, sizeof(ifr.ifr_name));

gprs: Try re-attaching when we switch cells

message-waiting: Fix reading EF_MWIS records

message-waiting: Update properly EF_MWIS SIM file

AUTHORS: Mention Johannes' contributions

udevng: add support for Ericsson N5321 gw

hfp_hf_bluez5: Fix crash on re-pairing a Device

It may happen that a Device object is unpaired an paired again
without being removed from DBus. This in turn triggers second
modem object to be created, but not fully initialized.
If this modem object is used, oFono will crash.

handsfree: Mark GetProperties method ASYNC

u8500: Fix compiler warning with logical expression

  CC       plugins/u8500.o
plugins/u8500.c: In function ‘reachable_cb’:
plugins/u8500.c:235:28: error: logical not is only applied to the left hand side of comparison [-Werror=logical-not-parentheses]
  if (!g_isi_msg_error(msg) < 0)
                            ^

n900: Fix compiler warning with logical expression

  CC       plugins/n900.o
plugins/n900.c: In function ‘mtc_reachable_cb’:
plugins/n900.c:241:28: error: logical not is only applied to the left hand side of comparison [-Werror=logical-not-parentheses]
  if (!g_isi_msg_error(msg) < 0)
                            ^

isiusb: Fix compiler warning with logical expression

  CC       plugins/isiusb.o
plugins/isiusb.c: In function ‘reachable_cb’:
plugins/isiusb.c:207:28: error: logical not is only applied to the left hand side of comparison [-Werror=logical-not-parentheses]
  if (!g_isi_msg_error(msg) < 0)
                            ^

gatchat: Fix compiler warning with logical expression

  CC       gatchat/gatchat.o
gatchat/gatchat.c: In function ‘have_line’:
gatchat/gatchat.c:586:28: error: logical not is only applied to the left hand side of comparison [-Werror=logical-not-parentheses]
  if (!strncmp(str, "AT", 2) == TRUE)
                            ^

AUTHORS: Mention Marko's contributions

configure.ac: fix typo

AUTHORS: Mention Sergey's contributions

build: make ofono build against musl

ifdef away GNU libc extensions and use a POSIXly correct pointer type

handsfree: Fix potential buffer overflow

Function: ag_features_list
 static const char *list[10];  (Out of bounds write, line 75)
  Incrementing i the value is now 10, for “hf-indicators”

Reported by: blanca.e.sabas.rosales@intel.com

sim: Fix up whitespace issues

Emit LockedPins after pin_type is queried

Fixes property change not being emited when hot-swapping a
PIN-enabled card.

sim: Reset pin_type on card remove

Fixes PinRequired not being emitted when a card is inserted

hfpmodem: Fix connecting to AG with existing mpty

If there is more then one active or held call, we are in mpty calls.
We won't get indicator update if any of them is released by CHLD=1x.
So we have to poll it.

gprs: Add comment to gprs_reset_contexts

test: Fix file mode for reset-contexts

gprs: Refactor to remove forward declaration

test: Add script for resetting contexts

doc: Add description for ResetContexts method

gprs: Add DBus method to reset contexts

Add DBus method that removes the current contexts and re-provisions
using the APN database.

cinterion: Correct use of freed structure

On error struct cb_data *cbd was used after in cinterion_set_online
after already being freed.

ste: Fix out-of-order free

CALLBACK_WITH_FAILURE used data structure freed just beforehand

dundee: Fix out-of-order free

CALLBACK_WITH_FAILURE used data structure freed just beforehand

AUTHORS: Mention Alex's contributions

udev: Add support for ehs6 name

As with tc65, ehs6 makes use of cinterion plugin

cinterion: Register as OFONO_VENDOR_CINTERION

This enables us to take advantage of vendor specific quirks
(e.g. signal strength handling specifics for tc65)

atmodem: Add Cinterion quirk for signal strength

Implement OFONO_VENDOR_CINTERION specific vendor support to register
textual +CIEV indications for signal strength using AT^SIND command.

tc65: Replace tc65 plugin with cinterion plugin

On the basis that tc6x and other Cinterion devices will likely
have similar firmware requirements, provide a generic Cinterion
plugin which is functionally identical to the replaced tc65 plugin.

The udev implementation retains support for "tc65" name for
backwards compatibility, and adds support for the new "cinterion"
name.

hfp_hf_bluez5: Implement sco_connected_hint

handsfree-audio: Call sco_connected_hint

include: Add sco_connected_hint to handsfree-audio

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