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

Commit 610d0967 authored by Mathew Inwood's avatar Mathew Inwood
Browse files

Log the fact that we don't have a prod key. 

Hopefully this will never happen, but better to know if it does.

Test: atest SignedConfigHostTest
Bug: 122579497
Change-Id: I4ecdf94f7963728aecf17a448c7bfbc7dbd5f39e
parent 45942518

cmds/statsd/src/atoms.proto

+1 −0
Original line number Diff line number Diff line
@@ -3884,6 +3884,7 @@ message SignedConfigReported { 
        OLD_CONFIG = 6;

        SIGNATURE_CHECK_FAILED = 7;

        NOT_APPLICABLE = 8;

        SIGNATURE_CHECK_FAILED_PROD_KEY_ABSENT = 9;

    }

    optional Status status = 2;

services/core/java/com/android/server/signedconfig/SignatureVerifier.java

+2 −0
Original line number Diff line number Diff line
@@ -119,6 +119,8 @@ public class SignatureVerifier { 
        }

        if (mProdKey ==  null) {

            Slog.e(TAG, "No prod key; construction failed?");

            mEvent.status =

                    StatsLog.SIGNED_CONFIG_REPORTED__STATUS__SIGNATURE_CHECK_FAILED_PROD_KEY_ABSENT;

            return false;

        }

        if (verifyWithPublicKey(mProdKey, data, signature)) {