提交 b3286a51 编写于 作者: P Peter Krempa

qemu: command: Return props as return value in qemuBuildPRManagerInfoProps

Also since we don't do any conditional formatting, fix the comment for
the function.
Signed-off-by: NPeter Krempa <pkrempa@redhat.com>
Reviewed-by: NJán Tomko <jtomko@redhat.com>
上级 00bf6cc1
......@@ -9697,20 +9697,20 @@ qemuBuildPanicCommandLine(virCommandPtr cmd,
/**
* qemuBuildPRManagerInfoProps:
* @src: storage source
* @propsret: Returns JSON object containing properties of the pr-manager-helper object
*
* Build the JSON properties for the pr-manager object.
*
* Returns: 0 on success (@propsret is NULL if no properties are needed),
* -1 on failure (with error message set).
*/
int
qemuBuildPRManagerInfoProps(virStorageSourcePtr src,
virJSONValuePtr *propsret)
virJSONValuePtr
qemuBuildPRManagerInfoProps(virStorageSourcePtr src)
{
return qemuMonitorCreateObjectProps(propsret,
"pr-manager-helper", src->pr->mgralias,
"s:path", src->pr->path, NULL);
virJSONValuePtr ret = NULL;
if (qemuMonitorCreateObjectProps(&ret,
"pr-manager-helper", src->pr->mgralias,
"s:path", src->pr->path, NULL) < 0)
return NULL;
return ret;
}
......@@ -9737,7 +9737,7 @@ qemuBuildMasterPRCommandLine(virCommandPtr cmd,
managedAdded = true;
}
if (qemuBuildPRManagerInfoProps(disk->src, &props) < 0)
if (!(props = qemuBuildPRManagerInfoProps(disk->src)))
goto cleanup;
if (virQEMUBuildObjectCommandlineFromJSON(&buf, props) < 0)
......
......@@ -55,8 +55,7 @@ virCommandPtr qemuBuildCommandLine(virQEMUDriverPtr driver,
int **nicindexes);
/* Generate the object properties for pr-manager */
int qemuBuildPRManagerInfoProps(virStorageSourcePtr src,
virJSONValuePtr *propsret);
virJSONValuePtr qemuBuildPRManagerInfoProps(virStorageSourcePtr src);
/* Generate the object properties for a secret */
int qemuBuildSecretInfoProps(qemuDomainSecretInfoPtr secinfo,
......
......@@ -400,7 +400,10 @@ qemuMaybeBuildPRManagerInfoProps(virDomainObjPtr vm,
return 0;
}
return qemuBuildPRManagerInfoProps(disk->src, propsret);
if (!(*propsret = qemuBuildPRManagerInfoProps(disk->src)))
return -1;
return 0;
}
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册