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

Commit f4b02474 authored by Dorin Drimus's avatar Dorin Drimus Committed by Automerger Merge Worker
Browse files

Use bluetooth.core.gap.le.privacy.enabled sysprop to init BLE Privacy am: 06f7bd00 

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



Change-Id: I62aef05d2600bb83beb8d63b117131b58df9ab50
Ignore-AOSP-First: this is an automerge
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 74876ad2 06f7bd00

system/btif/src/btif_dm.cc

+24 −2
Original line number Diff line number Diff line
@@ -51,6 +51,10 @@ 


#include <mutex>



#ifdef OS_ANDROID

#include <android/sysprop/BluetoothProperties.sysprop.h>

#endif



#include "advertise_data_parser.h"

#include "bta_csis_api.h"

#include "bta_dm_int.h"
@@ -132,6 +136,10 @@ const bool enable_address_consolidate = true; // TODO remove 
#error "default btif local name size exceeds stack supported length"

#endif



#ifndef PROPERTY_BLE_PRIVACY_ENABLED

#define PROPERTY_BLE_PRIVACY_ENABLED "bluetooth.core.gap.le.privacy.enabled"

#endif



#define ENCRYPTED_BREDR 2

#define ENCRYPTED_LE 4
@@ -1543,8 +1551,22 @@ void BTIF_dm_enable() { 
    BTA_DmSetDeviceName(btif_get_default_local_name());

  }



  /* Enable local privacy */

  BTA_DmBleConfigLocalPrivacy(BLE_LOCAL_PRIVACY_ENABLED);

  /* Enable or disable local privacy */

  bool ble_privacy_enabled = true;

#ifdef OS_ANDROID

  ble_privacy_enabled =

      android::sysprop::BluetoothProperties::isGapLePrivacyEnabled().value_or(

          true);

#else

  char ble_privacy_text[PROPERTY_VALUE_MAX] = "true";  // default is enabled

  if (osi_property_get(PROPERTY_BLE_PRIVACY_ENABLED, ble_privacy_text,

                       "true") &&

      !strcmp(ble_privacy_text, "false")) {

    ble_privacy_enabled = false;

  }

#endif

  LOG_INFO("%s BLE Privacy: %d", __func__, ble_privacy_enabled);

  BTA_DmBleConfigLocalPrivacy(ble_privacy_enabled);



  /* for each of the enabled services in the mask, trigger the profile

   * enable */

system/btif/src/stack_manager.cc

+16 −5
Original line number Diff line number Diff line
@@ -76,15 +76,26 @@ 


// Validate or respond to various conditional compilation flags



#if BLE_PRIVACY_SPT != TRUE

// Once BLE_PRIVACY_SPT is no longer exposed via bt_target.h

// this check and error statement may be removed.

#ifdef BLE_PRIVACY_SPT

// When BLE_PRIVACY_SPT is not used anymore this check and error statement may be removed.

#warning \

    "#define BLE_PRIVACY_SPT FALSE preprocessor compilation flag is unsupported"

    "#define BLE_PRIVACY_SPT FALSE preprocessor compilation flag is no longer used"

#warning \

    "  To disable LE privacy for a device use: #define BLE_LOCAL_PRIVACY_ENABLED FALSE"

    "  To disable LE privacy for a device \

set 'bluetooth.core.gap.le.privacy.enabled' sysprop to false"

#if BLE_PRIVACY_SPT == FALSE

#error "*** Conditional Compilation Directive error"

#endif

#endif



#ifdef BLE_LOCAL_PRIVACY_ENABLED

// When BLE_PRIVACY_SPT is not used anymore this warning statement may be removed.

#warning \

    "#define BLE_LOCAL_PRIVACY_ENABLED FALSE preprocessor compilation flag is no longer used"

#warning \

    "  To disable LE privacy for a device \

set 'bluetooth.core.gap.le.privacy.enabled sysprop' to false"

#endif



#if SDP_RAW_DATA_INCLUDED != TRUE

// Once SDP_RAW_DATA_INCLUDED is no longer exposed via bt_target.h

system/internal_include/bt_target.h

+0 −15
Original line number Diff line number Diff line
@@ -524,21 +524,6 @@ 
#define LOCAL_BLE_CONTROLLER_ID 1

#endif



/*

 * Toggles support for general LE privacy features such as remote address

 * resolution, local address rotation etc.

 */

#ifndef BLE_PRIVACY_SPT

#define BLE_PRIVACY_SPT TRUE

#endif



/*

 * Enables or disables support for local privacy (ex. address rotation)

 */

#ifndef BLE_LOCAL_PRIVACY_ENABLED

#define BLE_LOCAL_PRIVACY_ENABLED TRUE

#endif



/*

 * Toggles support for vendor specific extensions such as RPA offloading,

 * feature discovery, multi-adv etc.