[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH] spapr: Fix Coverity warning while validating nvdimm options
From: |
Shivaprasad G Bhat |
Subject: |
[PATCH] spapr: Fix Coverity warning while validating nvdimm options |
Date: |
Wed, 26 Feb 2020 06:10:38 -0600 |
User-agent: |
StGit/0.17.1-dirty |
Fixes Coverity issue,
CID 1419883: Error handling issues (CHECKED_RETURN)
Calling "qemu_uuid_parse" without checking return value
nvdimm_set_uuid() already verifies if the user provided uuid is valid or
not. So, need to check for the validity during pre-plug validation again.
As this a false positive in this case, assert if not valid to be safe.
Reported-by: Coverity (CID 1419883)
Signed-off-by: Shivaprasad G Bhat <address@hidden>
---
hw/ppc/spapr_nvdimm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/ppc/spapr_nvdimm.c b/hw/ppc/spapr_nvdimm.c
index 74eeb8bb74..051727536e 100644
--- a/hw/ppc/spapr_nvdimm.c
+++ b/hw/ppc/spapr_nvdimm.c
@@ -44,7 +44,7 @@ void spapr_nvdimm_validate_opts(NVDIMMDevice *nvdimm,
uint64_t size,
}
uuidstr = object_property_get_str(OBJECT(nvdimm), NVDIMM_UUID_PROP, NULL);
- qemu_uuid_parse(uuidstr, &uuid);
+ g_assert(qemu_uuid_parse(uuidstr, &uuid) == 0);
g_free(uuidstr);
if (qemu_uuid_is_null(&uuid)) {
- [PATCH] spapr: Fix Coverity warning while validating nvdimm options,
Shivaprasad G Bhat <=