diff options
| author | Markus Armbruster <armbru@redhat.com> | 2013-06-07 15:00:36 +0200 |
|---|---|---|
| committer | Anthony Liguori <aliguori@us.ibm.com> | 2013-06-14 08:58:58 -0500 |
| commit | 6e5c4540d18d1e9a5253104df161a7e0d408ca95 (patch) | |
| tree | 6f810fb56c9949a91697b32e6842697754225411 /hw/i386/smbios.c | |
| parent | 527cd96f15348f4454d5bb71f69aaeb95461c90e (diff) | |
| download | qemu-6e5c4540d18d1e9a5253104df161a7e0d408ca95.tar.gz | |
smbios: Check R in -smbios type=0, release=R parses okay
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Laszlo "ever the optimist" Ersek <lersek@redhat.com>
Message-id: 1370610036-10577-7-git-send-email-armbru@redhat.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Diffstat (limited to 'hw/i386/smbios.c')
| -rw-r--r-- | hw/i386/smbios.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/hw/i386/smbios.c b/hw/i386/smbios.c index 6431dd4d84..e708cb8919 100644 --- a/hw/i386/smbios.c +++ b/hw/i386/smbios.c @@ -140,7 +140,10 @@ static void smbios_build_type_0_fields(const char *t) bios_release_date_str), buf, strlen(buf) + 1); if (get_param_value(buf, sizeof(buf), "release", t)) { - sscanf(buf, "%hhu.%hhu", &major, &minor); + if (sscanf(buf, "%hhu.%hhu", &major, &minor) != 2) { + error_report("Invalid release"); + exit(1); + } smbios_add_field(0, offsetof(struct smbios_type_0, system_bios_major_release), &major, 1); |