Coverity reported the existing missing check of the return value and
subsequent use from a call to virJSONValueFromString() in testJSONAddRemove().
---
tests/jsontest.c | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/tests/jsontest.c b/tests/jsontest.c
index a37a980..1d385d4 100644
--- a/tests/jsontest.c
+++ b/tests/jsontest.c
@@ -60,11 +60,17 @@ testJSONAddRemove(const void *data)
{
const struct testInfo *info = data;
virJSONValuePtr json;
- virJSONValuePtr name;
+ virJSONValuePtr name = NULL;
char *result = NULL;
int ret = -1;
json = virJSONValueFromString(info->doc);
+ if (!json) {
+ if (virTestGetVerbose())
+ fprintf(stderr, "Fail to parse %s\n", info->doc);
+ ret = -1;
+ goto cleanup;
+ }
switch (virJSONValueObjectRemoveKey(json, "name", &name)) {
case 1:
--
1.8.1.4