diff options
| author | Karel Zak | 2013-10-01 17:26:33 +0200 |
|---|---|---|
| committer | Karel Zak | 2013-10-01 17:26:33 +0200 |
| commit | bbff0890581a283523d16eb83c09efcc6a5421e3 (patch) | |
| tree | f2642424cee77267257d2b990bcd8113cc3dfdb5 /sys-utils/lscpu-dmi.c | |
| parent | libmount: remove dead code [coverity scan] (diff) | |
| download | kernel-qcow2-util-linux-bbff0890581a283523d16eb83c09efcc6a5421e3.tar.gz kernel-qcow2-util-linux-bbff0890581a283523d16eb83c09efcc6a5421e3.tar.xz kernel-qcow2-util-linux-bbff0890581a283523d16eb83c09efcc6a5421e3.zip | |
lscpu: don't pass null pointer to string functions [coverity scan]
Signed-off-by: Karel Zak <kzak@redhat.com>
Diffstat (limited to 'sys-utils/lscpu-dmi.c')
| -rw-r--r-- | sys-utils/lscpu-dmi.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/sys-utils/lscpu-dmi.c b/sys-utils/lscpu-dmi.c index 8f02779d6..852817df1 100644 --- a/sys-utils/lscpu-dmi.c +++ b/sys-utils/lscpu-dmi.c @@ -155,11 +155,12 @@ static int hypervisor_from_dmi_table(uint32_t base, uint16_t len, data = next; i++; } - if (!strcmp(manufacturer, "innotek GmbH")) + if (manufacturer && !strcmp(manufacturer, "innotek GmbH")) return HYPER_INNOTEK; - else if (strstr(manufacturer, "HITACHI") && strstr(product, "LPAR")) + else if (manufacturer && strstr(manufacturer, "HITACHI") && + product && strstr(product, "LPAR")) return HYPER_HITACHI; - else if (!strcmp(vendor, "Parallels")) + else if (!vendor && strcmp(vendor, "Parallels")) return HYPER_PARALLELS; free(buf); |