diff options
| author | Konstantin Aladyshev <aladyshev22@gmail.com> | 2017-08-01 15:52:46 +0300 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-08-10 15:56:45 +0000 |
| commit | d0df1d7c4ebd4d8e654c8350c27a0d9f749a88f6 (patch) | |
| tree | 1ebb35d72d65917e20cec93746cae9c5d753cbc7 /src/drivers/intel | |
| parent | 0722613563247d41caa406c176ee452465df3572 (diff) | |
SMBIOS: Correct length calculation for empty string table
If all strings in SMBIOS table are empty, smbios_string_table_len
function should return 2, cause every table must end with "\0\0".
Also replace "eos" field type in smbios structures
from char to u8.
Change-Id: Ia3178b0030aa71e1ff11a3fd3d102942f0027eb1
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Reviewed-on: https://review.coreboot.org/20840
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Diffstat (limited to 'src/drivers/intel')
| -rw-r--r-- | src/drivers/intel/wifi/wifi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/drivers/intel/wifi/wifi.c b/src/drivers/intel/wifi/wifi.c index d1d5684a19..77918e42d0 100644 --- a/src/drivers/intel/wifi/wifi.c +++ b/src/drivers/intel/wifi/wifi.c @@ -40,7 +40,7 @@ static int smbios_write_wifi(struct device *dev, int *handle, u8 length; u16 handle; u8 str; - char eos[2]; + u8 eos[2]; } __packed; struct smbios_type_intel_wifi *t = |