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

Commit 64a5cbc0 authored by Lorenzo Colitti's avatar Lorenzo Colitti Committed by android-build-merger
Browse files

Merge "Address review comments on ag/962875." into nyc-dev am: 6d1a776c am: 95e0885c

am: 7e53fba3

* commit '7e53fba3':
  Address review comments on ag/962875.

Change-Id: I7474d3892e8e77f2b57aebcfc3e52d5bd1f44c5d
parents 9414f7cd 7e53fba3
Loading
Loading
Loading
Loading
+4 −4
Original line number Diff line number Diff line
@@ -2048,13 +2048,13 @@ public class NetworkManagementService extends INetworkManagementService.Stub
        }
    }

    private void closeSocketsForFirewallChain(int chain, String chainName) {
    private void closeSocketsForFirewallChainLocked(int chain, String chainName) {
        // UID ranges to close sockets on.
        UidRange[] ranges;
        // UID ranges whose sockets we won't touch.
        int[] exemptUids;

        SparseIntArray rules = getUidFirewallRules(chain);
        final SparseIntArray rules = getUidFirewallRules(chain);
        int numUids = 0;

        if (getFirewallType(chain) == FIREWALL_TYPE_WHITELIST) {
@@ -2119,7 +2119,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub
            mFirewallChainStates.put(chain, enable);

            final String operation = enable ? "enable_chain" : "disable_chain";
            String chainName;
            final String chainName;
            switch(chain) {
                case FIREWALL_CHAIN_STANDBY:
                    chainName = FIREWALL_CHAIN_NAME_STANDBY;
@@ -2146,7 +2146,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub
            // whitelist and blacklist chains allow RSTs through.
            if (enable) {
                if (DBG) Slog.d(TAG, "Closing sockets after enabling chain " + chainName);
                closeSocketsForFirewallChain(chain, chainName);
                closeSocketsForFirewallChainLocked(chain, chainName);
            }
        }
    }