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

Commit 3c1e7f4d authored by Mark Chien's avatar Mark Chien Committed by Automerger Merge Worker
Browse files

Merge "Fix miss tracking downstream problem in PrivateAddressCoordinator" am:... 

Merge "Fix miss tracking downstream problem in PrivateAddressCoordinator" am: 613e6e3e am: 84b074ee am: 6574c6c8 am: 335c9c52

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1459887

Change-Id: I9262e13e3f3919c2b36bb8765a09c4abf2e31aa4
parents d1fbe088 335c9c52

packages/Tethering/src/com/android/networkstack/tethering/PrivateAddressCoordinator.java

+3 −3
Original line number Diff line number Diff line
@@ -36,6 +36,7 @@ import android.util.ArrayMap; 
import android.util.ArraySet;

import android.util.SparseArray;



import androidx.annotation.NonNull;

import androidx.annotation.Nullable;



import com.android.internal.annotations.VisibleForTesting;
@@ -135,7 +136,6 @@ public class PrivateAddressCoordinator { 
    private void handleMaybePrefixConflict(final List<IpPrefix> prefixes) {

        for (IpServer downstream : mDownstreams) {

            final IpPrefix target = getDownstreamPrefix(downstream);

            if (target == null) continue;



            for (IpPrefix source : prefixes) {

                if (isConflictPrefix(source, target)) {
@@ -179,6 +179,7 @@ public class PrivateAddressCoordinator { 
        final LinkAddress cachedAddress = mCachedAddresses.get(ipServer.interfaceType());

        if (useLastAddress && cachedAddress != null

                && !isConflictWithUpstream(asIpPrefix(cachedAddress))) {

            mDownstreams.add(ipServer);

            return cachedAddress;

        }
@@ -370,7 +371,6 @@ public class PrivateAddressCoordinator { 
        // in mCachedAddresses.

        for (IpServer downstream : mDownstreams) {

            final IpPrefix target = getDownstreamPrefix(downstream);

            if (target == null) continue;



            if (isConflictPrefix(prefix, target)) return target;

        }
@@ -378,9 +378,9 @@ public class PrivateAddressCoordinator { 
        return null;

    }



    @NonNull

    private IpPrefix getDownstreamPrefix(final IpServer downstream) {

        final LinkAddress address = downstream.getAddress();

        if (address == null) return null;



        return asIpPrefix(address);

    }

packages/Tethering/tests/unit/src/com/android/networkstack/tethering/PrivateAddressCoordinatorTest.java

+84 −108

File changed.

Preview size limit exceeded, changes collapsed.