mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-23 17:30:02 +00:00
tpm_eventlog.c: fix binary_bios_measurements
The commit0cc698af36
("vTPM: support little endian guests") copied the event, but without the event data, did an endian conversion on the size and tried to output the event data from the copied version, which has only have one byte of the data, resulting in garbage event data. [jarkko.sakkinen@linux.intel.com: fixed minor coding style issues and renamed the local variable tempPtr as temp_ptr now that there is an excuse to do this.] Signed-off-by: Harald Hoyer <harald@redhat.com> Fixes:0cc698af36
("vTPM: support little endian guests") Reviewed-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> cc: stable@vger.kernel.org
This commit is contained in:
parent
4f3b193dee
commit
186d124f07
1 changed files with 10 additions and 4 deletions
|
@ -232,7 +232,7 @@ static int tpm_binary_bios_measurements_show(struct seq_file *m, void *v)
|
|||
{
|
||||
struct tcpa_event *event = v;
|
||||
struct tcpa_event temp_event;
|
||||
char *tempPtr;
|
||||
char *temp_ptr;
|
||||
int i;
|
||||
|
||||
memcpy(&temp_event, event, sizeof(struct tcpa_event));
|
||||
|
@ -242,10 +242,16 @@ static int tpm_binary_bios_measurements_show(struct seq_file *m, void *v)
|
|||
temp_event.event_type = do_endian_conversion(event->event_type);
|
||||
temp_event.event_size = do_endian_conversion(event->event_size);
|
||||
|
||||
tempPtr = (char *)&temp_event;
|
||||
temp_ptr = (char *) &temp_event;
|
||||
|
||||
for (i = 0; i < sizeof(struct tcpa_event) + temp_event.event_size; i++)
|
||||
seq_putc(m, tempPtr[i]);
|
||||
for (i = 0; i < (sizeof(struct tcpa_event) - 1) ; i++)
|
||||
seq_putc(m, temp_ptr[i]);
|
||||
|
||||
temp_ptr = (char *) v;
|
||||
|
||||
for (i = (sizeof(struct tcpa_event) - 1);
|
||||
i < (sizeof(struct tcpa_event) + temp_event.event_size); i++)
|
||||
seq_putc(m, temp_ptr[i]);
|
||||
|
||||
return 0;
|
||||
|
||||
|
|
Loading…
Reference in a new issue