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

Commit 5f07c24f authored by Sarvesh Kalwit's avatar Sarvesh Kalwit Committed by Automerger Merge Worker
Browse files

Fix empty data mask detection am: d0e8bb0d 

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



Change-Id: I6ff4cf0a4411de79252bbd6f17677743c0af82a3
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents c9ecf2f0 d0e8bb0d

android/app/src/com/android/bluetooth/le_scan/MsftAdvMonitor.java

+8 −3
Original line number Diff line number Diff line
@@ -70,7 +70,7 @@ class MsftAdvMonitor { 
        mMonitor.rssi_sampling_period = RSSI_SAMPLING_PERIOD;

        mMonitor.condition_type = MSFT_CONDITION_TYPE_PATTERNS;



        if (filter.getServiceDataUuid() != null && filter.getServiceDataMask() == null) {

        if (filter.getServiceDataUuid() != null && dataMaskIsEmpty(filter.getServiceDataMask())) {

            Pattern pattern = new Pattern();

            pattern.ad_type = (byte) 0x16; // Bluetooth Core Spec Part A, Section 1

            pattern.start_byte = FILTER_PATTERN_START_POSITION;
@@ -86,8 +86,7 @@ class MsftAdvMonitor { 
            mPatterns.add(pattern);

        } else if (filter.getAdvertisingData() != null

                && filter.getAdvertisingData().length != 0

                && (filter.getAdvertisingDataMask() == null

                        || filter.getAdvertisingDataMask().length == 0)) {

                && dataMaskIsEmpty(filter.getAdvertisingDataMask())) {

            Pattern pattern = new Pattern();

            pattern.ad_type = (byte) filter.getAdvertisingDataType();

            pattern.start_byte = FILTER_PATTERN_START_POSITION;
@@ -112,4 +111,10 @@ class MsftAdvMonitor { 
    Address getAddress() {

        return mAddress;

    }



    private boolean dataMaskIsEmpty(byte[] mask) {

        if (mask == null || mask.length == 0) return true;

        if (mask.length == 1 && mask[0] == 0) return true;

        return false;

    }

}