[#107] Additional component fields for version 2 #121
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Zero-value hashes, and hashes of no data, are now considered as matches to equal values instead of treating them as 'unknown'.
…on and include a link to the associated Endorsement Certificate.
… the Platform Credential has a status of PASS. If it does not, no matter the outcome of the Attributes validation, the status of the Attributes can not be PASS. Added an additional null check for a platform supply validation. Added a mapping object for platform credential to the associated attributes during validations. Added an additional null check for a platform supply validation. Added a mapping object for platform credential to the associated attributes during validations. Missed import statement.
Changes how tpm20.h, which contains the TPM2 SAPI, is imported to allow successful packaging of both debs and rpms.
The CentOS7 package selinux-policy-targeted does not allow Tomcat to use port 3306, which is the default MySQL port. This commit changes the ACA RPM to modify the SELinux policy to grant that permission on fresh installs. This makes the ACA RPM now require the policycoreutils package to be installed.
[#36] TPM Provisioner status output
ImaAcceptableRecordBaseline and its subclasses have been updated to include a containsHashes method to be able to match IMA measurement records based solely on their hashes. Supporting classes have been updated or created as necessary. Additionally, the set of path equivalencies as specified in the IMA policy have been updated to include additional entries. Closes #33.
…does not error on the line setting it only to null.
…does not error on the line setting it only to null.
cyrus-dev
added this to the TCG Platform Certificate Profile Specification Version 1.1 milestone
…ty V2 as an additional safeguard against type mismatch.
…ty V2 as an additional safeguard against type mismatch.
…yber/HIRS into additional_component_fields
busaboy1340
reviewed
Apr 1, 2019
HIRS_Utils/src/main/java/hirs/data/persist/certificate/attributes/V2/ComponentIdentifierV2.java
Show resolved
Hide resolved
…S into additional_component_fields
…ted CertificateIdentifer to deal with tagged objects being optional, therefore the check for required in constructor is unnecessary.
chubtub
approved these changes
Apr 2, 2019
HIRS_Utils/src/main/java/hirs/data/persist/certificate/attributes/V2/CertificateIdentifier.java
Show resolved
Hide resolved
HIRS_Utils/src/main/java/hirs/data/persist/certificate/attributes/V2/CertificateIdentifier.java
Outdated
Show resolved
Hide resolved
busaboy1340
approved these changes
Apr 3, 2019
|
FYI, I forgot about some older test patterns that do go over 4, 5 and 6. I'm going to go over this code with them to do a sanity check. |
chubtub
approved these changes
Apr 4, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This request is an initial set up for the new tagged elements for the component identifier. Which include
componentPlatformCert [5] IMPLICIT CertificateIdentifier OPTIONAL,
componentPlatformCertUri [6] IMPLICIT URIReference OPTIONAL,
status [7] IMPLICIT AttributeStatus OPTIONAL
And an update the PlatformProperty.java class that was updated with the AttributeStatus as well.
Closes #107