Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1566939

Re: [PATCH v6 0/2] enhance TPM 2.0 extend function to support multiple PCR banks

From Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Newsgroups linux.kernel
Subject Re: [PATCH v6 0/2] enhance TPM 2.0 extend function to support multiple PCR banks
Date 2017-01-25 21:50 +0100
Message-ID <t3CFY-6Do-19@gated-at.bofh.it> (permalink)
References <t1LX4-2d6-15@gated-at.bofh.it>
Organization Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo

Show all headers | View raw

On Fri, Jan 20, 2017 at 12:05:11PM -0500, Nayna Jain wrote:
> IMA extends its hash measurements in the TPM PCRs, based on policy.
> The existing in-kernel TPM extend function extends only the SHA1
> PCR bank. TPM 2.0 defines multiple PCR banks, to support different
> hash algorithms. The TCG TPM 2.0 Specification[1] recommends
> extending all active PCR banks to prevent malicious users from
> setting unused PCR banks with fake measurements and quoting them.
> This patch set adds support for extending all active PCR banks,
> as recommended.
> 
> The first patch implements the TPM 2.0 capability to retrieve
> the list of active PCR banks.
> 
> The second patch modifies the tpm_pcr_extend() and tpm2_pcr_extend()
> interface to support extending multiple PCR banks. The existing
> tpm_pcr_extend() interface expects only a SHA1 digest. Hence, to
> extend all active PCR banks with differing digest sizes for TPM 2.0,
> the SHA1 digest is padded with 0's as needed.
> 
> [1] TPM 2.0 Specification referred here is "TCG PC Client Specific
> Platform Firmware Profile for TPM 2.0"

I pushed these patches. I had to resolve merge conflicts caused
by the min_rsp_body_length parameter in tpm_transmit_cmd. Can you
verify that I didn't break anything?

/Jarkko

> 
> Changelog v6:
> - Patch "tpm: implement TPM 2.0 capability to get active PCR banks"
>   - Fixed the regression - missing tpm_buf_destroy() in
>   in tpm2_get_pcr_allocation(). Thanks Jarkko for noticing.
>   - Added TPM2_ALG_ERROR = 0x0000 to represent invalid algorithm.
> 
> Changelog v5:
> - Patch "tpm: implement TPM 2.0 capability to get active PCR banks"
>  - Included Jarkko's feedbacks
>    - Moved variable declaration to start of function in
>    tpm_pcr_extend()
> 
> 
> Changelog v4:
> - Updated cover letter as per Mimi's feedback.
> - Rebased to Jarkko's latest master branch (4064b6b tpm_tis: use
>   default timeout value if chip reports it as zero)
> - Patch "tpm: implement TPM 2.0 capability to get active PCR banks"
>  - Included Jarkko's feedbacks
>    - Moved call to tpm2_get_pcr_allocation to Patch 2
>    - Renamed struct tpm2_tpms_pcr_selection to struct tpm2_pcr_selection 
>    and moved the struct to before tpm2_get_pcr_allocation()
>    - Fixed code formatting
> - Patch "tpm: enchance TPM 2.0 PCR extend to support multiple banks"
>  - Included Jarkkos' feedbacks
>    - Updated commit msg to mention dependency on CRYPTO_HASH_INFO
>    - Renamed struct tpmt_hash to struct tpm2_digest 
>    - Removed struct tpml_digest_values, tpm2_pcr_extend() now accepts
>    count and digests list as two separate arguments. Added check for
>    count of hashes passed.
>  - Cleaned up struct tpm2_pcr_extend_in as not required anymore with
>  use of tpm_buf
>  - Moved struct tpm2_null_auth_area just before tpm2_pcr_extend() as
>  it is the only function using it for now.
>  - Fixed code formatting
> 
> Changelog v3:
> - Rebased to the Jarkko's latest master branch (8e25809 tpm:
>   Do not print an error message when doing TPM auto startup)
> - Patch "tpm: implement TPM 2.0 capability to get active PCR banks"
>   - Included Jarkko's feedbacks
>      - Removed getcap_in, getcap_out and used tpm_buf for getting
>      capability.
>      - Used ARRAY_SIZE in place of TPM_MAX_PCR_BANKS and included
>      other feedbacks.
> - Patch "tpm: enhance TPM 2.0 PCR extend to support multiple banks"
>      - Fixed kbuild errors
>        - Fixed buf.data uninitialized warning.
>        - Added TCG_TPM dependency on CONFIG_CRYPTO_HASH_INFO in Kconfig.
> 
> Changelog v2:
> 
> - Patch "tpm: implement TPM 2.0 capability to get active PCR banks"
>   - defined structs definition in tpm2-cmd.c.
>   - no_of_active_banks field is removed. Instead, constant
>   TPM2_MAX_PCR_BANKS is defined.
>   - renamed tpm2_get_active_pcr_banks() to tpm2_get_pcr_allocation()
>   - removed generic function tpm2_get_capability().
> 
> - Patch "tpm: enchance TPM 2.0 PCR extend to support multiple banks"
>  - Removed tpm2.h, and defined structs common for extend and event log
>   in tpm_eventlog.h
>  - uses tpm_buf in tpm2_pcr_extend().
> 
> Nayna Jain (2):
>   tpm: implement TPM 2.0 capability to get active PCR banks
>   tpm: enhance TPM 2.0 PCR extend to support multiple banks
> 
>  drivers/char/tpm/Kconfig         |   1 +
>  drivers/char/tpm/tpm-interface.c |  15 +++-
>  drivers/char/tpm/tpm.h           |   8 ++-
>  drivers/char/tpm/tpm2-cmd.c      | 150 ++++++++++++++++++++++++++++-----------
>  drivers/char/tpm/tpm_eventlog.h  |   7 ++
>  5 files changed, 137 insertions(+), 44 deletions(-)
> 
> -- 
> 2.5.0
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-security-module" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v6 0/2] enhance TPM 2.0 extend function to support multiple PCR banks  Nayna Jain <nayna@linux.vnet.ibm.com> - 2017-01-20 19:20 +0100
  [PATCH v6 1/2] tpm: implement TPM 2.0 capability to get active PCR banks Nayna Jain <nayna@linux.vnet.ibm.com> - 2017-01-20 20:30 +0100
    Re: [tpmdd-devel] [PATCH v6 1/2] tpm: implement TPM 2.0 capability to  get active PCR banks Stefan Berger <stefanb@linux.vnet.ibm.com> - 2017-01-26 13:30 +0100
      Re: [tpmdd-devel] [PATCH v6 1/2] tpm: implement TPM 2.0 capability  to get active PCR banks Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-01-27 07:40 +0100
  Re: [PATCH v6 0/2] enhance TPM 2.0 extend function to support  multiple PCR banks Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-01-20 22:00 +0100
  [PATCH v6 2/2] tpm: enhance TPM 2.0 PCR extend to support multiple banks Nayna Jain <nayna@linux.vnet.ibm.com> - 2017-01-21 00:00 +0100
    Re: [PATCH v6 2/2] tpm: enhance TPM 2.0 PCR extend to support  multiple banks Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-01-23 16:20 +0100
      Re: [PATCH v6 2/2] tpm: enhance TPM 2.0 PCR extend to support  multiple banks Nayna <nayna@linux.vnet.ibm.com> - 2017-01-23 17:50 +0100
        Re: [PATCH v6 2/2] tpm: enhance TPM 2.0 PCR extend to support  multiple banks Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-01-24 13:00 +0100
          Re: [PATCH v6 2/2] tpm: enhance TPM 2.0 PCR extend to support  multiple banks Nayna <nayna@linux.vnet.ibm.com> - 2017-01-24 14:10 +0100
            Re: [PATCH v6 2/2] tpm: enhance TPM 2.0 PCR extend to support  multiple banks Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-01-25 20:50 +0100
  Re: [PATCH v6 0/2] enhance TPM 2.0 extend function to support  multiple PCR banks Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-01-25 21:50 +0100
    Re: [tpmdd-devel] [PATCH v6 0/2] enhance TPM 2.0 extend function to  support multiple PCR banks Stefan Berger <stefanb@linux.vnet.ibm.com> - 2017-01-25 22:10 +0100
      Re: [tpmdd-devel] [PATCH v6 0/2] enhance TPM 2.0 extend function to  support multiple PCR banks Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-01-25 23:10 +0100
        Re: [tpmdd-devel] [PATCH v6 0/2] enhance TPM 2.0 extend function to  support multiple PCR banks Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-01-26 00:00 +0100
          Re: [tpmdd-devel] [PATCH v6 0/2] enhance TPM 2.0 extend function to  support multiple PCR banks Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-01-26 00:00 +0100
    Re: [PATCH v6 0/2] enhance TPM 2.0 extend function to support  multiple PCR banks Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-01-26 00:40 +0100
      Re: [PATCH v6 0/2] enhance TPM 2.0 extend function to support  multiple PCR banks Nayna <nayna@linux.vnet.ibm.com> - 2017-01-26 15:40 +0100
    Re: [PATCH v6 0/2] enhance TPM 2.0 extend function to support  multiple PCR banks Nayna <nayna@linux.vnet.ibm.com> - 2017-01-27 18:30 +0100
      Re: [PATCH v6 0/2] enhance TPM 2.0 extend function to support  multiple PCR banks Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-01-29 16:20 +0100
    Re: [PATCH v6 0/2] enhance TPM 2.0 extend function to support  multiple PCR banks Nayna <nayna@linux.vnet.ibm.com> - 2017-01-27 19:40 +0100

csiph-web