Hans Leidekker : ntdll: Use a common structure for the SMBIOS table entry header.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 16e5c78bfe8d261be895a7806d2305b38d2ebc0f
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=16e5c78bfe8d261be895a7806...

Author: Hans Leidekker <hans(a)codeweavers.com>
Date:   Tue Jul  2 15:28:34 2019 +0200

ntdll: Use a common structure for the SMBIOS table entry header.

Signed-off-by: Hans Leidekker <hans(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 dlls/ntdll/nt.c | 42 ++++++++++++++++++++----------------------
 1 file changed, 20 insertions(+), 22 deletions(-)

diff --git a/dlls/ntdll/nt.c b/dlls/ntdll/nt.c
index 2a2bd4f..7cfe57c 100644
--- a/dlls/ntdll/nt.c
+++ b/dlls/ntdll/nt.c
@@ -79,10 +79,14 @@ struct smbios_prologue {
     DWORD length;
 };
 
-struct smbios_bios {
+struct smbios_header {
     BYTE type;
     BYTE length;
     WORD handle;
+};
+
+struct smbios_bios {
+    struct smbios_header hdr;
     BYTE vendor;
     BYTE version;
     WORD start;
@@ -92,9 +96,7 @@ struct smbios_bios {
 };
 
 struct smbios_system {
-    BYTE type;
-    BYTE length;
-    WORD handle;
+    struct smbios_header hdr;
     BYTE vendor;
     BYTE product;
     BYTE version;
@@ -102,9 +104,7 @@ struct smbios_system {
 };
 
 struct smbios_board {
-    BYTE type;
-    BYTE length;
-    WORD handle;
+    struct smbios_header hdr;
     BYTE vendor;
     BYTE product;
     BYTE version;
@@ -112,9 +112,7 @@ struct smbios_board {
 };
 
 struct smbios_chassis {
-    BYTE type;
-    BYTE length;
-    WORD handle;
+    struct smbios_header hdr;
     BYTE vendor;
     BYTE shape;
     BYTE version;
@@ -2139,9 +2137,9 @@ static NTSTATUS get_firmware_info(SYSTEM_FIRMWARE_TABLE_INFORMATION *sfti, ULONG
 
             string_count = 0;
             bios = (struct smbios_bios*)buffer;
-            bios->type = 0;
-            bios->length = sizeof(struct smbios_bios);
-            bios->handle = 0;
+            bios->hdr.type = 0;
+            bios->hdr.length = sizeof(struct smbios_bios);
+            bios->hdr.handle = 0;
             bios->vendor = bios_vendor_len ? ++string_count : 0;
             bios->version = bios_version_len ? ++string_count : 0;
             bios->start = 0;
@@ -2158,9 +2156,9 @@ static NTSTATUS get_firmware_info(SYSTEM_FIRMWARE_TABLE_INFORMATION *sfti, ULONG
 
             string_count = 0;
             system = (struct smbios_system*)buffer;
-            system->type = 1;
-            system->length = sizeof(struct smbios_system);
-            system->handle = 0;
+            system->hdr.type = 1;
+            system->hdr.length = sizeof(struct smbios_system);
+            system->hdr.handle = 0;
             system->vendor = system_vendor_len ? ++string_count : 0;
             system->product = system_product_len ? ++string_count : 0;
             system->version = system_version_len ? ++string_count : 0;
@@ -2176,9 +2174,9 @@ static NTSTATUS get_firmware_info(SYSTEM_FIRMWARE_TABLE_INFORMATION *sfti, ULONG
 
             string_count = 0;
             board = (struct smbios_board*)buffer;
-            board->type = 2;
-            board->length = sizeof(struct smbios_board);
-            board->handle = 0;
+            board->hdr.type = 2;
+            board->hdr.length = sizeof(struct smbios_board);
+            board->hdr.handle = 0;
             board->vendor = board_vendor_len ? ++string_count : 0;
             board->product = board_product_len ? ++string_count : 0;
             board->version = board_version_len ? ++string_count : 0;
@@ -2194,9 +2192,9 @@ static NTSTATUS get_firmware_info(SYSTEM_FIRMWARE_TABLE_INFORMATION *sfti, ULONG
 
             string_count = 0;
             chassis = (struct smbios_chassis*)buffer;
-            chassis->type = 3;
-            chassis->length = sizeof(struct smbios_chassis);
-            chassis->handle = 0;
+            chassis->hdr.type = 3;
+            chassis->hdr.length = sizeof(struct smbios_chassis);
+            chassis->hdr.handle = 0;
             chassis->vendor = chassis_vendor_len ? ++string_count : 0;
             chassis->shape = 0x2; /* unknown */
             chassis->version = chassis_version_len ? ++string_count : 0;