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

Commit 5c4bd8f5 authored by Yan Yan's avatar Yan Yan Committed by Automerger Merge Worker
Browse files

Merge "Add a null pointer check to the VcnConfig" am: c80d564f am:... 

Merge "Add a null pointer check to the VcnConfig" am: c80d564f am: 186bbf0e am: 7fe99f25 am: 0fa2c9e9 am: bfce32c4 am: 2e8f613b

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



Change-Id: I5a8723a0c25e2252f987f9c27875044eafb236bf
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 0420fd93 2e8f613b

services/core/java/com/android/server/VcnManagementService.java

+8 −1
Original line number Diff line number Diff line
@@ -1065,13 +1065,20 @@ public class VcnManagementService extends IVcnManagementService.Stub { 
            boolean isRestricted = false;

            synchronized (mLock) {

                final Vcn vcn = mVcns.get(subGrp);

                final VcnConfig vcnConfig = mConfigs.get(subGrp);

                if (vcn != null) {

                    if (vcnConfig == null) {

                        // TODO: b/284381334 Investigate for the root cause of this issue

                        // and handle it properly

                        logWtf("Vcn instance exists but VcnConfig does not for " + subGrp);

                    }



                    if (vcn.getStatus() == VCN_STATUS_CODE_ACTIVE) {

                        isVcnManagedNetwork = true;

                    }



                    final Set<Integer> restrictedTransports = mDeps.getRestrictedTransports(

                            subGrp, mLastSnapshot, mConfigs.get(subGrp));

                            subGrp, mLastSnapshot, vcnConfig);

                    for (int restrictedTransport : restrictedTransports) {

                        if (ncCopy.hasTransport(restrictedTransport)) {

                            if (restrictedTransport == TRANSPORT_CELLULAR