component/bt: Fix bug of bt dynamic memory

diff --git a/components/bt/bluedroid/main/bte_init.c b/components/bt/bluedroid/main/bte_init.c
index 87605825ff5d7167905c02ed54580bcf02cd344f..df463280f62a9b6fe531bdbeac1466d216344b9f 100644 (file)
--- a/components/bt/bluedroid/main/bte_init.c
+++ b/components/bt/bluedroid/main/bte_init.c
@@ -173,7 +173,6 @@ void BTE_InitStack(void)
 #endif  // PAN
 #endif  // BNEP Included
 
-
     //AVDT and its profiles
 #if (defined(A2D_INCLUDED) && A2D_INCLUDED == TRUE)
     A2D_Init();
@@ -229,6 +228,12 @@ void BTE_InitStack(void)
     memset((void *)bta_dm_di_cb_ptr, 0, sizeof(tBTA_DM_DI_CB));
     //memset((void *)bta_prm_cb_ptr, 0, sizeof(tBTA_PRM_CB));
     //memset((void *)bta_ag_cb_ptr, 0, sizeof(tBTA_AG_CB));
+#if (defined BTA_JV_INCLUDED && BTA_JV_INCLUDED == TRUE)
+    if ((bta_jv_cb_ptr = (tBTA_JV_CB *)osi_malloc(sizeof(tBTA_JV_CB))) == NULL) {
+        return;
+    }
+    memset((void *)bta_jv_cb_ptr, 0, sizeof(tBTA_JV_CB));
+#endif //JV
 #if BTA_HS_INCLUDED == TRUE
     memset((void *)bta_hs_cb_ptr, 0, sizeof(tBTA_HS_CB));
 #endif
@@ -253,7 +258,7 @@ void BTE_InitStack(void)
 #if BTA_HH_INCLUDED==TRUE
     if ((bta_hh_cb_ptr = (tBTA_HH_CB *)osi_malloc(sizeof(tBTA_HH_CB))) == NULL) {
         return;
-    } 
+    }
     memset((void *)bta_hh_cb_ptr, 0, sizeof(tBTA_HH_CB));
 #endif
 #if BTA_HL_INCLUDED==TRUE
@@ -277,3 +282,76 @@ void BTE_InitStack(void)
 
 #endif // BTA_INCLUDED == TRUE
 }
+
+/*****************************************************************************
+**
+** Function         BTE_DeinitStack
+**
+** Description      Deinitialize control block memory for each component.
+**
+**                  Note: This API must be called
+**                      after freeing the BTU Task.
+**
+** Returns          void
+**
+******************************************************************************/
+void BTE_DeinitStack(void)
+{
+    //BTA Modules
+#if (BTA_INCLUDED == TRUE && BTA_DYNAMIC_MEMORY == TRUE)
+#if GATTS_INCLUDED == TRUE
+    osi_free(bta_gatts_cb_ptr);
+    bta_gatts_cb_ptr = NULL;
+#endif
+#if GATTC_INCLUDED==TRUE
+    osi_free(bta_gattc_cb_ptr);
+    bta_gattc_cb_ptr = NULL;
+#endif
+#if BTA_HH_INCLUDED==TRUE
+    osi_free(bta_hh_cb_ptr);
+    bta_hh_cb_ptr = NULL;
+#endif
+#if BTA_AV_INCLUDED==TRUE
+    osi_free(bta_av_cb_ptr);
+    bta_av_cb_ptr = NULL;
+#endif
+#if BTA_AR_INCLUDED==TRUE
+    osi_free(bta_ar_cb_ptr);
+    bta_ar_cb_ptr = NULL;
+#endif
+#if BTA_SDP_INCLUDED == TRUE
+    osi_free(bta_sdp_cb_ptr);
+    bta_sdp_cb_ptr = NULL;
+#endif
+#if (defined BTA_JV_INCLUDED && BTA_JV_INCLUDED == TRUE)
+    osi_free(bta_jv_cb_ptr);
+    bta_jv_cb_ptr = NULL;
+#endif //JV
+    osi_free(bta_dm_di_cb_ptr);
+    bta_dm_di_cb_ptr = NULL;
+    osi_free(bta_dm_search_cb_ptr);
+    bta_dm_search_cb_ptr = NULL;
+    osi_free(bta_dm_cb_ptr);
+    bta_dm_cb_ptr = NULL;
+    osi_free(bta_sys_cb_ptr);
+    bta_sys_cb_ptr = NULL;
+#endif // BTA_INCLUDED == TRUE
+
+#if (defined(AVCT_INCLUDED) && AVCT_INCLUDED == TRUE && AVCT_DYNAMIC_MEMORY == TRUE)
+    osi_free(avct_cb_ptr);
+    avct_cb_ptr = NULL;
+#endif
+
+#if (defined(AVDT_INCLUDED) && AVDT_INCLUDED == TRUE && AVDT_DYNAMIC_MEMORY == TRUE)
+    osi_free(avdt_cb_ptr);
+    avdt_cb_ptr = NULL;
+#endif
+
+#if (defined(AVRC_INCLUDED) && AVRC_INCLUDED == TRUE)
+    AVRC_Deinit();
+#endif
+
+#if (defined(A2D_INCLUDED) && A2D_INCLUDED == TRUE)
+    A2D_Deinit();
+#endif
+}
\ No newline at end of file

diff --git a/components/bt/bluedroid/stack/a2dp/a2d_api.c b/components/bt/bluedroid/stack/a2dp/a2d_api.c
index 8c144b8ba3a528f1f6295700bf054ba7a967ee98..1d77e338ee25dd29d73a8d8248de9ab073bbd8c6 100644 (file)
--- a/components/bt/bluedroid/stack/a2dp/a2d_api.c
+++ b/components/bt/bluedroid/stack/a2dp/a2d_api.c
@@ -391,4 +391,22 @@ void A2D_Init(void)
 #endif
 }
 
+/*******************************************************************************
+**
+** Function         A2D_Deinit
+**
+** Description      This function is called to deinitialize the control block
+**                  for this layer.
+**
+** Returns          void
+**
+*******************************************************************************/
+void A2D_Deinit(void)
+{
+#if (A2D_DYNAMIC_MEMORY)
+    osi_free(a2d_cb_ptr);
+    a2d_cb_ptr = NULL;
+#endif /* #if (A2D_DYNAMIC_MEMORY) */
+}
+
 #endif /* #if (defined(A2D_INCLUDED) && A2D_INCLUDED == TRUE) */

diff --git a/components/bt/bluedroid/stack/avrc/avrc_sdp.c b/components/bt/bluedroid/stack/avrc/avrc_sdp.c
index cdec74c87103daf407df35ee4df62a13f5cf3307..31d4aeeb90ad1966ae87973177d637092da4b51f 100644 (file)
--- a/components/bt/bluedroid/stack/avrc/avrc_sdp.c
+++ b/components/bt/bluedroid/stack/avrc/avrc_sdp.c
@@ -357,4 +357,23 @@ void AVRC_Init(void)
 #endif
 }
 
+/*******************************************************************************
+**
+** Function         AVRC_Deinit
+**
+** Description      This function is called at stack shotdown to free the
+**                  control block (if using dynamic memory), and deinitializes the
+**                  control block and tracing level.
+**
+** Returns          void
+**
+*******************************************************************************/
+void AVRC_Deinit(void)
+{
+#if AVRC_DYNAMIC_MEMORY
+    osi_free(avrc_cb_ptr);
+    avrc_cb_ptr = NULL;
+#endif /* #if AVRC_DYNAMIC_MEMORY */
+}
+
 #endif /* #if (defined(AVRC_INCLUDED) && AVRC_INCLUDED == TRUE) */

diff --git a/components/bt/bluedroid/stack/btu/btu_init.c b/components/bt/bluedroid/stack/btu/btu_init.c
index 60381c183049c108b3495aa49e2e4e096c905ec5..586ffc4b15f698b19f42c1d334c4a3eb281c7fec 100644 (file)
--- a/components/bt/bluedroid/stack/btu/btu_init.c
+++ b/components/bt/bluedroid/stack/btu/btu_init.c
@@ -118,6 +118,10 @@ void btu_free_core(void)
     // Free the mandatory core stack components
     l2c_free();
 
+#if (defined(SDP_INCLUDED) && SDP_INCLUDED == TRUE)
+    sdp_deinit();
+#endif
+
 #if BLE_INCLUDED == TRUE
 #if (defined(GATT_INCLUDED) && GATT_INCLUDED == true)
     gatt_free();

diff --git a/components/bt/bluedroid/stack/btu/btu_task.c b/components/bt/bluedroid/stack/btu/btu_task.c
index ee6bb6d86566e36b550d93be83526e7afd53922d..196aca4c93b8c732ebd80b2935efd655ad1d3144 100644 (file)
--- a/components/bt/bluedroid/stack/btu/btu_task.c
+++ b/components/bt/bluedroid/stack/btu/btu_task.c
@@ -86,6 +86,7 @@ extern void avdt_rcv_sync_info (BT_HDR *p_buf);
 //#endif
 
 extern void BTE_InitStack(void);
+extern void BTE_DeinitStack(void);
 
 /* Define BTU storage area
 */
@@ -314,6 +315,7 @@ void btu_task_shut_down(void)
 #if (defined(BTA_INCLUDED) && BTA_INCLUDED == TRUE)
     bta_sys_free();
 #endif
+    BTE_DeinitStack();
 
     btu_free_core();
 }

diff --git a/components/bt/bluedroid/stack/include/a2d_api.h b/components/bt/bluedroid/stack/include/a2d_api.h
index 1b999d5c4feb3690e35a2b3e4b9be6c832a87387..eaf6b37f140fdef5a89ae793cfa410d2ae4d16b8 100644 (file)
--- a/components/bt/bluedroid/stack/include/a2d_api.h
+++ b/components/bt/bluedroid/stack/include/a2d_api.h
@@ -251,5 +251,6 @@ extern UINT8 A2D_BitsSet(UINT8 num);
 **
 *******************************************************************************/
 extern void A2D_Init(void);
+extern void A2D_Deinit(void);
 #endif  ///A2D_INCLUDED
 #endif /* A2D_API_H */

diff --git a/components/bt/bluedroid/stack/include/avrc_api.h b/components/bt/bluedroid/stack/include/avrc_api.h
index 3ca3e2d76f93f12fe08be5310cb4f91677752778..2634c97fb726cea3b6d4c6b990cf06cbced2060d 100644 (file)
--- a/components/bt/bluedroid/stack/include/avrc_api.h
+++ b/components/bt/bluedroid/stack/include/avrc_api.h
@@ -554,6 +554,19 @@ extern UINT8 AVRC_SetTraceLevel (UINT8 new_level);
 *******************************************************************************/
 extern void AVRC_Init(void);
 
+/*******************************************************************************
+**
+** Function         AVRC_Deinit
+**
+** Description      This function is called at stack shotdown to free the
+**                  control block (if using dynamic memory), and deinitializes the
+**                  control block and tracing level.
+**
+** Returns          void
+**
+*******************************************************************************/
+extern void AVRC_Deinit(void);
+
 /*******************************************************************************
 **
 ** Function         AVRC_ParsCommand

diff --git a/components/bt/bluedroid/stack/sdp/include/sdpint.h b/components/bt/bluedroid/stack/sdp/include/sdpint.h
index 9e721d9f9d82af08b6d15ed055078d11145622da..058257b8ef170e476dfc749ce42772f08203c50a 100644 (file)
--- a/components/bt/bluedroid/stack/sdp/include/sdpint.h
+++ b/components/bt/bluedroid/stack/sdp/include/sdpint.h
@@ -237,6 +237,7 @@ extern tSDP_CB *sdp_cb_ptr;
 
 /* Functions provided by sdp_main.c */
 extern void     sdp_init (void);
+extern void     sdp_deinit (void);
 extern void     sdp_disconnect (tCONN_CB *p_ccb, UINT16 reason);
 
 #if (defined(SDP_DEBUG) && SDP_DEBUG == TRUE)

diff --git a/components/bt/bluedroid/stack/sdp/sdp_main.c b/components/bt/bluedroid/stack/sdp/sdp_main.c
index db6a7a3d6c9af0b5215ce46c1479fa1733f7ab06..b8e76b1d8e13a1c9b79d0ef6f70ddd374d6b0fa3 100644 (file)
--- a/components/bt/bluedroid/stack/sdp/sdp_main.c
+++ b/components/bt/bluedroid/stack/sdp/sdp_main.c
@@ -138,6 +138,14 @@ void sdp_init (void)
     }
 }
 
+void sdp_deinit (void)
+{
+#if SDP_DYNAMIC_MEMORY
+    osi_free(sdp_cb_ptr);
+    sdp_cb_ptr = NULL;
+#endif /* #if SDP_DYNAMIC_MEMORY */
+}
+
 #if (defined(SDP_DEBUG) && SDP_DEBUG == TRUE)
 /*******************************************************************************
 **