| a5b34cb0 |
android/pts: Update PICS/PIXIT for GATT |
Marcin Kraglak |
11 years ago |
2 files, +2, -2 |
| 5b2c710c |
android/pts: HDP regression tests
Regression round for HDP against PTS 5.2 and Android 4.4.4 |
Sebastian Chlad |
11 years ago |
3 files, +9, -9 |
| a5c9529a |
android/doc: Update repository for Intel device build
Use repositories from github/01org/ |
Andrei Emeltchenko |
11 years ago |
1 file, +4, -4 |
| 9aaa38fd |
android/health: Shutdown MDL socket when freeing channel
PTS expects to close all data channels before sending delete
confirmation to peer. FD is passed over IPC to Android Bluetooth
application so it needs to be shutdown while freeing channel to
avoid races. |
Szymon Janc |
11 years ago |
1 file, +5, -0 |
| 790795f6 |
android/health: Call MDL shutdown after mdl_deleted_cb callback
Shutdown_mdl shutdown io channels and unrefs it. After that related
callbacks will shutdown or close and free its data. Due to timing
issue process_md_delete_mdl_req send delete confirmation response to
peer before properly shutting down all opened data channels. |
Ravi kumar Veeramally |
11 years ago |
2 files, +6, -25 |
| 8aa494f4 |
android/health: Remove mcl instance on mcl_disconnected call back
MCAP library is maintaining two sets of mcl list. On mcl_disconnected
call mcl instance is removed from regular mcl list and cached in
mcl_cache list. health.c doesn't maintain any cached mcls list. So
mcl_conn variable doesn't make any sense of not being connected on
mcl_disconnected call back. So unref mcl on mcl_disconnected and don't
do anything in mcl_unached callback. |
Ravi kumar Veeramally |
11 years ago |
1 file, +3, -14 |
| 72a76bb6 |
android/tester: Force remotes ACL disconnection
This is needed or else we would need to extend the timeout for a single
test case in tester framework and wait two more seconds for ACL to
disconnect. |
Marcin Kraglak |
11 years ago |
1 file, +15, -0 |
| 66e5db60 |
emulator/bthost: Add hci disconnect |
Marcin Kraglak |
11 years ago |
2 files, +14, -0 |
| 2515c432 |
android/tester: Wait for ACL disconnection before reconnect
We need the remote to drop its ACL before it tries to connect again. |
Jakub Tyszkowski |
11 years ago |
2 files, +16, -1 |
| 0cda9502 |
monitor/sdp: Make use of l2cap_frame_get* |
Luiz Augusto von Dentz |
11 years ago |
1 file, +8, -8 |
| b8e28074 |
monitor: Add helper functions to get data from L2CAP frames |
Luiz Augusto von Dentz |
11 years ago |
1 file, +109, -9 |
| b70c7057 |
android: Don't depend on libcap when building for Linux
Those are used only when building for Android target. |
Szymon Janc |
11 years ago |
1 file, +2, -0 |
| dfe72697 |
android/pts: Update MPS test result
Test TC_SRC_TG_HFAV_CLH_MD_BV_04_I pass when Music application triggers
music play/pause after it gets both: Button Pushed and Button
Released events.
Note that AOSP Music player triggers play/pause just after it gets
Button Pushed |
Lukasz Rymanowski |
11 years ago |
1 file, +2, -1 |
| b4ba03d2 |
android/pts: Update GAP PTS test results |
Marcin Kraglak |
11 years ago |
1 file, +11, -4 |
| f318a113 |
android/handsfree: Fix alignment |
Andrei Emeltchenko |
11 years ago |
1 file, +25, -25 |
| 2af0c998 |
android: Fix header include
Since Android KitKat we can use sys/prctl.h and sys/capability.h
as specified in the manual page. Fixes also build warnings:
...
warning: implicit declaration of function 'prctl'
[-Wimplicit-function-declaration]
if (prctl(PR_SET_KEEPCAPS, 1) < 0) {
^
... |
Andrei Emeltchenko |
11 years ago |
1 file, +1, -3 |
| b9fb944a |
audio/avdtp: Fix invalid reference counting
If the AVDTP session is terminated via any other mean than
'connection_lost()' (the only place that btd_device_unref() was called),
there was a dangling reference to the device.
This can be easily reproduced by suspending the host while using an A2DP
speaker. It has the very misleading effect of being unable to create the
device when the system resumes. |
Vinicius Costa Gomes |
11 years ago |
1 file, +1, -1 |
| df745234 |
monitor: Fix AVRCP rejected packet handling error
Rectified the handling of AVRCP rejected packets in btmon. |
Vikrampal Yadav |
11 years ago |
1 file, +6, -3 |
| 042b4fab |
android/avrcp: Register CT record
This fixes TC_SDAS_BV_03_I which requires CT record. |
Luiz Augusto von Dentz |
11 years ago |
1 file, +97, -8 |
| b8f25233 |
tools: fix headers for bcm43xx
PATH_MAX is defined in limits.h so include it explicitly.
This fixes the following compile error on musl libc (Alpine Linux):
tools/hciattach_bcm43xx.c: In function 'bcm43xx_locate_patch':
tools/hciattach_bcm43xx.c:313:14: error: 'PATH_MAX' undeclared (first use in this function)
char path[PATH_MAX];
^ |
Natanael Copa |
11 years ago |
1 file, +1, -0 |
| d59c6f08 |
shared: include limits.h for PATH_MAX
PATH_MAX is defined in limits.h so include it explicitly.
This fixes the following compile error on musl libc (Alpine Linux):
src/shared/util.c: In function 'util_get_dt':
src/shared/util.c:99:16: error: 'PATH_MAX' undeclared (first use in this function)
char filename[PATH_MAX];
^ |
Natanael Copa |
11 years ago |
1 file, +1, -0 |
| 212d3b0f |
android/bluetooth: Add debug log to select_device_bearer
This patch refactor a bit select_device_bearer function so we can get
debug log out of it. This log might be very useful in dual mode devices
scenarios |
Lukasz Rymanowski |
11 years ago |
1 file, +10, -3 |
| e73a2942 |
android/bluetooth: Fix bdaddr_type in pairing scenario
With this patch correct bdaddr type is used in ssp pairing scenario |
Lukasz Rymanowski |
11 years ago |
1 file, +5, -3 |
| 05fd7669 |
android/health: Fix NULL dereference
In a case get_app(), get_device(), get_channel() fail prevent
dereference of NULL pointer. Fixes clang warnings:
...
android/health.c:1980:15: warning: Access to field 'dev' results in a
dereference of a null pointer (loaded from variable 'channel')
queue_remove(channel->dev->channels, channel);
^~~~~~~~~~~~
1 warning generated.
... |
Andrei Emeltchenko |
11 years ago |
1 file, +5, -3 |
| 21fb4064 |
android/bluetooth: Update pairing status on pair complete event
So far BfA updates android bond state after link key(s) are
distributed. With LE pairing it might happen that no keys are
distributed e.g remote device has Initiator Key Distribution and
Responder Key Distribution set to 0. In such case link is encrypted
with STK and after its disconnected, pairing needs to be done again.
With this patch Android will get update about pairing complete just
after BfA gets pairing complete from the kernel. It solve issue withnot
updated Android settings application, which is hanging on "Pairing", in
case when no keys are distributed.
However, bacause Android does not see the difference between bonded /
paired devices and because Android does not allow to update its bond
state if FSM it is not in 'pending' state, BfA updates Android with
BONDED state on pair_complete with success. This is how we are doing
so far for bonding and non-bonding pairing.
Note: BfA will update its bonded state after link key(s) are
distribiuted. |
Lukasz Rymanowski |
11 years ago |
1 file, +5, -8 |
| 4f589213 |
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) |
Luiz Augusto von Dentz |
11 years ago |
1 file, +8, -2 |
| 907772ec |
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) |
Luiz Augusto von Dentz |
11 years ago |
1 file, +0, -1 |
| 4ce066c4 |
input: Fix crash when receiving event from uHID
uHID device is created only once so it is possible that the kernel send
events when there is no connection causing the following crash:
Invalid read of size 4
at 0x4EBC650: g_io_channel_unix_get_fd (in /usr/lib64/libglib-2.0.so.0.3800.2)
by 0x42C56A: hidp_send_message (device.c:160)
by 0x46EF1C: queue_foreach (queue.c:206)
by 0x42E3B0: uhid_read_handler (uhid.c:98)
by 0x46E82C: read_callback (io-glib.c:168)
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 0x40A972: main (main.c:607)
Address 0x70 is not stack'd, malloc'd or (recently) free'd |
Luiz Augusto von Dentz |
11 years ago |
1 file, +5, -0 |
| 717d36fb |
doc: Update Android tests coverage |
Szymon Janc |
11 years ago |
1 file, +2, -2 |
| 0c47d481 |
android/pts: Update PAN PICS and tests results
Specification Errata #3558 makes LLMNR support optional as LLMNR Draft
was never adopted by IETF. AOSP doesn't support LLMNR. |
Szymon Janc |
11 years ago |
2 files, +3, -2 |
| bc0e5488 |
android/pts: Updated GAP test result
Updated test result and instruction for TC_BOND_BON_BV_04_C |
Lukasz Rymanowski |
11 years ago |
1 file, +4, -4 |
| 96eca81c |
android/pts: Update GAP result for handling directed advertising
Previously used connection procedure was to slow for remotes preforming
directed advertising. This issue is no longer valid on the latest kernel
and auto connection support added recently to the daemon. |
Jakub Tyszkowski |
11 years ago |
1 file, +0, -1 |
| aebcf9b5 |
android/pts: Update PTS files for L2CAP
PICS and PIXITs checked against PTS 5.2. Regression round on PTS
5.2 against Android 4.4.4. Interim results. |
Sebastian Chlad |
11 years ago |
3 files, +32, -27 |
| ef74914b |
android/pts: Update GAP PTS test results |
Marcin Kraglak |
11 years ago |
1 file, +25, -9 |
| f884c34d |
android/bluetooth: Fix device found notification
If device is dual mode and can be seen on both LE and BREDR, then
bdaddr_type in device struct keeps LE address type of that device.
During discovery, we should take current bdaddr_type to decide about
passing that device up to application or not. Using dev->braddr_type might
be misleading
This patch fix scenario when remote device is known as LE device and
recently has been found on inquiry seesion as DUAL mode device. In such
case, based on braddr_type and eir flags we could incorectly skip
new device notification for Android framework. |
Lukasz Rymanowski |
11 years ago |
1 file, +4, -3 |
| 62526c71 |
android/health: Simplify search_cb failure case |
Ravi kumar Veeramally |
11 years ago |
1 file, +1, -4 |
| e32d55d4 |
android/health: Add some information messages |
Ravi kumar Veeramally |
11 years ago |
1 file, +7, -2 |
| e21de873 |
android/health: Fix connect channel failure case
There is a possibilty of created channel is being added to queue,
but on failure case it does channel free but not removed from queue. |
Ravi kumar Veeramally |
11 years ago |
1 file, +1, -0 |
| 69ccc52e |
smp-tester: Fix invalid data test to expect a disconnect |
Johan Hedberg |
11 years ago |
1 file, +6, -2 |
| 5fd1e9e5 |
smp-tester: Add support for expecting specific HCI commands |
Johan Hedberg |
11 years ago |
1 file, +50, -0 |
| 49f19d04 |
smp-tester: Add support for multiple test completion conditions |
Johan Hedberg |
11 years ago |
1 file, +29, -3 |
| d32c25e0 |
android/pts: Update PBAP results |
Jakub Tyszkowski |
11 years ago |
1 file, +1, -1 |
| b6bfed3c |
android/bluetooth: Fix getting RFCOMM channel instead of L2CAP PSM |
Johan Hedberg |
11 years ago |
1 file, +1, -1 |
| eec3ccf6 |
android: Fix memory leak
protos gets allocated and needs to be freed |
Andrei Emeltchenko |
11 years ago |
1 file, +7, -2 |
| e64c877c |
monitor: Fix L2CAP fragment tracking for duplex data streams
The fragmentation state is independent in each direction of the data
flow. We should therefore track outgoing and incoming fragmentation
state separately. This patch extends the state tracking table with a
second column to fix this. |
Johan Hedberg |
11 years ago |
1 file, +28, -28 |
| 0fead97e |
shared/att: Handle disconnects.
This patch adds disconnect handling to bt_att, in which
io_set_disconnect_handler is used to set up a handler which cancels all
pending and queued ATT operations, marks the bt_att structure as invalid
and notifies the user via a specialized callback which can be set using
bt_att_set_disconnect_cb.
Once the bt_att structure is invalidated, either due to a timed-out ATT
protocol request/indication or a disconnect, it now destroys the
underlying io structure. |
Arman Uguray |
11 years ago |
2 files, +67, -15 |
| 63df3a1c |
shared/att: Use io_set_close_on_destroy.
It's better to use io_set_close_on_destroy as opposed to keeping a
special "close_on_unref" flag in bt_att. This not only achieves the
exact same result, but also allows the code to automatically close the
file descriptor from several places by simply calling io_destroy. |
Arman Uguray |
11 years ago |
1 file, +1, -7 |
| 77c0e84f |
shared/io-mainloop: Increment the ref count inside io_callback.
Incrementing the reference count before invoking the event handlers and
decrementing it thereafter allows io_destroy to be safely called from within
the event callbacks. This patch achieves that. |
Arman Uguray |
11 years ago |
1 file, +5, -0 |
| 8587edd4 |
android/bluetooth: Minor coding style fixes |
Johan Hedberg |
11 years ago |
1 file, +10, -10 |
| da3aae03 |
android/pts: Update IOPT test results |
Grzegorz Kolodziejczyk |
11 years ago |
1 file, +3, -1 |