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

Commit 7df715b2 authored by Shikha Panwar's avatar Shikha Panwar Committed by Automerger Merge Worker
Browse files

Secretkeeper implementation: in-HAL/nonsecure impl am: eb223ba8 am: afca7e50 am: 28877b08 

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2797756



Change-Id: I3f7e17bf6ee1dae2f9874c154c74a0bbf3341943
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 94a0388f 28877b08

compatibility_matrices/compatibility_matrix.9.xml

+8 −0
Original line number Diff line number Diff line
@@ -304,6 +304,14 @@ 
            <instance>default</instance>

        </interface>

    </hal>

    <hal format="aidl" optional="true">

        <name>android.hardware.security.secretkeeper</name>

        <version>1</version>

        <interface>

            <name>ISecretkeeper</name>

            <instance>nonsecure</instance>

        </interface>

    </hal>

    <hal format="aidl" optional="true" updatable-via-apex="true">

        <name>android.hardware.security.keymint</name>

        <version>1-3</version>

security/secretkeeper/default/Android.bp

0 → 100644
+37 −0
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2023 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */



package {

    default_applicable_licenses: ["Android-Apache-2.0"],

}



rust_binary {

    name: "android.hardware.security.secretkeeper-service.nonsecure",

    relative_install_path: "hw",

    vendor: true,

    init_rc: ["secretkeeper.rc"],

    vintf_fragments: ["secretkeeper.xml"],

    rustlibs: [

        "android.hardware.security.secretkeeper-V1-rust",

        "libandroid_logger",

        "libbinder_rs",

        "liblog_rust",

        "libsecretkeeper_comm_nostd",

    ],

    srcs: [

        "src/main.rs",

    ],

}

security/secretkeeper/default/secretkeeper.rc

0 → 100644
+5 −0
Original line number Diff line number Diff line 
service vendor.secretkeeper /vendor/bin/hw/android.hardware.security.secretkeeper-service.nonsecure

    interface aidl android.hardware.security.secretkeeper.ISecretkeeper/nonsecure

    class hal

    user nobody

    group nobody

security/secretkeeper/default/secretkeeper.xml

0 → 100644
+28 −0
Original line number Diff line number Diff line 
<manifest version="1.0" type="device">

<!--

/*

** Copyright 2022, The Android Open Source Project.

**

** Licensed under the Apache License, Version 2.0 (the "License");

** you may not use this file except in compliance with the License.

** You may obtain a copy of the License at

**

**     http://www.apache.org/licenses/LICENSE-2.0

**

** Unless required by applicable law or agreed to in writing, software

** distributed under the License is distributed on an "AS IS" BASIS,

** WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

** See the License for the specific language governing permissions and

** limitations under the License.

*/

-->



    <hal format="aidl">

        <name>android.hardware.security.secretkeeper</name>

        <version>1</version>

        <interface>

            <name>ISecretkeeper</name>

            <instance>nonsecure</instance>

        </interface>

    </hal>

</manifest>

security/secretkeeper/default/src/main.rs

0 → 100644
+121 −0
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2023 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */



use binder::{BinderFeatures, Interface};

use log::{error, info, Level};

use secretkeeper_comm::data_types::error::SecretkeeperError;

use secretkeeper_comm::data_types::packet::{RequestPacket, ResponsePacket};

use secretkeeper_comm::data_types::request::Request;

use secretkeeper_comm::data_types::request_response_impl::{

    GetVersionRequest, GetVersionResponse, Opcode,

};

use secretkeeper_comm::data_types::response::Response;



use android_hardware_security_secretkeeper::aidl::android::hardware::security::secretkeeper::ISecretkeeper::{

    BnSecretkeeper, BpSecretkeeper, ISecretkeeper,

};



const CURRENT_VERSION: u64 = 1;



#[derive(Debug, Default)]

pub struct NonSecureSecretkeeper;



impl Interface for NonSecureSecretkeeper {}



impl ISecretkeeper for NonSecureSecretkeeper {

    fn processSecretManagementRequest(&self, request: &[u8]) -> binder::Result<Vec<u8>> {

        Ok(self.process_opaque_request(request))

    }

}



impl NonSecureSecretkeeper {

    // A set of requests to Secretkeeper are 'opaque' - encrypted bytes with inner structure

    // described by CDDL. They need to be decrypted, deserialized and processed accordingly.

    fn process_opaque_request(&self, request: &[u8]) -> Vec<u8> {

        // TODO(b/291224769) The request will need to be decrypted & response need to be encrypted

        // with key & related artifacts pre-shared via Authgraph Key Exchange HAL.

        self.process_opaque_request_unhandled_error(request)

            .unwrap_or_else(

                // SecretkeeperError is also a valid 'Response', serialize to a response packet.

                |sk_err| {

                    Response::serialize_to_packet(&sk_err)

                        .into_bytes()

                        .expect("Panicking due to serialization failing")

                },

            )

    }



    fn process_opaque_request_unhandled_error(

        &self,

        request: &[u8],

    ) -> Result<Vec<u8>, SecretkeeperError> {

        let request_packet = RequestPacket::from_bytes(request).map_err(|e| {

            error!("Failed to get Request packet from bytes: {:?}", e);

            SecretkeeperError::RequestMalformed

        })?;

        let response_packet = match request_packet

            .opcode()

            .map_err(|_| SecretkeeperError::RequestMalformed)?

        {

            Opcode::GetVersion => Self::process_get_version_request(request_packet)?,

            _ => todo!("TODO(b/291224769): Unimplemented operations"),

        };



        response_packet

            .into_bytes()

            .map_err(|_| SecretkeeperError::UnexpectedServerError)

    }



    fn process_get_version_request(

        request: RequestPacket,

    ) -> Result<ResponsePacket, SecretkeeperError> {

        // Deserialization really just verifies the structural integrity of the request such

        // as args being empty.

        let _request = GetVersionRequest::deserialize_from_packet(request)

            .map_err(|_| SecretkeeperError::RequestMalformed)?;

        let response = GetVersionResponse::new(CURRENT_VERSION);

        Ok(response.serialize_to_packet())

    }

}



fn main() {

    // Initialize Android logging.

    android_logger::init_once(

        android_logger::Config::default()

            .with_tag("NonSecureSecretkeeper")

            .with_min_level(Level::Info)

            .with_log_id(android_logger::LogId::System),

    );

    // Redirect panic messages to logcat.

    std::panic::set_hook(Box::new(|panic_info| {

        error!("{}", panic_info);

    }));



    let service = NonSecureSecretkeeper::default();

    let service_binder = BnSecretkeeper::new_binder(service, BinderFeatures::default());

    let service_name = format!(

        "{}/nonsecure",

        <BpSecretkeeper as ISecretkeeper>::get_descriptor()

    );

    binder::add_service(&service_name, service_binder.as_binder()).unwrap_or_else(|e| {

        panic!(

            "Failed to register service {} because of {:?}.",

            service_name, e

        );

    });

    info!("Registered Binder service, joining threadpool.");

    binder::ProcessState::join_thread_pool();

}