[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 28/42] tpm-passthrough: don't save guessed cancel_pa
From: |
Marc-André Lureau |
Subject: |
[Qemu-devel] [PATCH 28/42] tpm-passthrough: don't save guessed cancel_path in options |
Date: |
Tue, 10 Oct 2017 00:56:09 +0200 |
The value is later unneeded, and may leak if the free visitor doesn't
consider it since has_cancel_path is false. And for consistency with
"path" it shouldn't be returned in get_tpm_options().
Signed-off-by: Marc-André Lureau <address@hidden>
---
hw/tpm/tpm_passthrough.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/hw/tpm/tpm_passthrough.c b/hw/tpm/tpm_passthrough.c
index 8c002e4da6..048edb1a1a 100644
--- a/hw/tpm/tpm_passthrough.c
+++ b/hw/tpm/tpm_passthrough.c
@@ -226,9 +226,7 @@ static int
tpm_passthrough_open_sysfs_cancel(TPMPassthruState *tpm_pt)
if (snprintf(path, sizeof(path), "/sys/class/misc/%s/device/cancel",
dev) < sizeof(path)) {
fd = qemu_open(path, O_WRONLY);
- if (fd >= 0) {
- tpm_pt->options->cancel_path = g_strdup(path);
- } else {
+ if (fd < 0) {
error_report("tpm_passthrough: Could not open TPM cancel "
"path %s : %s", path, strerror(errno));
}
--
2.14.1.146.gd35faa819
- Re: [Qemu-devel] [PATCH 23/42] tpm-tis: no longer expose TPMState, (continued)
- [Qemu-devel] [PATCH 28/42] tpm-passthrough: don't save guessed cancel_path in options,
Marc-André Lureau <=
- [Qemu-devel] [PATCH 29/42] tpm-be: update optional function pointers, Marc-André Lureau, 2017/10/09
- [Qemu-devel] [PATCH 30/42] tpm-passthrough: pass TPMPassthruState to handle_device_opts, Marc-André Lureau, 2017/10/09
- [Qemu-devel] [PATCH 31/42] tpm-backend: move set 'id' to common code, Marc-André Lureau, 2017/10/09
- Re: [Qemu-devel] [PATCH 31/42] tpm-backend: move set 'id' to common code, Stefan Berger, 2017/10/10