Edward O'Callaghan submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Thomas Heijligen: Looks good to me, approved Angel Pons: Looks good to me, but someone else must approve Nikolai Artemiev: Looks good to me, but someone else must approve
dmi.c: Ensure g_has_dmi_support is default on shutdown

Ensure the g_has_dmi_support variable has the default state
of false after the life-time has expired.

BUG=none
TEST=builds

Change-Id: I0674950304736e53d014117d287682a4f6349879
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/62996
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Thomas Heijligen <src@posteo.de>
Reviewed-by: Nikolai Artemiev <nartemiev@google.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
---
M dmi.c
1 file changed, 1 insertion(+), 0 deletions(-)

diff --git a/dmi.c b/dmi.c
index a82b494..7c0fc4c 100644
--- a/dmi.c
+++ b/dmi.c
@@ -386,6 +386,7 @@
free(dmi_strings[i].value);
dmi_strings[i].value = NULL;
}
+ g_has_dmi_support = false;
return 0;
}


1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.

To view, visit change 62996. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I0674950304736e53d014117d287682a4f6349879
Gerrit-Change-Number: 62996
Gerrit-PatchSet: 3
Gerrit-Owner: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Nikolai Artemiev <nartemiev@google.com>
Gerrit-Reviewer: Thomas Heijligen <src@posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-MessageType: merged