[coreboot-gerrit] Change in coreboot[master]: security/tpm: Report tpm_unmarshal_response warning

[Philipp Deppenwiese (Code Review)]

Philipp Deppenwiese has submitted this change and it was merged. ( https://review.coreboot.org/27931 )

Change subject: security/tpm: Report tpm_unmarshal_response warning
......................................................................

security/tpm: Report tpm_unmarshal_response warning

In procedure tpm_unmarshal_response(), variable "rc" is used early to
decide if it should return NULL. Later however, the code proceeds to its
end even if one subroutine reports error. If "rc" is not 0, report that
there was a partial error in the procedure.

BUG=b:112253891
TEST=Build and boot grunt.

Change-Id: I7575bc75104fd97f138224aa57561e68f6548e58
Signed-off-by: Richard Spiegel 
Reviewed-on: https://review.coreboot.org/27931
Reviewed-by: Martin Roth 
Reviewed-by: Marshall Dawson 
Reviewed-by: Philipp Deppenwiese 
Tested-by: build bot (Jenkins) 
---
M src/security/tpm/tss/tcg-2.0/tss_marshaling.c
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Martin Roth: Looks good to me, approved
  Philipp Deppenwiese: Looks good to me, approved
  Marshall Dawson: Looks good to me, but someone else must approve

diff --git a/src/security/tpm/tss/tcg-2.0/tss_marshaling.c b/src/security/tpm/tss/tcg-2.0/tss_marshaling.c
index 86f2231..94bfbcf 100644
--- a/src/security/tpm/tss/tcg-2.0/tss_marshaling.c
+++ b/src/security/tpm/tss/tcg-2.0/tss_marshaling.c
@@ -554,6 +554,9 @@
    	       command, ibuf_remaining(ib));
    	return NULL;
    }
+	if (rc)
+		printk(BIOS_WARNING, "Warning: %s had one or more failures.\n",
+					__func__);

/* The entire message have been parsed. */
    return tpm2_resp;