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

Commit 23eb297c authored by Irfan Sheriff's avatar Irfan Sheriff
Browse files

Handle hostapd and tether interface changes 

The set config will only update hostapd config file right now and we need to restart after that happens.

Also, softap interface is changed to be the regular interface

Bug: 5069108
Change-Id: Ibe80129b58e5339f06f186870bcd8f37569c6db3
parent 556d0476

core/java/android/os/INetworkManagementService.aidl

+1 −1
Original line number Diff line number Diff line
@@ -180,7 +180,7 @@ interface INetworkManagementService 
    /**

     * Stop Wifi Access Point

     */

    void stopAccessPoint();

    void stopAccessPoint(String wlanIface);



    /**

     * Set Access Point config

services/java/com/android/server/NetworkManagementService.java

+3 −2
Original line number Diff line number Diff line
@@ -849,7 +849,6 @@ class NetworkManagementService extends INetworkManagementService.Stub { 
        mContext.enforceCallingOrSelfPermission(

                android.Manifest.permission.CHANGE_WIFI_STATE, "NetworkManagementService");

        try {

            mConnector.doCommand(String.format("softap stop " + wlanIface));

            mConnector.doCommand(String.format("softap fwreload " + wlanIface + " AP"));

            mConnector.doCommand(String.format("softap start " + wlanIface));

            if (wifiConfig == null) {
@@ -897,13 +896,15 @@ class NetworkManagementService extends INetworkManagementService.Stub { 
        }

    }



    public void stopAccessPoint() throws IllegalStateException {

    public void stopAccessPoint(String wlanIface) throws IllegalStateException {

        mContext.enforceCallingOrSelfPermission(

                android.Manifest.permission.CHANGE_NETWORK_STATE, "NetworkManagementService");

        mContext.enforceCallingOrSelfPermission(

                android.Manifest.permission.CHANGE_WIFI_STATE, "NetworkManagementService");

        try {

            mConnector.doCommand("softap stopap");

            mConnector.doCommand("softap stop " + wlanIface);

            mConnector.doCommand(String.format("softap fwreload " + wlanIface + " STA"));

        } catch (NativeDaemonConnectorException e) {

            throw new IllegalStateException("Error communicating to native daemon to stop soft AP",

                    e);

services/java/com/android/server/connectivity/Tethering.java

+2 −0
Original line number Diff line number Diff line
@@ -221,6 +221,8 @@ public class Tethering extends INetworkManagementEventObserver.Stub { 
    }



    public void interfaceLinkStateChanged(String iface, boolean up) {

        if (DEBUG) Log.d(TAG, "interfaceLinkStateChanged " + iface + ", " + up);

        interfaceStatusChanged(iface, up);

    }



    private boolean isUsb(String iface) {

wifi/java/android/net/wifi/WifiStateMachine.java

+74 −28
Original line number Diff line number Diff line
@@ -97,7 +97,7 @@ public class WifiStateMachine extends StateMachine { 
    private static final String NETWORKTYPE = "WIFI";

    private static final boolean DBG = false;



    /* TODO: fetch a configurable interface */

    /* TODO: This is no more used with the hostapd code. Clean up */

    private static final String SOFTAP_IFACE = "wl0.1";



    private WifiMonitor mWifiMonitor;
@@ -208,8 +208,10 @@ public class WifiStateMachine extends StateMachine { 
    static final int CMD_SET_AP_CONFIG                    = BASE + 23;

    /* Get the soft access point configuration */

    static final int CMD_GET_AP_CONFIG                    = BASE + 24;

    /* Set configuration on tether interface */

    static final int CMD_TETHER_INTERFACE                 = BASE + 25;



    static final int CMD_BLUETOOTH_ADAPTER_STATE_CHANGE   = BASE + 25;

    static final int CMD_BLUETOOTH_ADAPTER_STATE_CHANGE   = BASE + 26;



    /* Supplicant events */

    /* Connection to supplicant established */
@@ -421,8 +423,10 @@ public class WifiStateMachine extends StateMachine { 
    /* Waiting for WPS to be completed*/

    private State mWaitForWpsCompletionState = new WaitForWpsCompletionState();



    /* Soft Ap is running */

    /* Soft ap is running */

    private State mSoftApStartedState = new SoftApStartedState();

    /* Soft ap is running and we are tethered through connectivity service */

    private State mTetheredState = new TetheredState();





    /**
@@ -513,13 +517,9 @@ public class WifiStateMachine extends StateMachine { 
            new BroadcastReceiver() {

                @Override

                public void onReceive(Context context, Intent intent) {



                    ArrayList<String> available = intent.getStringArrayListExtra(

                            ConnectivityManager.EXTRA_AVAILABLE_TETHER);

                    ArrayList<String> active = intent.getStringArrayListExtra(

                            ConnectivityManager.EXTRA_ACTIVE_TETHER);

                    updateTetherState(available, active);



                    sendMessage(CMD_TETHER_INTERFACE, available);

                }

            },new IntentFilter(ConnectivityManager.ACTION_TETHER_STATE_CHANGED));
@@ -559,6 +559,7 @@ public class WifiStateMachine extends StateMachine { 
                addState(mDriverStoppedState, mSupplicantStartedState);

            addState(mSupplicantStoppingState, mDefaultState);

            addState(mSoftApStartedState, mDefaultState);

                addState(mTetheredState, mSoftApStartedState);



        setInitialState(mInitialState);
@@ -1048,14 +1049,17 @@ public class WifiStateMachine extends StateMachine { 
     * Internal private functions

     ********************************************************/



    private void updateTetherState(ArrayList<String> available, ArrayList<String> tethered) {



        boolean wifiTethered = false;

        boolean wifiAvailable = false;



    private void checkAndSetConnectivityInstance() {

        if (mCm == null) {

            mCm = (ConnectivityManager) mContext.getSystemService(Context.CONNECTIVITY_SERVICE);

        }

    }



    private void startTethering(ArrayList<String> available) {



        boolean wifiAvailable = false;



        checkAndSetConnectivityInstance();



        String[] wifiRegexs = mCm.getTetherableWifiRegexs();
@@ -1091,6 +1095,29 @@ public class WifiStateMachine extends StateMachine { 
        }

    }



    private void stopTethering() {



        checkAndSetConnectivityInstance();



        /* Clear the interface config to allow dhcp correctly configure new

           ip settings */

        InterfaceConfiguration ifcg = null;

        try {

            ifcg = nwService.getInterfaceConfig(mInterfaceName);

            if (ifcg != null) {

                ifcg.addr = new LinkAddress(NetworkUtils.numericToInetAddress(

                            "0.0.0.0"), 0);

                nwService.setInterfaceConfig(mInterfaceName, ifcg);

            }

        } catch (Exception e) {

            Log.e(TAG, "Error resetting interface " + mInterfaceName + ", :" + e);

        }



        if (mCm.untether(mInterfaceName) != ConnectivityManager.TETHER_ERROR_NO_ERROR) {

            Log.e(TAG, "Untether initiate failed!");

        }

    }



    /**

     * Set the country code from the system setting value, if any.

     */
@@ -1616,12 +1643,15 @@ public class WifiStateMachine extends StateMachine { 
            if (currentStatus == SOFT_AP_STOPPED) {

                nwService.startAccessPoint(config, mInterfaceName, SOFTAP_IFACE);

            } else if (currentStatus == SOFT_AP_RUNNING) {

                nwService.setAccessPoint(config, mInterfaceName, SOFTAP_IFACE);

                //nwService.setAccessPoint(config, mInterfaceName, SOFTAP_IFACE);

                //TODO: when we have a control channel to hostapd, we should not need to do this

                nwService.stopAccessPoint(mInterfaceName);

                nwService.startAccessPoint(config, mInterfaceName, SOFTAP_IFACE);

            }

        } catch (Exception e) {

            Log.e(TAG, "Exception in softap start " + e);

            try {

                nwService.stopAccessPoint();

                nwService.stopAccessPoint(mInterfaceName);

                nwService.startAccessPoint(config, mInterfaceName, SOFTAP_IFACE);

            } catch (Exception ee) {

                Log.e(TAG, "Exception during softap restart : " + ee);
@@ -1774,6 +1804,7 @@ public class WifiStateMachine extends StateMachine { 
                case CMD_STOP_DRIVER:

                case CMD_START_AP:

                case CMD_STOP_AP:

                case CMD_TETHER_INTERFACE:

                case CMD_START_SCAN:

                case CMD_DISCONNECT:

                case CMD_RECONNECT:
@@ -2828,10 +2859,7 @@ public class WifiStateMachine extends StateMachine { 
                    deferMessage(message);

                    break;

                case CMD_REQUEST_CM_WAKELOCK:

                    if (mCm == null) {

                        mCm = (ConnectivityManager)mContext.getSystemService(

                                Context.CONNECTIVITY_SERVICE);

                    }

                    checkAndSetConnectivityInstance();

                    mCm.requestNetworkTransitionWakelock(TAG);

                    break;

                case CMD_SET_SCAN_MODE:
@@ -3109,16 +3137,9 @@ public class WifiStateMachine extends StateMachine { 
                case CMD_STOP_AP:

                    Log.d(TAG,"Stopping Soft AP");

                    setWifiApState(WIFI_AP_STATE_DISABLING);



                    if (mCm == null) {

                        mCm = (ConnectivityManager) mContext.getSystemService(

                                Context.CONNECTIVITY_SERVICE);

                    }

                    if (mCm.untether(SOFTAP_IFACE) != ConnectivityManager.TETHER_ERROR_NO_ERROR) {

                        Log.e(TAG, "Untether initiate failed!");

                    }

                    stopTethering();

                    try {

                        nwService.stopAccessPoint();

                        nwService.stopAccessPoint(mInterfaceName);

                    } catch(Exception e) {

                        Log.e(TAG, "Exception in stopAccessPoint()");

                    }
@@ -3140,6 +3161,11 @@ public class WifiStateMachine extends StateMachine { 
                    Log.e(TAG,"Cannot start supplicant with a running soft AP");

                    setWifiState(WIFI_STATE_UNKNOWN);

                    break;

                case CMD_TETHER_INTERFACE:

                    ArrayList<String> available = (ArrayList<String>) message.obj;

                    startTethering(available);

                    transitionTo(mTetheredState);

                    break;

                default:

                    return NOT_HANDLED;

            }
@@ -3147,4 +3173,24 @@ public class WifiStateMachine extends StateMachine { 
            return HANDLED;

        }

    }



    class TetheredState extends State {

        @Override

        public void enter() {

            if (DBG) Log.d(TAG, getName() + "\n");

            EventLog.writeEvent(EVENTLOG_WIFI_STATE_CHANGED, getName());

        }

        @Override

        public boolean processMessage(Message message) {

            if (DBG) Log.d(TAG, getName() + message.toString() + "\n");

            switch(message.what) {

               case CMD_TETHER_INTERFACE:

                    // Ignore any duplicate interface available notifications

                    // when in tethered state

                    return HANDLED;

                default:

                    return NOT_HANDLED;

            }

        }

    }

}