Skip to content

Commit df333d0

Browse files
ColinIanKingjarkkojs
authored andcommitted
tpm: remove redundant variable len
Variable len is being accumulated but the value is never read. It is redundant and can be removed. Cleans up clang scan build warning: drivers/char/tpm/eventlog/tpm1.c:276:2: warning: Value stored to 'len' is never read [deadcode.DeadStores] Signed-off-by: Colin Ian King <colin.i.king@gmail.com> Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> Reviewed-by: Jerry Snitselaar <jsnitsel@redhat.com> Signed-off-by: Jarkko Sakkinen <jarkko@kernel.org>
1 parent 280db21 commit df333d0

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

  • drivers/char/tpm/eventlog

drivers/char/tpm/eventlog/tpm1.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,6 @@ static int tpm1_binary_bios_measurements_show(struct seq_file *m, void *v)
251251

252252
static int tpm1_ascii_bios_measurements_show(struct seq_file *m, void *v)
253253
{
254-
int len = 0;
255254
char *eventname;
256255
struct tcpa_event *event = v;
257256
unsigned char *event_entry =
@@ -273,7 +272,7 @@ static int tpm1_ascii_bios_measurements_show(struct seq_file *m, void *v)
273272
/* 3rd: event type identifier */
274273
seq_printf(m, " %02x", do_endian_conversion(event->event_type));
275274

276-
len += get_event_name(eventname, event, event_entry);
275+
get_event_name(eventname, event, event_entry);
277276

278277
/* 4th: eventname <= max + \'0' delimiter */
279278
seq_printf(m, " %s\n", eventname);

0 commit comments

Comments
 (0)