Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c1969ebd authored by William Escande's avatar William Escande Committed by Automerger Merge Worker
Browse files

SDP: TEST Handle race condition in HID SDP am: f53d7b4e 

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/20653139



Change-Id: I787ffc5e56c51663db16370d8bc69e58c924b829
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 53dde89c f53d7b4e

system/stack/test/sdp/stack_sdp_test.cc

+32 −0
Original line number Diff line number Diff line
@@ -140,3 +140,35 @@ TEST_F(StackSdpMainTest, sdp_service_search_request_queuing) { 
  ASSERT_EQ(p_ccb1->con_state, SDP_STATE_IDLE);

  ASSERT_EQ(p_ccb2->con_state, SDP_STATE_IDLE);

}



void sdp_callback(tSDP_RESULT result) {

  if (result == SDP_SUCCESS) {

    ASSERT_TRUE(SDP_ServiceSearchRequest(addr, sdp_db, nullptr));

  }

}



TEST_F(StackSdpMainTest, sdp_service_search_request_queuing_race_condition) {

  // start first request

  ASSERT_TRUE(SDP_ServiceSearchRequest(addr, sdp_db, sdp_callback));

  const int cid1 = L2CA_ConnectReq2_cid;

  tCONN_CB* p_ccb1 = find_ccb(cid1, SDP_STATE_CONN_SETUP);

  ASSERT_NE(p_ccb1, nullptr);

  ASSERT_EQ(p_ccb1->con_state, SDP_STATE_CONN_SETUP);



  tL2CAP_CFG_INFO cfg;

  sdp_cb.reg_info.pL2CA_ConfigCfm_Cb(p_ccb1->connection_id, 0, &cfg);



  ASSERT_EQ(p_ccb1->con_state, SDP_STATE_CONNECTED);



  sdp_disconnect(p_ccb1, SDP_SUCCESS);

  sdp_cb.reg_info.pL2CA_DisconnectCfm_Cb(p_ccb1->connection_id, 0);



  const int cid2 = L2CA_ConnectReq2_cid;

  ASSERT_NE(cid1, cid2);  // The callback a queued a new request

  tCONN_CB* p_ccb2 = find_ccb(cid2, SDP_STATE_CONN_SETUP);

  ASSERT_NE(p_ccb2, nullptr);

  // If race condition, this will be stuck in PEND

  ASSERT_EQ(p_ccb2->con_state, SDP_STATE_CONN_SETUP);



  sdp_disconnect(p_ccb2, SDP_SUCCESS);

}