bt-service/bt-service-event-receiver: Fix performance and style issues

Change-Id: I682dc2ab84ae44fef604f627338a3e39be343d09

Merge changes I9d4640ac,Iaf9170a7,I05dcd3ef,I7d6816f0 into tizen

* changes:
  offer __launch_bt_service() function only for Mobile profile
  keep current bluetooth state when starting bt-service
  check bt adapter state from bt-service daemon
  remove bt_status logic from bt-core

Merge "bt-api/bt-event-handler: Fix array index issue" into tizen

bt-api/bt-event-handler: Fix array index issue

Array index 'i' is used before limits check.

Change-Id: I008cde77339b4d68aa11bcecccf15543906234e5

Remove extraneous statements.

Change-Id: Ic3f83c8530b1b948169abb07a3aade8914326829

offer __launch_bt_service() function only for Mobile profile

Change-Id: I9d4640ac7ede87726c019e6a3eacda3d7b765271
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

keep current bluetooth state when starting bt-service

Previsouly when bt-service was started, it always tries to enable the bluetooth.
Therefore when the bluetooth is already enabled when starting bt-service,
it was not clean.

This patch allows to keep consistency between bluez and tizen bluetooth states.
By this way, if bluetooth is enabled when starting bt-service, bt-service just set vconf
and other variables. Following same logic, if bluetooth is down when starting
bt-service, we keep it down and align vconf status.

Change-Id: Iaf9170a739aa4a4c0ac37f1d98caafea1df50643
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

check bt adapter state from bt-service daemon

By this way if bt-service is not running when calling
bt_adapter_get_state() CAPI, bt-service will be started
by dbus activation.

Change-Id: I05dcd3ef1aa9201042bd403c5e357c1126244f7c
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

Merge "bt-service/bt-popup: Fix memory leak issue" into tizen

packaging: improvement of dependencies

Change-Id: I27ff5f634eb6f1f25128efaaef09603ef1e0961d
Signed-off-by: José Bollo <jose.bollo@open.eurogiciel.org>

bt-service/bt-popup: Fix memory leak issue

Change-Id: Ic60b32a648a4f6e28e71659c0eedc629297cfd8f

clean spec file

 - remove bashism

Change-Id: I0971a374da886d59c4f1bf057c24da15d7188908
Signed-off-by: Ronan Le Martret <ronan@fridu.net>

remove bt_status logic from bt-core

This is no more relevant since enable/disable function is done through dbus.
Now a changed power state is acknowledged in bluez event receiver.
Moreover bt-core could stay in a wrong BT_ACTIVATING or BT_DEACTIVATING state
if bluetooth state is modified by an external way.

Keep Tizen Mobile compatibility using __TIZEN_MOBILE__ macro in calling scripts
to enable/disable bt adapter. Those scripts allows to also stop/start other Tizen
bluetooth components (i.e. bluetooth-share, bluetooth-agent,...).

Change-Id: I7d6816f0a750f775f1b5a659174a63d663d81c3f
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

set 'User' smack label to bluetooth vconf keys

As bt-service is run as a user (app or guest), vconf keys needs to
be labelled as 'User' in order to be checked/modified by bt-service.

Change-Id: Ifcd44b0855ca4939fc8f2c5ce5d6c6a8eaedd39a
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

enable/disable bluetooth through ConnMan instead of BlueZ

Bluetooth can not be enabled on Tizen Common because of RFkill
blocks bluetooth. ConnMan manages RFkill and so call its DBus API
to enable/disable bluetooth adapter fixes this problem.

Change-Id: I277bf12536b6ae95a2d4a4a086b3d2dd63b69f40
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

add device disappear event

When BlueZ removes some scanned devices, it previously always trigs the
bond destroy event which is sent by API layer.

This fix consists of adding paired devices to g_list and verify paired
device status when removing device interface and so fire device disappeared
or bond destroy events.

This patch is combined with ea4787499cb21d2bfc744aad7b8ba103efe67845 from
platform/core/api/bluetooth API component.

It also removes static attribute from _bt_adapter_set_status() function in
order to update adapter power status in bt-service-event-receiver.

Change-Id: Idd7406b7add402689edaad06590f7c0b4a14bee5
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

Merge "remove bonded device from glist" into tizen

remove bonded device from glist

It prevents from sending wrong paired/connected states to API
when scanning.

Change-Id: Icb12e5fff0af615d76c10071b19efbb7231afaf4
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

Merge "do not retrieve adapter path from dbus message" into tizen

do not retrieve adapter path from dbus message

Every where, bluetooth-frwk checks the adapter state before doing an action.
Adapter state is retrieved from parsing dbus message from 'GetManagedObjects' method.
This is only needed to get adapter path from dbus message.
As adapter path is always '/org/bluez/hci0' using Bluez5, define it improves performance.

For example, it fixes a bug in Modello UI when getting default adapter with
bonded devices.

Change-Id: I19e86c0ac269b60773ac3b3e3f62d802ebd40013
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

Don't set DBUS_SESSION_BUS_ADDRESS

This variable is specific for user and used not only by bt-service.
Since systemd v212 package this variable is set in user@.service, thus
it can be removed from here.

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

Remove initscripts

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

fix unbond feature

Currently bt-fwk asks for unbonding device once it catches InterfaceRemoved event
from bluez whatever the dbus interface.
This patch allows to unbond device only when bluez removes device interface.

Change-Id: Ie0b44b851f2507e06e67fa6bf785a291db3c29a6
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

use bluez dbus api to enable/disable adapter

It was previously done invoking bluetooth-tools scripts.
For example, to disable adapter, one of the script kills bluez daemon process.
Now we just modify 'Powered' adapter property.

The patch also removes the timeout callback when enabling adapter since it can
disable adapter if the status is not correct. This adapter status is handled
by bluez4 adapter dbus api code which needs to be adapted.

Change-Id: I4bb5c4c03462a7fe9957e32dd8b196d37f358d30
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

fix wrong 'Paired' dbus property typo

related to TC-1712

Change-Id: Id4e3d29bf6a94982d007a2920c165dbcf2ac4151
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

correcly fill device_info struct when parsing device properties

Change-Id: Ibf35a2f7cab622b35035e0344f22919f84e6bc68
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

make sure priv->adapter exists before unref it when freeing gap agent

It fixes such error :
GLib-GObject-CRITICAL **: g_object_ref: assertion `G_IS_OBJECT (object)' failed

Also it makes no sense to reference priv->adapter pointer before setting it
to NULL. So unref priv->adapter it instead.

Change-Id: I0b27d0d16192e8e647ffc00fd9704a90700417b2
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

launch bt-service as 'guest' user instead of 'app'

'app' user is no more present on Tizen Common.
So we can run bt-service as 'guest' user for the moment
(e.g. TZ_SYS_DEFAULT_USER macro refers to 'guest' user)

Add a workaround in spec file to keep compatibility with IVI
that still needs to run bt-service by 'app' user even
if there is a 'guest' user.

Change-Id: I5d1e48cd5352852074f9eda220ffacf0405fbc63
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

add bt icon file to package when using notifications

Change-Id: I34cba413756af80927d3530c272b5a6722eefbd5
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

remove debian packaging folder

Change-Id: I6936d78544ef6149c7321caf3e60068518ef0aaa
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

remove syspopup dependency

Even if %bluetooth_frwk_libnotification OBS build macro is defined,
there was still some dependency on syspopup. So fix it.

Bug-Tizen: related to TC-1476

Change-Id: I5d9a7272f2176e86e4cffeea3730ec3365569d1f
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

Merge "Fix minor issues on adapter name property" into tizen

Fix minor issues on adapter name property

- catch 'Alias' dbus adapter property changed event
instead of 'Name' which is a readonly property.

- remove a useless hash table that brings a GLib-CRITICAL issue

Change-Id: I56d2924bb1c534117ea2ebfb1035e932fded02c2
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

Update deprecated libprivilege-control API functions.

Change-Id: I454bb0d61cf4bc52f132f92a80205b2595027a94
Signed-off-by: Marcin Lis <m.lis@samsung.com>

[Bluetooth-FRWK] change bug at event handler

Dbus property handler is changed.

Change-Id: I66f0fa8d726abb08115a5e9a3f3f7f4b647d1ba2
Signed-off-by: seungku kim <seungku.kim@samsung.com>

Remove hardcoded path for multiuser support

Bug-Tizen: PTREL-365

Change-Id: I1496814e5ae4b691f7a459a76b8959538f943715
Signed-off-by: Sabera Djelti (sdi2) <sabera.djelti@open.eurogiciel.org>

Add a condition on X for BuildRequire of UtilX.

Change-Id: If9dc0cc716d1237d2d6f2d648c4c0fe2f5d305dd
Signed-off-by: Ronan Le Martret <ronan@fridu.net>

Bluetooth-Frwk: modify the error audio UUID connection/disconnection

Normally, mobile/ivi will conn/discon headset/headfree device.
headset/headfree should be sink role.

Therefore, audio sink UUID is used to conn/discon.

Change-Id: I5393140f7bb23d61af57900e4647ec3b1a99451d
Signed-off-by: Wu Zheng <wu.zheng@intel.com>

Set bt-core status to match IVI

On IVI, bluetooth is enabled defaultly.

Therefore, bt-core doesn't know the BT status.
It results in that bt-core can't close BT.

The patch will fix the bug on IVI.

Change-Id: Ic16b0430fe0cb65856b84e0e2f9f903ecf800cbe

Merge "Modify the condition of checking BT enabled" into tizen

Merge "Fix build for x64 arch: Remove harcoded path" into tizen

Fix build for x64 arch: Remove harcoded path

Change-Id: I4310f1914a9adfbf42b10fd4b59715e87c858007
Signed-off-by: Jean-Benoit MARTIN <jean-benoit.martin@eurogiciel.fr>

Modify the condition of checking BT enabled

BT chip is enabled after IVI startup.

It is different with mobile.

It results in that the condition of checking BT enabled
need to be modified on IVI.

Change-Id: Ie4d41c763284b0faa4a063da680321ecffa790d4

Add the device lists of scanning to fix the crash issue

Add the device lists of scanning.
So that providing the device lists when UI is needed.

Bluetooth-frwk is stable when it own the list by itself.

If getting from Bluez. Bluetooth-frwk will crash sometime.
The root cause is  message format issues.

Thefefore, Bluetooth-frwk owner the device lists.
Bluetooth-frwk will be more stable.

Change-Id: If69def056791303fdb06f898259f78024607a5fd

Fix RegisterPlayer fail issue

Change-Id: Ief2fe9443c833811c37a4fc0a38469e6d8275fdd

Add the condition to register media player

IVI is AVRCP CT role.
Mobile is AVRCP TG role.

Therefore, add the condition to decide if
register media player or not.

Change-Id: Iebe3ec084f1424b3107dd58e5502cf3eb8be2dc3

Fix that bt-serivce can't find DBUS_SESSION_BUS_ADDRESS

Add the related env param when bt-service start.

The param still is needed.

After then, the bug can be fixed.

Change-Id: I5f05df3dc73c3683ddf735f6d813e9409eddd0de

Fix dbus service file

There are quite several errors in the dbus service file.

* it's not necessary to export DBUS_SESSION_BUS_ADDRESS, dbus activation
  will do that, and more important, it's a totally wrong path, what's
  the $UID?
* do not touch system-wide log policy because in fact we can't. These
  debug settings are managed by dlogctl.

Change-Id: I0522fed5d5053a49884dc78ab4054d2dfa780b0e
Signed-off-by: Chengwei Yang <chengwei.yang@intel.com>

Fix the build error in capi-network-bluetooth

After upgrading bluez 5.x version, some enum value and structure
variables were removed. Because of this there was a build break
in capi-network-bluetooth. So in now rollback removed values.
In later, capi-network-bluetooth will apply bluez 5.x values.
And then we can remove previous values.

Change-Id: Ia13e86310fffdf058d5db0ed1668e3dba49237b8
Signed-off-by: Dohyun Pyun <dh79.pyun@samsung.com>

Reland fixes to spec that were clobbered by recent downgrade

Change-Id: I043697347eb5a908ffa44488a8bc7a1c84bf5ba5

PTREL-140: Remove bogus Requires on eglibc

Change-Id: I49edf22fb8ad1d839676c3727d6fd7059acfd206
Signed-off-by: Rusty Lynch <rusty.lynch@intel.com>

Temp devicefound events are sent after DISCOVERY_STARTED

Before temp device found events are sent before DISCOVERY_STARTED.

It is not right. The patch will modify it.

The patch has been approved.

Change-Id: I54da96c21eca318f1f6e72d5694b1c8cae8216f0

Upgrade Bluetooth-frwk to match Bluez5.X

The following patches have been merged.

1.Modify the Tizen config files
2.Tethering: Remove register NAP service for ConnMan tethering
3.packaging: remove explicit requires on sys-assert
4.Fixes Tizen bluetooth device api fails to work for web apps TIVI-1275
5.Add a libnotify/gtk popup system to bluetooth-frwk
6.Start bt-service with systemd user with multi-user option
7.Add notifications in bt-service
8.Change the agent interface name to org.bluez.Agent1

Add notifications in bt-service

- use LIBNOTIFICATION_SUPPORT build option
- bt-service-agent-notification.c source code sends bluetooth notifications to
the notification daemon.

This fix is a part of TIVI-1690 (bluetooth pairing) since it just sends requests
to the notification daemon. Homescreen part should display the notification themselves.

Change-Id: Id82dee9de9199a37b700d27c0d0804c3e11bdd1a

Fix permission denied on open dev node /dev/rfcomm

The permission denied issue was due to early access to the /dev/rfcomm.
By adding a sleep time (10ms) and 3 retry, bt-service can now open this dev
node.

- Fix part of TZPC-3937

Signed-off-by: Victor Hakoun <victor.hakoun@eurogiciel.fr>

Start bt-service with systemd user with multi-user option

In multi-user environment bt-service cannot be launch by dbus-activation or
systemd-dbus activation.

- Add bcond_with multi-user option
- Add MULTI_USER_SUPPORT build option
- Remove rc.d/init.d in multi-user environment because done with systemd
user and  bluetooth-frwk-service scprit is for app user.
- Change dbus policy for multi-user
- Fix systemd unit file Install section
- Fix TZPC-4380

Signed-off-by: Victor Hakoun <victor.hakoun@eurogiciel.fr>

Add a libnotify/gtk popup system to bluetooth-frwk.

This is helping having Bluetooth popups with no OSP application dependencies (bt-syspopup and osp-launcher) and libnotify/gtk on the patform.

- Add bcond_whith bluetooth_frwk_libnotify in bluetooth-frwk.spec.
- Add requires to bluetooth-tools needed for bt-core/bt_core.c (calls to /usr/etc/bluetooth/bt-*.sh)
- Add compilation directive LIBNOTIFY_SUPPORT.
- Remove  bt-syspopup dependencies with LIBNOTIFY_SUPPORT directive.
- Modify bt-service/bt-service-agent.c in order to use bt-syspopup or libnotify (with compilation directive)
- Add bt-service/bt-popup.c to display libnotify/gtk popups.
- Add errno to bt-service/bt-service-rfcomm-client.c.

https://bugs.tizen.org/jira/browse/TWFD-3

Signed-off-by: Victor Hakoun <victor.hakoun@eurogiciel.fr>

Fixes Tizen bluetooth device api fails to work for web apps TIVI-1275

- Fixes issue where calling any bluetooth device web apis will result in seeing "The name org.projectx.bt was not provided by any .service files" error.

- Moving the dbus service files for bt-core and bt-service to /usr/share/dbus-1/system-services/ so the wrt can activite the daemon.

- Change DBUS_SESSION_BUS_ADDRESS to use UID instead of user name.

- Added dbus policy conf files for bt-core and bt-service to add permission to run on the system, since the default policy blocks everything.

Change-Id: Iceb9a4a507f71dfa781c102030801b3d9c8353c7
Signed-off-by: Jimmy Huang <jimmy.huang@intel.com>

packaging: remove explicit requires on sys-assert

Change-Id: I4c293a85a52c503f32c4106614e93360ef89fb6a

Tethering: Remove register NAP service for ConnMan tethering

When bluetooth adapter is added, NAP service register should be
moved to ConnMan to handle tethering.

Signed-off-by: Liu, Guoqiang <guoqiangx.liu@intel.com>

resetting manifest requested domain to floor

Update Changelog

Use macro %{_unitdir_user} unstead  of %{_libdir}/systemd/user in specfile

Update Changelog

Replace manifest with default floor

Fixed build and remove hardcoding of libdir

Fix rpmlint errors

Implement User Space SMACK

To control OSP privileges implement the codes

Change-Id: Id52f99b3cb1533d00e0ac6197e6aa822a13eeb63

Synchronize the manifest file with private git

Change-Id: I6a73608c01be48f890afc352a4542c6a991992b9

Fix the bug in HFP API

If we call bluetooth_telephony_audio_open API in next step 6,
the API call fails.

bluetooth-audio.c: bt_ag_open_sco(259) > ALREADY_DONE(0xffff8103)

1. bt_audio_initialize();
2. bt_ag_open_sco();
3. bt_ag_close_sco();
4. bt_audio_deinitialize();
5. bt_audio_initialize();
6. bt_ag_open_sco();

The cause was the headset state was not initailized.
(Because telephony library could not get the event
 - bt_audio_deinitailize also remove the event handler)

Change-Id: Ia724205e394dda8ee956805e951fb40385e623d2

Revert "Fix OSP TC fail cases in discovering"

This reverts commit 736fc1598dd52fe5553f3141b3c56e4538d59053

Fix OSP TC fail cases in discovering

There is a patch in bluez that BT don't restart the discovery.
Because of this code application have a delay by receiving
discovery finished callback. So it also had a problem.

Change-Id: I0dcb2208258d7c91540a45d3740f3d101516de2c

Increase the version number to upload

Change-Id: Ifb1c9b16b890c91f532293658677ae0a021adda5

Replaced deprecated Notification API

Change-Id: Ie71a07309b01c04e547c084c3b2956cbe57bfadb

Fix the JIRA issue (TREL-8)

[M0] Can't power on device any more after reboot device
     with Bluetooth/NFC enabled

Change-Id: I83b125011be7f32dc5a6eb1c3e75bd27353993c7

JIRA issue S1-13098 - Fix dbus add match rule overflow

Issue- In bluetooth telephony library dbus_bus_add_match was
used everytime BT activated. Once the number of match
reach the MAX limit crash was observed.

Fix- Removed multiple use of dbus_bus_add_match and also
added use of dbus_bus_remove_match in
bluetooth_telephony_deinit API.

Change-Id: Ifa0a38ab387f3076a199d167c9fc467fffaf64c6

Klocwork issue: NULL variable dereference

Change-Id: I132e6a2bfea560fc233b12073e09b7720374a907

Merge "Fix for x86_64 compatibility." into tizen_2.1

Remove 'smack request' lines

Change-Id: I815a2d8a8d0dbdd19919324fed2822c4d3ab80ac

Remove bt-core label access in the manifest file

Change-Id: I3e6751a447e5777236bc01a73cbffc151dd59d07

Fix for x86_64 compatibility.

- Adapt CFLAGS & LDFLAGS for x86-64.
- Fix hardcoding library path.

Change-Id: I36df4134d391b958f904ddbbd183e5f97e6820b9

Fix the issue that we can't see the dlog message

The environment variable did not set in dbus auto-activation.

Change-Id: I25ea708ab2f2ae7910918ea8eb06948046f3230e

Merge branch 'master' into tizen_2.1

Change-Id: Iaae67b4019fd0604bcd60a7df10fcc5189c0532b

merge with master

Tizen 2.1 base

Initial empty repository