mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-12-25 07:05:28 +00:00
qemuMonitorJSONCheckError: Use g_autofree
Eliminate cleanup code by using g_autofree. Signed-off-by: Peter Krempa <pkrempa@redhat.com> Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
parent
9633dfbcfc
commit
cda31f3dba
@ -393,8 +393,8 @@ qemuMonitorJSONCheckError(virJSONValuePtr cmd,
|
|||||||
{
|
{
|
||||||
if (virJSONValueObjectHasKey(reply, "error")) {
|
if (virJSONValueObjectHasKey(reply, "error")) {
|
||||||
virJSONValuePtr error = virJSONValueObjectGet(reply, "error");
|
virJSONValuePtr error = virJSONValueObjectGet(reply, "error");
|
||||||
char *cmdstr = virJSONValueToString(cmd, false);
|
g_autofree char *cmdstr = virJSONValueToString(cmd, false);
|
||||||
char *replystr = virJSONValueToString(reply, false);
|
g_autofree char *replystr = virJSONValueToString(reply, false);
|
||||||
|
|
||||||
/* Log the full JSON formatted command & error */
|
/* Log the full JSON formatted command & error */
|
||||||
VIR_DEBUG("unable to execute QEMU command %s: %s",
|
VIR_DEBUG("unable to execute QEMU command %s: %s",
|
||||||
@ -411,20 +411,16 @@ qemuMonitorJSONCheckError(virJSONValuePtr cmd,
|
|||||||
qemuMonitorJSONCommandName(cmd),
|
qemuMonitorJSONCommandName(cmd),
|
||||||
qemuMonitorJSONStringifyError(error));
|
qemuMonitorJSONStringifyError(error));
|
||||||
|
|
||||||
VIR_FREE(cmdstr);
|
|
||||||
VIR_FREE(replystr);
|
|
||||||
return -1;
|
return -1;
|
||||||
} else if (!virJSONValueObjectHasKey(reply, "return")) {
|
} else if (!virJSONValueObjectHasKey(reply, "return")) {
|
||||||
char *cmdstr = virJSONValueToString(cmd, false);
|
g_autofree char *cmdstr = virJSONValueToString(cmd, false);
|
||||||
char *replystr = virJSONValueToString(reply, false);
|
g_autofree char *replystr = virJSONValueToString(reply, false);
|
||||||
|
|
||||||
VIR_DEBUG("Neither 'return' nor 'error' is set in the JSON reply %s: %s",
|
VIR_DEBUG("Neither 'return' nor 'error' is set in the JSON reply %s: %s",
|
||||||
NULLSTR(cmdstr), NULLSTR(replystr));
|
NULLSTR(cmdstr), NULLSTR(replystr));
|
||||||
virReportError(VIR_ERR_INTERNAL_ERROR,
|
virReportError(VIR_ERR_INTERNAL_ERROR,
|
||||||
_("unable to execute QEMU command '%s'"),
|
_("unable to execute QEMU command '%s'"),
|
||||||
qemuMonitorJSONCommandName(cmd));
|
qemuMonitorJSONCommandName(cmd));
|
||||||
VIR_FREE(cmdstr);
|
|
||||||
VIR_FREE(replystr);
|
|
||||||
return -1;
|
return -1;
|
||||||
}
|
}
|
||||||
return 0;
|
return 0;
|
||||||
|
Loading…
Reference in New Issue
Block a user