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

Commit 93d77b63 authored by David Brown's avatar David Brown
Browse files

Don't log PII when launching intents with "sms:" or "sip:" URIs

We were already redacting phone numbers from "tel:" and "smsto:" URIs.

Now, do that for "sms:" and "sip:" too, since those schemes are equally
likely to contain PII in the scheme-specific part.

TESTED: Received incoming call; selected "Respond via SMS" -> "Custom
        message"; confirmed no PII in the system log.

Bug: 5244735
Change-Id: If2444c8901249aac9fd8e6b809d6da01db2e27c1
parent b6957413
Loading
Loading
Loading
Loading
+4 −0
Original line number Original line Diff line number Diff line
@@ -5627,6 +5627,10 @@ public class Intent implements Parcelable, Cloneable {
            if (scheme != null) {
            if (scheme != null) {
                if (scheme.equalsIgnoreCase("tel")) {
                if (scheme.equalsIgnoreCase("tel")) {
                    b.append("tel:xxx-xxx-xxxx");
                    b.append("tel:xxx-xxx-xxxx");
                } else if (scheme.equalsIgnoreCase("sip")) {
                    b.append("sip:xxxxxxxxxx");
                } else if (scheme.equalsIgnoreCase("sms")) {
                    b.append("sms:xxx-xxx-xxxx");
                } else if (scheme.equalsIgnoreCase("smsto")) {
                } else if (scheme.equalsIgnoreCase("smsto")) {
                    b.append("smsto:xxx-xxx-xxxx");
                    b.append("smsto:xxx-xxx-xxxx");
                } else {
                } else {