Fix dbus policies

If unknown process owns these well-known name, unexpected deny can be occured.
So, subdivide deny policy.

Change-Id: I8e6cc73d80d60d348e2b510cabc77c73a6cb32ca

Remove unnecessary connection getting functions

Remove duplicated function _bt_gdbus_init_system_gconn()
Remove duplicated function __get_gdbus_connection()
Rename _bt_gdbus_get_system_gconn() to _bt_get_system_private_conn()
Rename _bt_init_system_gdbus_conn() to _bt_get_system_shared_conn()
Add 'static' keyword for static functions

Change-Id: I85571703b1c9a74365c83dc9d701cb418963ae59

Return errors without retry if bond info. already exists or it's in progress

  If create_bond is called in case GATT is connected and it's already bonded,
  then bond info. is removed and GATT is disconnected unexpectedly.

Change-Id: I078da398322f68522e427c7b7f18a8fd8e229f91
Signed-off-by: Deokhyun Kim <dukan.kim@samsung.com>

Apply send_destination_prefix policy#3

Change rfcomm connection from private to shared

Change-Id: I59924564c134a7d4f7ea13a4c25ffd41345792d8

Merge "Enable FD based GATT client data transfer" into tizen

Apply send_destination_prefix policy#2

gdbus closing logic is not needed anymore

Change-Id: If529a348bee88dfe413fd936fcf07296eb6ae8d0

Enable FD based GATT client data transfer

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

Apply send_destination_prefix policy#1

default deny/allow options are needed for policy checker

Change-Id: I1feac171c04fc7ec8f63a68a9499149d8c5a9528

Fix the bug for the OPP server progress callback's file name

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

Add new method to disconnect device

Change-Id: Ia0dd37ce4cd13b2666c3e8f6f112ed0b8321d4b6
Signed-off-by: Deokhyun Kim <dukan.kim@samsung.com>

Fix svace issue(40981)

Change-Id: I2fa8b274e275a8aa949c274016b2ba8cfa6c7acb
Signed-off-by: Deokhyun Kim <dukan.kim@samsung.com>

Merge "Improve efficiency of using g_array_index" into tizen

Enable the connection request event for custom obex server

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

Improve efficiency of using g_array_index

Change-Id: Ia2c56e65d5900f67d0089290663cfb0c95bb6a85
Signed-off-by: Deokhyun Kim <dukan.kim@samsung.com>

Correct the privilege check logic for MTU size APIs

Change-Id: I509d0967b16b9ca17a978ecffc914184b744557d

gatt-service : Fix some minor bugs

[Problem] init -> register service -> deinit -> init -> register service
-> start server : service is not added.
There are some warning about reference to invalid GOBJECT.
[Cause & Measure] Remove pending count. Unref GOBJECT after check of
validation.

Change-Id: I4829505ebac3e3c0668aa2a386de3085163d0b8d

Fix invalid read

[Problem] There is invalid read report from valgrind
[Cause & Measure] The node was accessed after it was freed. This patch
 frees current node after moving to the next node so that invalid read
 issue has been resolved.

Change-Id: I15eae8c16c8f215aee30fd33ac8f3aadea73b4eb

Improve inefficiency

Change-Id: Icd2b33c68d384680e24eec64de9256efb60c1084
Signed-off-by: Deokhyun Kim <dukan.kim@samsung.com>

Fix dlog format error

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

Correct the privilege check logic for MTU size APIs

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

Modify dbus policies

- Remove org.projectx.bt_event policy
- Apply default deny for org.projectx.bluetooth.share
- Add send_member for org.tizen.csd.Call.Instance

Change-Id: Ia50f24d807b0f180bcd74f02886f28cc57474d5e

Modify gdbus proxy flag to G_DBUS_PROXY_FLAGS_DO_NOT_LOAD_PROPERTIES

G_DBUS_PROXY_FLAGS_NONE causes some unnecessary error log

Change-Id: I8be66086d58431bb9894f30d7cbddc4f3ac76a63

Merge "Match the correct privilege for RFCOMM listen and accept API" into tizen

Match the correct privilege for RFCOMM listen and accept API

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

Check le_adv_slot before access

 - bt-service crashes by invalid memory access when _bt_is_advertising is called
  before adv info is initialized.

Change-Id: I75b4efb9c4ccb209860453e467f9847669cc7624
Signed-off-by: Deokhyun Kim <dukan.kim@samsung.com>

Remove g_bus_own_name() logic in bt-telephony

It is not good to call g_bus_own_name() in api layer, it causes
service name of sender is changed to org.tizen.csd.Call.Instance.

This interface has been used on bluetooth-agent and it can call
method on this interface by using sender_name instead of service_name.

Change-Id: I6f77e23ee15b9b1f9e4a4ce53f4993e852006bce

Migrate insmod logic into bt-stack-up.service

insmod logic should be executed before bt-stack-up.sh logic. But,
insmod logic has been executed after bt-stack-up.sh logic on ARTIK530.
This patch migrates insmod logic into bt-stack-up.sh and creates bt-stack-up.service
This patch should be merged with bluetooth-tools.

Change-Id: I15e3ab996379600173e441effb9d52577ae3203b

Merge "Revert "Fix incorrect interface name and improve dbus policy"" into tizen

Revert "Fix incorrect interface name and improve dbus policy"

This reverts commit fb6c7048cb2a150aebcef0e2821c369e46c5a9ad.

Change-Id: Ia78dd8b41f341441d4c3cbb15bd1450e24dbaf22

Fix profile check logic to distinguish ivi and iot

Change-Id: Iaa8e0786787bfba81987d6fdb29d2a95dd867ab9

Invoke callback function after all internal work is done

 - Prevent to access freed service after GATT is destroyed in callback function

Change-Id: I8aa33c48cdcf246578552a58aaa2cad7a94a589c
Signed-off-by: Deokhyun Kim <dukan.kim@samsung.com>

Revert "Remove and modify for some dbus policies"

This reverts commit b493a680894d58efc1879d90be656c6fcdda9c50.

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

Remove and modify for some dbus policies

- org.projectx.bt_event
 no need policy for using signal
- org.projectx.bluetooth.share
 bluetooth-share owns this service
- org.tizen.csd.Call.Instance
 user owns this service and called by network_fw

Change-Id: I3727e144ffcae9ad9ff12748c7ac5b51c08e0486

Merge "Fix incorrect interface name and improve dbus policy" into tizen

Disable GATT Client fd based function

After applying the security patchset, we will enable this feature.
To pass the web manaul TC, we disable this feature in now.

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

Fix incorrect interface name and improve dbus policy

Dbus -> DBus
Objectmanager -> ObjectManager

Change-Id: I97a05856ff7b8357aeb92d595d5d4aacfc786bee

Fixed the return status for indicate property

Return status for indicate property of characteristic
is fixed in set characteristic changed value cb for
TFIVE-14433 TFIVE-14439 issue

Change-Id: Ia3ef28961ef71fb81e7d74bce457e54a99ba3cb5
Signed-off-by: Amit Purwar <amit.purwar@samsung.com>

Modify to path activation for hciconfig command

Change-Id: Ia091b30464fc2cf1fef96d65c95ff3b0ea296312

Fix memory double free(crash) issue

g_variant_get_string returns the constant string whereas
g_variant_get_dup_string returns a newly allocated string.
It should not be freed.

    at /usr/src/debug/bluetooth-frwk-0.3.0/bt-oal/bluez_hal/src/bt-hal-adapter-dbus-handler.c:525

Change-Id: I3d8f6bc0eec3d7736a263edbb47c3f3deaee4cef

Added the GATT server disconnect

Change-Id: I2c38505d92d8610801670cd063d31fbf7f8df4b4
Signed-off-by: Amit Purwar <amit.purwar@samsung.com>

Merge "Prevent Issue fixes" into tizen

Prevent Issue fixes

Prevent Issue fixes :
- 1026071
- 1026065
- 1026059
- 1026057
- 1026047
- 1026042
- 1026038

Change-Id: I4517e99e89e5ad5a1fefe07fc2263c27587e61ab
Signed-off-by: Abhishek Chandra <abhishek.ch@samsung.com>

removed gatt service info from list during gatt cleanup

Change-Id: I1dbe49424fcf20047c2d86ec6e234929e05f6de2
Signed-off-by: Amit Purwar <amit.purwar@samsung.com>

Add "Not Supported" check logic for haltests

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

Modify bt-oal initialize code readably

Change-Id: Ic35ea6d65139f41f63caccd185474ad598f0d2d4

Remove duplicated check logic and incorrect return type

Change-Id: I5be38b4424e2f8418d06af82c63b4de90b062651

Merge "Fix the coding style errors" into tizen

Merge "Fix the build error in aarch64" into tizen

Merge "TFIVE-12959: fixed scan response and adv data length" into tizen

Merge "TFIVE-12640: fixed the size of handle string" into tizen

TFIVE-12959: fixed scan response and adv data length

Change-Id: Ib1290516f289f1cd4a6bc23cb1b8579a3f4d1b5d
Signed-off-by: Amit Purwar <amit.purwar@samsung.com>

Fix the coding style errors

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

Fix the build error in aarch64

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

Add Werror option in CFLAG

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

Fix the build warnings and typo error

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

Remove unused local variable and codes -  wait_for_async( )

Change-Id: Ib9c2e29d3ee4f6e4ca3240e3f8633fff15655021
Signed-off-by: Taesoo Jun <steve.jun@samsung.com>

TFIVE-12640: fixed the size of handle string

Change-Id: I944c5d7d6e45d96b88be179d0c57113afd87cc05
Signed-off-by: Amit Purwar <amit.purwar@samsung.com>

Remove duplicated logic

Remove incorrect comment

Change-Id: I40ad8a799e2c90b6af5d580489dfb5f26ed179f1

Fix return-type mismatch issue

Change-Id: I07ed4db7b8f6395ba9b532ceaa744d9148de8e47

Removed duplicate g_variant_get call

g_variant_get is called to extract
address , offset for read value request
in __bt_gatt_char_method_call twice .
Removed the redundant call.

Change-Id: Id5694e6437cef8ac016e701964b9c2a288161db6
Signed-off-by: Abhishek Chandra <abhishek.ch@samsung.com>

Add HFP HF API : bt_hf_request_call_status_info_list()

Change-Id: I4ddd7cfadb91a8ef39231cd927e5fdb0a23e152e

Stop sending notification data on fd from BT_GATT_SERVER_UPDATE_VALUE

Change-Id: Ifad181ac984b8209495df0f59ec00dc6e9cf9117
Signed-off-by: Avichal Agarwal <avichal.a@samsung.com>

Fixing BLUETOOTH_EVENT_GATT_CHAR_VAL_CHANGED for notify

using correct  stucture parameter value.

Change-Id: I6a59d53cbe9e1f003ced8823cfa953cd253f44f4
Signed-off-by: Avichal Agarwal <avichal.a@samsung.com>

Handling characteristics properties Indicate and Notify functionality

in case of property Indicate StartNotify will be called
in case of property Notify AcquireNotify will be called

Change-Id: Ice299d23be8e68c44f6b0a6746bafa8169529102
Signed-off-by: Avichal Agarwal <avichal.a@samsung.com>

Gatt server ReadValue parsing fix

Gatt server ReadValue parsing was always returning
NULL as addr was always NULL when passed to
__bt_find_remote_gatt_client_info  ,To fix this
address and request id is extracted from the variant
parameter.

Change-Id: If4b19109e893662b66663013c0051111461cc9bd
Signed-off-by: Abhishek Chandra <abhishek.ch@samsung.com>

Add AVRCP control volume up / down method

Change-Id: I3c25106ef79181ed699123a703a0b50300c7853a

Fix oal_bt_deinit crash issue

Change-Id: I67bd8b537358e1f22f33490320d0b4e1e691bb23

Merge "Add dbus policy for GATT service" into tizen

Add dbus policy for GATT service

No send_destination because private connection does not have well-known service name

Change-Id: I599f17e562207387e9c61815f31e19e39f3000dd

resolving  merge conflict with StartNotify

Using same structure as defined for
signal BLUETOOTH_EVENT_GATT_CHAR_VAL_CHANGED

Change-Id: Iac3a18c7b76105e765892ba0e8d6d33115cd1e68

Added feature : notification of GATT char Changed Value

Change-Id: I387bb74f169da38481ee4a03a65817f0ec0e89f9
Signed-off-by: Amit Purwar <amit.purwar@samsung.com>

BLE Gatt Client Socket write and notify implementation

Client can write data using file descriptor on Characteristic
with property “write-without-response” . AcquireWrite
Method handler of bluez called to get write file descriptor.

Client can read notifications from File descriptors
on characteristics with property “Notify” .AcquireNotify
method handler of bluez called to get the notify file descriptor.

Change-Id: Idb06a14ae5368540f796d78e666d6896739c5579
Signed-off-by: Avichal Agarwal <avichal.a@samsung.com>

BLE Gatt Server Socket notify implementation.

Server will notify data using file descriptor on Characteristic
 with property “NOYIFY”. AcquireNotify Method handler of is
implemented in HAL,which provide file descriptor to bluez.

Change-Id: I7b567bd3ede9c1dfdb022d1af286758d93a59c72
Signed-off-by: Avichal Agarwal <avichal.a@samsung.com>

Remove duplicated dbus policy

- Some policies are existing in bluez already

Change-Id: I7d71d53f885a8ceb793bbe6a2af484da5fd7cd70

BLE Gatt Server Socket write implemetation

Server with characterstics property 'write-without-response' will receive the data
on socket file descriptor.AcquireWrite method handler is implemented in OAL

Change-Id: I4b8134b8eda87f775ebf72b5e98ce6753e6a9b84
Signed-off-by: Avichal Agarwal <avichal.a@samsung.com>

Modify dbus policies based on default deny

Change-Id: I6df07a1032f1d3cd58ea68ae87323f9095143619

PBAP: seperate a PBAP connect error cause in detail

Change-Id: I45c97b3c5a79103c0ab8af8abf2ccc65016714fa
Signed-off-by: Deokhyun Kim <dukan.kim@samsung.com>

Modify the package name for bluetooth-haltests

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

Modify OAL hal test binary name

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

Notification was faling as Gatt serv info was NULL

String pointer were compaired for the path in
__bt_gatt_get_service_state , which was causing the
API to return NULL.

Fix : string compairition for service path is used.

Change-Id: I5ec7d768f9a63780cbc18ef1f8b00c31542c3adc
Signed-off-by: Abhishek Chandra <abhishek.ch@samsung.com>

Remove the coverage check option for OAL library

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

Fix memory leak: cb_device_properties in oal-device-mgr.c

Change-Id: Ie5cec6a51ea2e5b20d24df202d727eae31680a79
Signed-off-by: Taesoo Jun <steve.jun@samsung.com>

BT 5.0: Implement Is feature support API's

This patch implements FRWK API's which check
whether adapter supports LE 2M PHY and
LE CODED PHY features or not

Change-Id: I31284348e3c18f2b2122883c7b83b5d7486dfb62
Signed-off-by: Anupam Roy <anupam.r@samsung.com>

Merge "Implement local LE features adapter property" into tizen

Implement local LE features adapter property

This patch handles Adapter's local LE features property
which will indicate whether adapter supports LE 2M PHY
and LE CODED PHY. This property is emitted by bluetoothd
as adapter's property changed event and handled in
bluetooth HAL layer. LE features are further propagated to
bt-service via new OAL event OAL_EVENT_BLE_LOCAL_FEATURES.
LE features will be finally saved in bt-service daemon.
Application can read these values directly from bt-service.

Change-Id: I9e1b617851693b7bab038a9f10861c6bf3ba8835
Signed-off-by: Anupam Roy <anupam.r@samsung.com>

device : Handle "Does Not Exist" error in get_ida

[Problem] Even though LE RPA device is not paired, operation failed
 error code (BLUETOOTH_ERROR_INTERNAL) is reported.
[Cause & Measure] Check error->message and report BLUETOOTH_ERROR_NOT_PAIRED
 when LE RPA device is not bonded
[Checking Method] N/A

Change-Id: I726c389c8f2cc0d9458a803ac4bb3463a9e25b60

Enhance BT On scenario in power off

[Problem] BT is not turned on when flight mode is unset during power
 off. This is not usual case but it happens sometimes in wearable model.
 Specific case is described in "Checking Method".
[Cause & Measure] If BT on is requested during power off, just update
 vconf of bt state. So that in next boot, BT is turned on.
[Checking Method] BT On -> Cool down (LimitAction) / Flight mode on ->
 BT Off -> Cool down (ShutDown) -> Flight mode off -> BT On, but it's not done
 -> Power off -> Power On -> BT should be turned on

Change-Id: I63775e38b09b77d7eb7c4cf9ae8e8445902b793a

Merge "Add new error codes." into tizen

Added Event Handling in HAL

[Problem] Start/Stop notify ,Gatt Connected and
disconnected event was not received till bt-service .

[Solution] Added Event handling for Notification change,
Gatt Connected and Disconnected was missing in the
HAL layer .

Change-Id: I72112ace1ed56d711e5a95e799cf2d27989f38ae
Signed-off-by: Abhishek Chandra <abhishek.ch@samsung.com>

Add new error codes.

 - BLUETOOTH_ERROR_NO_DATA
 - BLUETOOTH_ERROR_AUTHORIZATION_REJECTED

Change-Id: I8637ca558bcc0856008d357de96edd80f4665b7c
Signed-off-by: Deokhyun Kim <dukan.kim@samsung.com>

Verify 16 testcases for BT HAL (OAL layer)

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

Add HAL testcases for oal adapater

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

Merge "Modified dbus permission for obex" into tizen

Fix Coverity issues in bluetooth-frwk

859540, 859527

Change-Id: Ic453cb1dd6094daca0aabc022660a304e766cd0e

Modified dbus permission for obex

Change-Id: Ibb789a70fc7cf0f7cd265bc6e4d440fcbda4bbd6
Signed-off-by: Amit Purwar <amit.purwar@samsung.com>

Merge "Add HAL testcases for oal manager" into tizen

Fix : Bluetooth log is not removed on user binary

[Problem] Bluetooth log is not deleted on user binary.

[Cause & Measure] Directory owner / group is modified manually. It
 causes inconsistency with system setting. The base directory which is
 passed from dump daemon has setuid bit. So the files and directories
 created by bluetooth script have the same group id to the base
 directory. So bluetooth script doesn't need to change ownership of
 files and directories. This patch removes chown part from script and
 adds write permission to group.

[Checking Method] *#9900# on User binary

Change-Id: Ibfde6fd7a37483dcdd3b0c4f8872dbb436095870

Fix : Bluetooth disabled event is sent twice

[Problem] Bluetooth disabled event is sent twice when it is turned off
 by entering flight mode. It causes applications malfunction.
 For example, normally disabled event is received by weconnd after all GATT
 services have been removed. But in problem situation, the first
 disabled event happens too early so bluetooth resources (GATT services)
 remain at that time. It makes weconnd enter to strange condition (BT off but
 there are some GATT services). And after bluetooth resources are
 cleared, the second disabled event happens again. But this event is not
 processed by weconnd because it is same state to previous one.

[Cause & Measure] The first event was sent in wrong context (at Powered
 property change). This patch makes the event happen only when
 bluetoothd is terminated. So that, only one disabled event is sent
 after all bluetooth resources are cleared.

[Checking Method] Check disabled event when entering to flight mode.
 This patch is tested under below situation also.
 1. BT on / off
 2. Flight mode on -> Flight mode off
 3. Flight mode on -> BT on -> Flight mode off
 4. Recovery (TxTimeout or Hardware error by sending dbus signal)

Change-Id: Ief9dff1b4074143e1cab41b688cc960f0c73760e

Add HAL testcases for oal manager

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

Add the base code for Bluetooth HAL testcases

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