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

Commit e7890cfa authored by Rakesh Iyer's avatar Rakesh Iyer
Browse files

Don't leak PII into the logs. 

Guard a couple of the Log.v statements with the if (V) guard that
is used everywhere else in the file.

Bug: 17531755
Change-Id: I7ea06333132c400fca225d0c92cfca0d6c3024ab
parent 700ea968

android/app/src/com/android/bluetooth/pbap/BluetoothPbapVcardManager.java

+3 −3
Original line number Diff line number Diff line
@@ -574,10 +574,10 @@ public class BluetoothPbapVcardManager { 
                                + composer.getErrorReason());

                        return ResponseCodes.OBEX_HTTP_INTERNAL_ERROR;

                    }

                    Log.v (TAG , "vCard from composer: " + vcard);

                    if (V) Log.v (TAG , "vCard from composer: " + vcard);

                    if (!ignorefilter) {

                        vcard = vcardfilter.applyFilter(vcard, vcardType21);

                        Log.v (TAG , "vCard on applying filter: " + vcard);

                        if (V) Log.v (TAG , "vCard on applying filter: " + vcard);

                    }

                    vcard = StripTelephoneNumber(vcard);

                    if (V) {
@@ -663,7 +663,7 @@ public class BluetoothPbapVcardManager { 
                    Vcard = Vcard.concat(attr[i] + "\n");

                }

            }

        Log.v(TAG, "Vcard with stripped telephone no.: " + Vcard);

        if (V) Log.v(TAG, "Vcard with stripped telephone no.: " + Vcard);

        return Vcard;

    }