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

Commit e885c6ac authored by Treehugger Robot's avatar Treehugger Robot Committed by Gerrit Code Review
Browse files

Merge "Add assert check for INVALID NUMBERS in NUI Voicemail"

parents 056c1911 668b99f9
Loading
Loading
Loading
Loading
+9 −0
Original line number Diff line number Diff line
@@ -23,6 +23,8 @@ import android.support.v4.content.CursorLoader;
import com.android.dialer.DialerPhoneNumber;
import com.android.dialer.NumberAttributes;
import com.android.dialer.calllog.database.contract.AnnotatedCallLogContract.AnnotatedCallLog;
import com.android.dialer.common.Assert;
import com.android.dialer.common.LogUtil;
import com.android.dialer.voicemail.model.VoicemailEntry;
import com.google.protobuf.InvalidProtocolBufferException;

@@ -84,6 +86,13 @@ final class VoicemailCursorLoader extends CursorLoader {
      throw new IllegalStateException("Couldn't parse NumberAttributes bytes");
    }

    // Voicemail numbers should always be valid so the CP2 information should never be incomplete,
    // and there should be no need to query PhoneLookup at render time.
    Assert.checkArgument(
        !numberAttributes.getIsCp2InfoIncomplete(),
        "CP2 info incomplete for number: %s",
        LogUtil.sanitizePii(number.getNormalizedNumber()));

    return VoicemailEntry.builder()
        .setId(cursor.getInt(ID))
        .setTimestamp(cursor.getLong(TIMESTAMP))