From 69a1c9ca658c9708698cde6aee32af7bb2e16f9a Mon Sep 17 00:00:00 2001
From: David Sommerseth <davids@redhat.com>
Date: Thu, 20 Jun 2013 12:58:12 +0200
Subject: [PATCH] Fixed a missing break statement in a switch for DMI section
3.3.7.2
This missing break could cause duplicated 'installed' attributes in
<InstalledSize/> or <EnabledSize/> XML tags. This is only happening
when dmi_memory_module_size() is called and only on some hardware.
Signed-off-by: David Sommerseth <davids@redhat.com>
---
src/dmidecode.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/dmidecode.c b/src/dmidecode.c
index 215c3f4..dae2fef 100644
--- a/src/dmidecode.c
+++ b/src/dmidecode.c
@@ -1516,6 +1516,7 @@ void dmi_memory_module_size(xmlNode *node, const char *tagname, u8 code)
case 0x7F:
dmixml_AddAttribute(data_n, "installed", "0");
check_conn = 0;
+ break;
default:
dmixml_AddAttribute(data_n, "installed", "1");
dmixml_AddAttribute(data_n, "unit", "MB");
--
1.7.10.2