lkml.org 
[lkml]   [2006]   [May]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 1/2] tpm: bios log parsing fixes
From
Date
Ack'ed by: Kylene Hall <kjhall@us.ibm.com>

On Fri, 2006-05-19 at 08:27 +0900, Seiji Munetoh wrote:
> This patch fixes "tcpa_pc_event" misalignment between enum, strings and
> TCG PC spec and output of the event contains a hash data.
>
> Signed-off-by: Seiji Munetoh <seiji.munetoh@gmail.com>
> --- linux-2.6.17-rc4/drivers/char/tpm/tpm_bios.c 2006-05-16
> 09:33:06.000000000 +0900
> +++ linux-2.6.17-rc4-tpm/drivers/char/tpm/tpm_bios.c 2006-05-19
> 08:12:30.000000000 +0900
> @@ -105,6 +105,12 @@ static const char* tcpa_event_type_strin
> "Non-Host Info"
> };
>
> +struct tcpa_pc_event {
> + u32 event_id;
> + u32 event_size;
> + u8 event_data[0];
> +};
> +
> enum tcpa_pc_event_ids {
> SMBIOS = 1,
> BIS_CERT,
> @@ -114,14 +120,16 @@ enum tcpa_pc_event_ids {
> NVRAM,
> OPTION_ROM_EXEC,
> OPTION_ROM_CONFIG,
> + UNUSED2,
> OPTION_ROM_MICROCODE,
> S_CRTM_VERSION,
> S_CRTM_CONTENTS,
> POST_CONTENTS,
> + HOST_TABLE_OF_DEVICES,
> };
>
> static const char* tcpa_pc_event_id_strings[] = {
> - ""
> + "",
> "SMBIOS",
> "BIS Certificate",
> "POST BIOS ",
> @@ -130,11 +138,12 @@ static const char* tcpa_pc_event_id_stri
> "NVRAM",
> "Option ROM",
> "Option ROM config",
> - "Option ROM microcode",
> + "",
> + "Option ROM microcode ",
> "S-CRTM Version",
> - "S-CRTM Contents",
> - "S-CRTM POST Contents",
> - "POST Contents",
> + "S-CRTM Contents ",
> + "POST Contents ",
> + "Table of Devices",
> };
>
> /* returns pointer to start of pos. entry of tcg log */
> @@ -206,7 +215,7 @@ static int get_event_name(char *dest, st
> const char *name = "";
> char data[40] = "";
> int i, n_len = 0, d_len = 0;
> - u32 event_id;
> + struct tcpa_pc_event *pc_event;
>
> switch(event->event_type) {
> case PREBOOT:
> @@ -235,31 +244,32 @@ static int get_event_name(char *dest, st
> }
> break;
> case EVENT_TAG:
> - event_id = be32_to_cpu(*((u32 *)event_entry));
> + pc_event = (struct tcpa_pc_event *)event_entry;
>
> /* ToDo Row data -> Base64 */
>
> - switch (event_id) {
> + switch (pc_event->event_id) {
> case SMBIOS:
> case BIS_CERT:
> case CMOS:
> case NVRAM:
> case OPTION_ROM_EXEC:
> case OPTION_ROM_CONFIG:
> - case OPTION_ROM_MICROCODE:
> case S_CRTM_VERSION:
> - case S_CRTM_CONTENTS:
> - case POST_CONTENTS:
> - name = tcpa_pc_event_id_strings[event_id];
> + name = tcpa_pc_event_id_strings[pc_event->event_id];
> n_len = strlen(name);
> break;
> + /* hash data */
> case POST_BIOS_ROM:
> case ESCD:
> - name = tcpa_pc_event_id_strings[event_id];
> + case OPTION_ROM_MICROCODE:
> + case S_CRTM_CONTENTS:
> + case POST_CONTENTS:
> + name = tcpa_pc_event_id_strings[pc_event->event_id];
> n_len = strlen(name);
> for (i = 0; i < 20; i++)
> - d_len += sprintf(data, "%02x",
> - event_entry[8 + i]);
> + d_len += sprintf(&data[2*i], "%02x",
> + pc_event->event_data[i]);
> break;
> default:
> break;
>
>

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-05-19 03:23    [W:0.025 / U:2.384 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site