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

Commit 7bdfb34b authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Ignore CRLF strings in display SSIDs to avoid display errors" into tm-qpr-dev am: 9447ec6d 

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/19168028



Change-Id: I37125c165dbb9d48d672666512faec8248bce5d2
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 1a508148 9447ec6d

src/com/android/settings/wifi/addappnetworks/AddAppNetworksFragment.java

+8 −1
Original line number Diff line number Diff line
@@ -39,6 +39,7 @@ import android.os.SimpleClock; 
import android.os.SystemClock;

import android.provider.Settings;

import android.text.TextUtils;

import android.util.EventLog;

import android.util.Log;

import android.view.LayoutInflater;

import android.view.View;
@@ -518,7 +519,13 @@ public class AddAppNetworksFragment extends InstrumentedFragment implements 


        UiConfigurationItem(String displayedSsid, WifiNetworkSuggestion wifiNetworkSuggestion,

                int index, int level) {

            if (displayedSsid.contains("\n") || displayedSsid.contains("\r")) {

                mDisplayedSsid = displayedSsid.replaceAll("\\r|\\n", "");

                Log.e(TAG, "Ignore CRLF strings in display SSIDs to avoid display errors!");

                EventLog.writeEvent(0x534e4554, "224545390", -1 /* UID */, "CRLF injection");

            } else {

                mDisplayedSsid = displayedSsid;

            }

            mWifiNetworkSuggestion = wifiNetworkSuggestion;

            mIndex = index;

            mLevel = level;

tests/robotests/src/com/android/settings/wifi/addappnetworks/AddAppNetworksFragmentTest.java

+30 −0
Original line number Diff line number Diff line
@@ -455,6 +455,36 @@ public class AddAppNetworksFragmentTest { 
                AddAppNetworksFragment.MESSAGE_SHOW_SAVE_FAILED)).isTrue();

    }



    @Test

    public void uiConfigurationItem_putCrToDisplayedSsid_shouldRemoveCr() {

        String testSsid = "\r" + FAKE_NEW_WPA_SSID + "\r";



        AddAppNetworksFragment.UiConfigurationItem item =

                new AddAppNetworksFragment.UiConfigurationItem(testSsid, null, 0, 0);



        assertThat(item.mDisplayedSsid).isEqualTo(FAKE_NEW_WPA_SSID);

    }



    @Test

    public void uiConfigurationItem_putLfToDisplayedSsid_shouldRemoveLf() {

        String testSsid = "\n" + FAKE_NEW_WPA_SSID + "\n";



        AddAppNetworksFragment.UiConfigurationItem item =

                new AddAppNetworksFragment.UiConfigurationItem(testSsid, null, 0, 0);



        assertThat(item.mDisplayedSsid).isEqualTo(FAKE_NEW_WPA_SSID);

    }



    @Test

    public void uiConfigurationItem_putCrLfToDisplayedSsid_shouldRemoveCrLf() {

        String testSsid = "\r\n" + FAKE_NEW_WPA_SSID + "\r\n";



        AddAppNetworksFragment.UiConfigurationItem item =

                new AddAppNetworksFragment.UiConfigurationItem(testSsid, null, 0, 0);



        assertThat(item.mDisplayedSsid).isEqualTo(FAKE_NEW_WPA_SSID);

    }



    private void setUpOneScannedNetworkWithScanedLevel4() {

        final ArrayList list = new ArrayList<>();

        list.add(mWifiEntry);