Disable handling browsing PDU in AVRCP 63/139363/2
authorSreeraj Mohan Kottapuzhackal <sreeraj.mk@samsung.com>
Wed, 15 Feb 2017 09:02:57 +0000 (14:32 +0530)
committerSreeraj Mohan Kottapuzhackal <sreeraj.mk@samsung.com>
Tue, 18 Jul 2017 13:08:18 +0000 (13:08 +0000)
[Model] All
[BinType] AP
[Customer] All

[Issue#] P170214-06960
[Request] PLM
[Occurrence Version] All

[Problem] Code panic while handling AVRCP browsing PDU
[Cause & Measure] Cause: Tizen doesn't support AVRCP browsing channel
which is a part of AVRCP 1.4 and higher versions. AVRCP browsing channel is not supported by open source bluez as well, but partially implemented. This is causing a panic as not all the commands are handled properly.
                  Measure: Disabled creating GIO channel for browsing PSM as we need not listen for browsing commands
[Checking Method] Unit testing

[Team] Connectivity
[Developer] sreeraj.mk
[Solution company] Samsung
[Change Type] N/A

Change-Id: I054bcab73fc4f4a9d69bc81f30c35d537dc16081

profiles/audio/avctp.c

index 2a5f464..a0b2f0e 100644 (file)
@@ -1533,6 +1533,9 @@ int avctp_register(struct btd_adapter *adapter, gboolean master)
                g_free(server);
                return -1;
        }
+
+/* Tizen doesn't support AVRCP BROWSING CHANNEL */
+#ifndef TIZEN_FEATURE_BLUEZ_MODIFY
        server->browsing_io = avctp_server_socket(src, master, L2CAP_MODE_ERTM,
                                                        AVCTP_BROWSING_PSM);
        if (!server->browsing_io) {
@@ -1544,6 +1547,7 @@ int avctp_register(struct btd_adapter *adapter, gboolean master)
                g_free(server);
                return -1;
        }
+#endif
 
        server->adapter = btd_adapter_ref(adapter);