| 220723b2 |
Fix remove emitting Paired=false when deleting device
When adapter is requested to remove device, device's property
Paired=false is emitted after the adapter notifies clients that
list Devices is empty. Some clients get confused.
Emitting property was introduced in clean-up patch
c92a4af85f480c16f3ff017783c821304c899ce5. Since such notification
does not have practical meaning, it is better to remove it. |
Dmitriy Paliy |
14 years ago |
1 file, +2, -1 |
| bde01ad6 |
Use medium security level when reconnecting
For GATT based profiles encryption is mandatory. Proximity Monitor
shall support LE Security Mode 1 and Security Level 2. Over basic rate,
switch the security level to medium for testing purpose only. |
Claudio Takahasi |
14 years ago |
1 file, +2, -2 |
| 553ebe6d |
Fix invalid free when stopping adapter
This patch fix an "invalid free" error when the adapter is powered off
with an active discovery session. Error happens because session_remove
function removes the elements from the list also. Partial valgrind log:
Address 0x6012a00 is 0 bytes inside a block of size 16 free'd
at 0x4C27DCC: free (vg_replace_malloc.c:366)
by 0x4E927AC: g_slist_remove (in
by 0x19F788: session_remove (adapter.c:689)
by 0x19F82A: session_free (adapter.c:708)
by 0x4E92CD6: g_slist_foreach (in
by 0x4E92CFA: g_slist_free_full (in
by 0x1A3ADD: btd_adapter_stop (adapter.c:2491) |
Claudio Takahasi |
14 years ago |
1 file, +13, -7 |
| 3169c1be |
headset: Fix missing check for non-existent pending message |
Johan Hedberg |
14 years ago |
1 file, +2, -1 |
| 26bc2d99 |
Remove redundant empty line |
Johan Hedberg |
14 years ago |
1 file, +0, -1 |
| 9390bf46 |
AVDTP: Fix state if authorization fails |
Johan Hedberg |
14 years ago |
1 file, +1, -0 |
| d4a6604e |
Fix agent fall-back mechanism
We can't change the agent for a request internally within agent.c since
externally code may depend on a specific agent being used so that
agent_cancel work properly. This patch exports the fall-back behavior
from agent.c into device.c. |
Johan Hedberg |
14 years ago |
2 files, +59, -64 |
| bd48d4b7 |
AVDTP: Refactor audio_device_cancel_authorization usage |
Johan Hedberg |
14 years ago |
1 file, +17, -11 |
| 2b4d684e |
build: Use private copy of Bluetooth library for audio support |
Marcel Holtmann |
14 years ago |
1 file, +4, -3 |
| 47209516 |
Fix crash when update service record with an invalid XML
If we pass an invalid xml to sdp_xml_parse_record(), then it returns
NULL. Further we are passing the this NULL pointer to the
sdp_record_free(), which leads to invalid memory access. |
Syam Sidhardhan |
14 years ago |
1 file, +0, -1 |
| 3c4e447b |
Add valid range descriptor uuid |
Santiago Carot-Nemesio |
14 years ago |
1 file, +1, -0 |
| 082dba70 |
Fix magic numbers for local/remote name length |
Claudio Takahasi |
14 years ago |
1 file, +6, -6 |
| c13b94f2 |
Remove unneeded headers included in storage.c |
Claudio Takahasi |
14 years ago |
1 file, +0, -4 |
| 47c48729 |
Remove not referenced function in storage.c |
Claudio Takahasi |
14 years ago |
2 files, +0, -30 |
| 2a39f5f3 |
Remove create_file calls for read operations
This patch fixes some read only functions in storage.c removing
create_file function calls. |
Claudio Takahasi |
14 years ago |
1 file, +0, -8 |
| 6be26a3f |
Remove storing device type
Device type doesn't need to be stored since GATT services information
exported through basic rate can be retrieved from the SDP records.
Device "type" is also a wrong expression to represent the operation
mode: over which transport GATT service is being exported. |
Claudio Takahasi |
14 years ago |
4 files, +0, -71 |
| 6a6da5de |
Fix allocation of attribute values
Now that pointers to attribute are passed as reference to functions
we cannot have they change during run time, what g_try_realloc()
could do. |
Vinicius Costa Gomes |
14 years ago |
2 files, +25, -10 |
| 4dc449c1 |
avrcp: get/set three-byte company-id |
David Stockwell |
14 years ago |
1 file, +26, -9 |
| 232ba155 |
avrcp: fix handling of metadata item 0x7
Metadata field number 0x7 should be the total playing time of the track
(TrackDuration) in msec, not current position within track. |
David Stockwell |
14 years ago |
1 file, +6, -12 |
| b0de2a7b |
avrcp: use LAST element on media_info_id enum |
David Stockwell |
14 years ago |
1 file, +2, -1 |
| f2ac0034 |
avrcp: fix overwrite of number of attributes
The response of GetCurrentPlayerApplicationSettingValue expects the
first operand to be the number of attributes in response. Since we start
with len=0, we were overwriting this number with the value of the first
attribute.
Also use g_memdup instead of g_malloc + memcpy. |
Lucas De Marchi |
14 years ago |
1 file, +5, -7 |
| 66b9d4c7 |
avrcp: fix missing error code |
Lucas De Marchi |
14 years ago |
1 file, +3, -1 |
| 71a84481 |
AVRCP: change debug to better reflect packets protocol/profile |
Luiz Augusto von Dentz |
14 years ago |
1 file, +5, -5 |
| a762de7a |
serial: fix DBus message reply
If reply is NULL we have a crash. |
Gustavo F. Padovan |
14 years ago |
1 file, +2, -4 |
| 5ca964ac |
serial: add Serial.ConnectFD()
It's similar to Serial.Connect() but returns the actual RFCOMM file
descriptor instead of creating a device in /dev |
Gustavo F. Padovan |
14 years ago |
2 files, +40, -1 |
| 659190bf |
build: Link all tools with local copy of Bluetooth library |
Marcel Holtmann |
14 years ago |
1 file, +30, -30 |
| 99f32ae9 |
AVRCP: move handling of vendor dependent PDU from control.c to avrcp.c
This helps to isolate AVRCP PDU handling and MediaPlayer interface. |
Luiz Augusto von Dentz |
14 years ago |
10 files, +1720, -1647 |
| 8d18c289 |
AVRCP: split AVCTP specific code from control.c
The code is moved to avctp.c to simplify control.c |
Luiz Augusto von Dentz |
14 years ago |
6 files, +1244, -772 |
| c612aee0 |
AVRCP: rename avrcp_spec_avc_pdu to avrcp_header
Since old avrcp_header got renamed to avc_header now we can use
avrcp_header for AVRCP PDUs. |
Luiz Augusto von Dentz |
14 years ago |
1 file, +22, -22 |
| 222b464a |
AVRCP: rename avrcp_header to avc_header
AVCTP carries AV/C packets/PDUs not AVRCP as avrcp_header suggests. |
Luiz Augusto von Dentz |
14 years ago |
1 file, +48, -48 |
| 09984fd9 |
AVRCP: use a vtable to simplify PDU parsing/handling
This simplify a bit the handling by introducing common checks before
calling the handler callback, it is also much easier to add/remove
new PDUs in this way. |
Luiz Augusto von Dentz |
14 years ago |
1 file, +124, -197 |
| a48d5487 |
Add test for serial proxy and serial proxy manager |
Bruno Dilly |
14 years ago |
2 files, +67, -3 |
| e327f964 |
Return EALREADY if the adapter is already up/down |
Claudio Takahasi |
14 years ago |
1 file, +2, -2 |
| 11ddc90c |
Serialize powering up/down in maemo6 plugin
This patch fixes wrong adapter state when using Management interface.
When bluetoothd starts, the adapter should be switched to the state
informed by Maemo6 MCE. In the current implementation, Maemo6 plugin
sends set powered command when the adapter is still powering up, the
last command sent fails(kernel returns EALREADY). |
Claudio Takahasi |
14 years ago |
1 file, +68, -68 |
| 1851670a |
Fix adapter->up declaration
Minor code convention fix. This patch fix "up" variable declaration in
adapter structure. Convert "up" to gboolean making it compliant with
its usage. |
Claudio Takahasi |
14 years ago |
1 file, +2, -2 |
| 2784b03a |
Fix minor coding-style issue |
Johan Hedberg |
14 years ago |
1 file, +1, -1 |
| 97fa351a |
Prevents sending Write Local Name when not needed
Fix unneeded Write Local Name command being sent when bluetoothd starts
or switching the adapter on. Using an array of chars it is not possible
to distinguish if there is a valid value to be set since empty is a
valid value. |
Claudio Takahasi |
14 years ago |
1 file, +16, -11 |
| 7aeeea79 |
Fix loop when setting adapter name
When management interface is enabled, name changed event comes when the
adapter is initialized as consequence of the Read Local Name. Use the
same function to set and handle event causes looping when bluetoothd
starts if the name stored in the controller is different from the name
provided by the adapter name plugin.
Splitting the adapter_update_local_name also fix the PropertyChanged
(for Name) signal being sent before AdapterAdded. |
Claudio Takahasi |
14 years ago |
5 files, +32, -28 |
| 4211e255 |
Remove not referenced function |
Claudio Takahasi |
14 years ago |
2 files, +0, -6 |
| d1bd2e58 |
Remove leftover function declaration |
Claudio Takahasi |
14 years ago |
1 file, +0, -1 |
| 8654b1bc |
Remove unneeded header include |
Claudio Takahasi |
14 years ago |
3 files, +1, -12 |
| 2c696825 |
Remove forward declaration |
Claudio Takahasi |
14 years ago |
1 file, +16, -19 |
| 362cef3d |
Fix ATT disconnect callback not being called
GAttrib disconnect function needs to be set after discovering the
primary services if the connection callback list is not empty. After
creating a device the connection can stay up, for this scenario the
registered disconnection callbacks are not being called. |
Claudio Takahasi |
14 years ago |
1 file, +3, -1 |
| c0930456 |
Write Link Loss Alert level if connected
When SetProperty gets called for LinkLossAlertLevel, the alert level
can be written in the remote is the link is up. |
Claudio Takahasi |
14 years ago |
1 file, +3, -0 |
| d337cc50 |
sbc: Use __asm__ keyword
There are two reasons for this change:
First: consistency. __asm__ was already used elsewhere in the files, so
using that throughout is cleaner.
Second: both asm and __asm__ are GCC-specific extensions, not defined in
the C standard. When compiling with --std=gnu99 both are recognized, but
when using --std=c99 only __asm__ is recognized to make it perfectly
clear that you're not using some standard C99 construct, but a
GCC-extension. |
Maarten Bosmans |
14 years ago |
4 files, +25, -25 |
| 39660783 |
Add support for blacklisting events
Add support for mgmt interface events for device blocking/unblocking.
Kernel sends a mgmt event when a device has been blocked by another
management socket or with ioctl command (e.g. by hciconfig).
Parameter update_only is added to device_block/unblock functions in
device.c to avoid code copying. When update_only is true, blocking
command is not sent to kernel, but only device status is updated and
dbus signal is sent. |
Antti Julku |
14 years ago |
6 files, +113, -11 |
| 14ca2df1 |
Add definition for L2CAP Ext Window Size
This is needed e.g. by hcidump. |
Andrei Emeltchenko |
14 years ago |
1 file, +1, -0 |
| dec26ee5 |
Fix fd usage when not connected
When the avctp channel is not connected, we call
g_io_channel_unix_get_fd() with a NULL pointer. Glib does not check the
pointer before dereferencing it, causing bluetoothd to segv.
Move the function call to the place it's actually needed, after the
safety checks. |
Lucas De Marchi |
14 years ago |
1 file, +3, -1 |
| 1e36c9b4 |
Print an error message if g_dbus_setup_bus fails.
That is standard processing when dbus_error_is_set() returns true, according to
dbus/dbus/dbus-errors.c.
In this case of g_dbus_setup_bus(), it provides extra help for determing the
specific cause of the error. |
Burt Silverman |
14 years ago |
1 file, +1, -0 |
| 04a4e9a6 |
Add Serial Proxy and Serial Proxy Manager doc |
Bruno Dilly |
14 years ago |
1 file, +106, -0 |