Date: Mon, 19 Dec 2011 23:50:32 +0100

This is a fix up for commit 3d1d6bb4 which forgot to add `EOT` at the end.

        commit 3d1d6bb4ecb15a12f48f871c623882bee9c0c576
        Author: Jonathan A. Kollasch <[email protected]>
        Date:   Mon Nov 7 10:56:42 2011 -0600

            superiotool: add detection and dump of Infineon SLB9635 TPM

Carl-Daniel explained the continuous integration tool Jenkins did not
catch that, since this is a run time bug. It accesses uninitialized
memory until it crashes and writes random data to random I/O locations.

Reported-by (on IRC): Carl-Daniel Hailfinger <[email protected]>
Signed-off-by: Paul Menzel <[email protected]>
---
 util/superiotool/superiotool.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/util/superiotool/superiotool.h b/util/superiotool/superiotool.h
index 5583ec5..94918b8 100644
--- a/util/superiotool/superiotool.h
+++ b/util/superiotool/superiotool.h
@@ -247,7 +247,7 @@ static const struct {
        {probe_idregs_amd,      {0xaa, EOT}},
 #endif
        {probe_idregs_serverengines,    {0x2e, EOT}},
-       {probe_idregs_infineon, {0x2e, 0x4e}},
+       {probe_idregs_infineon, {0x2e, 0x4e, EOT}},
 };
 
 /** Table of functions to print out supported Super I/O chips. */
-- 
1.7.7.3

Attachment: signature.asc
Description: This is a digitally signed message part

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to