Re: [PATCH v8 04/21] hmp: drop Error pointer indirection in hmp_handle

qemu-devel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v8 04/21] hmp: drop Error pointer indirection in hmp_handle_e

From:	Philippe Mathieu-Daudé
Subject:	Re: [PATCH v8 04/21] hmp: drop Error pointer indirection in hmp_handle_error
Date:	Fri, 6 Dec 2019 08:29:32 +0100
User-agent:	Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2

On 12/5/19 6:46 PM, Vladimir Sementsov-Ogievskiy wrote:

We don't need Error **, as all callers pass local Error object, which
isn't used after the call. Use Error * instead.


Good idea.

Reviewed-by: Philippe Mathieu-Daudé <address@hidden>

Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Acked-by: Dr. David Alan Gilbert <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
---
  include/monitor/hmp.h      |   2 +-
  dump/dump-hmp-cmds.c       |   4 +-
  hw/core/machine-hmp-cmds.c |   6 +-
  monitor/hmp-cmds.c         | 155 ++++++++++++++++++-------------------
  qdev-monitor.c             |   4 +-
  qom/qom-hmp-cmds.c         |   4 +-
  6 files changed, 87 insertions(+), 88 deletions(-)

diff --git a/include/monitor/hmp.h b/include/monitor/hmp.h
index a0e9511440..3d329853b2 100644
--- a/include/monitor/hmp.h
+++ b/include/monitor/hmp.h
@@ -16,7 +16,7 @@
#include "qemu/readline.h"-void hmp_handle_error(Monitor *mon, Error **errp);
+void hmp_handle_error(Monitor *mon, Error *err);
void hmp_info_name(Monitor *mon, const QDict *qdict);
  void hmp_info_version(Monitor *mon, const QDict *qdict);
diff --git a/dump/dump-hmp-cmds.c b/dump/dump-hmp-cmds.c
index 3dbf44372c..e5053b04cd 100644
--- a/dump/dump-hmp-cmds.c
+++ b/dump/dump-hmp-cmds.c
@@ -32,7 +32,7 @@ void hmp_dump_guest_memory(Monitor *mon, const QDict *qdict)
if (zlib + lzo + snappy + win_dmp > 1) {
          error_setg(&err, "only one of '-z|-l|-s|-w' can be set");
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
@@ -66,7 +66,7 @@ void hmp_dump_guest_memory(Monitor *mon, const QDict *qdict)qmp_dump_guest_memory(paging, prot, true, detach, has_begin, begin,
                            has_length, length, true, dump_format, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
      g_free(prot);
  }
diff --git a/hw/core/machine-hmp-cmds.c b/hw/core/machine-hmp-cmds.c
index cd970cc4c5..b76f7223af 100644
--- a/hw/core/machine-hmp-cmds.c
+++ b/hw/core/machine-hmp-cmds.c
@@ -55,7 +55,7 @@ void hmp_cpu_add(Monitor *mon, const QDict *qdict)
cpuid = qdict_get_int(qdict, "id");
      qmp_cpu_add(cpuid, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_hotpluggable_cpus(Monitor *mon, const QDict *qdict)
@@ -66,7 +66,7 @@ void hmp_hotpluggable_cpus(Monitor *mon, const QDict *qdict)
      CpuInstanceProperties *c;
if (err != NULL) {
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
@@ -135,7 +135,7 @@ void hmp_info_memdev(Monitor *mon, const QDict *qdict)
      monitor_printf(mon, "\n");
qapi_free_MemdevList(memdev_list);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_info_numa(Monitor *mon, const QDict *qdict)
diff --git a/monitor/hmp-cmds.c b/monitor/hmp-cmds.c
index b2551c16d1..c5dea307b6 100644
--- a/monitor/hmp-cmds.c
+++ b/monitor/hmp-cmds.c
@@ -60,11 +60,10 @@
  #include <spice/enums.h>
  #endif
-void hmp_handle_error(Monitor *mon, Error **errp)
+void hmp_handle_error(Monitor *mon, Error *err)
  {
-    assert(errp);
-    if (*errp) {
-        error_reportf_err(*errp, "Error: ");
+    if (err) {
+        error_reportf_err(err, "Error: ");
      }
  }
@@ -734,7 +733,7 @@ void hmp_info_vnc(Monitor *mon, const QDict *qdict)info2l = qmp_query_vnc_servers(&err);
      if (err) {
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
      if (!info2l) {
@@ -850,7 +849,7 @@ void hmp_info_balloon(Monitor *mon, const QDict *qdict)
info = qmp_query_balloon(&err);
      if (err) {
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
@@ -1172,7 +1171,7 @@ void hmp_sync_profile(Monitor *mon, const QDict *qdict)
          Error *err = NULL;
error_setg(&err, QERR_INVALID_PARAMETER, op);
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
      }
  }
@@ -1191,7 +1190,7 @@ void hmp_exit_preconfig(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_x_exit_preconfig(&err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_cpu(Monitor *mon, const QDict *qdict)
@@ -1220,7 +1219,7 @@ void hmp_memsave(Monitor *mon, const QDict *qdict)
      }
qmp_memsave(addr, size, filename, true, cpu_index, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_pmemsave(Monitor *mon, const QDict *qdict)
@@ -1231,7 +1230,7 @@ void hmp_pmemsave(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_pmemsave(addr, size, filename, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_ringbuf_write(Monitor *mon, const QDict *qdict)
@@ -1242,7 +1241,7 @@ void hmp_ringbuf_write(Monitor *mon, const QDict *qdict)
qmp_ringbuf_write(chardev, data, false, 0, &err);- hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_ringbuf_read(Monitor *mon, const QDict *qdict)
@@ -1255,7 +1254,7 @@ void hmp_ringbuf_read(Monitor *mon, const QDict *qdict)
data = qmp_ringbuf_read(chardev, size, false, 0, &err);
      if (err) {
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
@@ -1280,7 +1279,7 @@ void hmp_cont(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_cont(&err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_system_wakeup(Monitor *mon, const QDict *qdict)
@@ -1288,7 +1287,7 @@ void hmp_system_wakeup(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_system_wakeup(&err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_nmi(Monitor *mon, const QDict *qdict)
@@ -1296,7 +1295,7 @@ void hmp_nmi(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_inject_nmi(&err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_set_link(Monitor *mon, const QDict *qdict)
@@ -1306,7 +1305,7 @@ void hmp_set_link(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_set_link(name, up, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_block_passwd(Monitor *mon, const QDict *qdict)
@@ -1316,7 +1315,7 @@ void hmp_block_passwd(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_block_passwd(true, device, false, NULL, password, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_balloon(Monitor *mon, const QDict *qdict)
@@ -1325,7 +1324,7 @@ void hmp_balloon(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_balloon(value, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_block_resize(Monitor *mon, const QDict *qdict)
@@ -1335,7 +1334,7 @@ void hmp_block_resize(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_block_resize(true, device, false, NULL, size, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_drive_mirror(Monitor *mon, const QDict *qdict)
@@ -1358,11 +1357,11 @@ void hmp_drive_mirror(Monitor *mon, const QDict *qdict)
if (!filename) {
          error_setg(&err, QERR_MISSING_PARAMETER, "target");
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
      qmp_drive_mirror(&mirror, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_drive_backup(Monitor *mon, const QDict *qdict)
@@ -1388,12 +1387,12 @@ void hmp_drive_backup(Monitor *mon, const QDict *qdict)
if (!filename) {
          error_setg(&err, QERR_MISSING_PARAMETER, "target");
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
qmp_drive_backup(&backup, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_snapshot_blkdev(Monitor *mon, const QDict *qdict)
@@ -1409,7 +1408,7 @@ void hmp_snapshot_blkdev(Monitor *mon, const QDict *qdict)
          /* In the future, if 'snapshot-file' is not specified, the snapshot
             will be taken internally. Today it's actually required. */
          error_setg(&err, QERR_MISSING_PARAMETER, "snapshot-file");
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
@@ -1418,7 +1417,7 @@ void hmp_snapshot_blkdev(Monitor *mon, const QDict *qdict)
                                 filename, false, NULL,
                                 !!format, format,
                                 true, mode, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_snapshot_blkdev_internal(Monitor *mon, const QDict *qdict)
@@ -1428,7 +1427,7 @@ void hmp_snapshot_blkdev_internal(Monitor *mon, const 
QDict *qdict)
      Error *err = NULL;
qmp_blockdev_snapshot_internal_sync(device, name, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_snapshot_delete_blkdev_internal(Monitor *mon, const QDict *qdict)
@@ -1440,7 +1439,7 @@ void hmp_snapshot_delete_blkdev_internal(Monitor *mon, 
const QDict *qdict)
qmp_blockdev_snapshot_delete_internal_sync(device, !!id, id,
                                                 true, name, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_loadvm(Monitor *mon, const QDict *qdict)
@@ -1454,7 +1453,7 @@ void hmp_loadvm(Monitor *mon, const QDict *qdict)
      if (load_snapshot(name, &err) == 0 && saved_vm_running) {
          vm_start();
      }
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_savevm(Monitor *mon, const QDict *qdict)
@@ -1462,7 +1461,7 @@ void hmp_savevm(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
save_snapshot(qdict_get_try_str(qdict, "name"), &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_delvm(Monitor *mon, const QDict *qdict)
@@ -1476,7 +1475,7 @@ void hmp_delvm(Monitor *mon, const QDict *qdict)
                        "deleting snapshot on device '%s': ",
                        bdrv_get_device_name(bs));
      }
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_info_snapshots(Monitor *mon, const QDict *qdict)
@@ -1652,7 +1651,7 @@ void hmp_migrate_continue(Monitor *mon, const QDict 
*qdict)
          qmp_migrate_continue(val, &err);
      }
- hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_migrate_incoming(Monitor *mon, const QDict *qdict)
@@ -1662,7 +1661,7 @@ void hmp_migrate_incoming(Monitor *mon, const QDict 
*qdict)
qmp_migrate_incoming(uri, &err);- hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_migrate_recover(Monitor *mon, const QDict *qdict)
@@ -1672,7 +1671,7 @@ void hmp_migrate_recover(Monitor *mon, const QDict *qdict)
qmp_migrate_recover(uri, &err);- hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_migrate_pause(Monitor *mon, const QDict *qdict)
@@ -1681,7 +1680,7 @@ void hmp_migrate_pause(Monitor *mon, const QDict *qdict)
qmp_migrate_pause(&err);- hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
/* Kept for backwards compatibility */
@@ -1697,7 +1696,7 @@ void hmp_migrate_set_cache_size(Monitor *mon, const QDict 
*qdict)
      Error *err = NULL;
qmp_migrate_set_cache_size(value, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
/* Kept for backwards compatibility */
@@ -1728,7 +1727,7 @@ void hmp_migrate_set_capability(Monitor *mon, const QDict 
*qdict)
end:
      qapi_free_MigrationCapabilityStatusList(caps);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_migrate_set_parameter(Monitor *mon, const QDict *qdict)
@@ -1869,7 +1868,7 @@ void hmp_migrate_set_parameter(Monitor *mon, const QDict 
*qdict)
   cleanup:
      qapi_free_MigrateSetParameters(p);
      visit_free(v);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_client_migrate_info(Monitor *mon, const QDict *qdict)
@@ -1886,14 +1885,14 @@ void hmp_client_migrate_info(Monitor *mon, const QDict 
*qdict)
      qmp_client_migrate_info(protocol, hostname,
                              has_port, port, has_tls_port, tls_port,
                              !!cert_subject, cert_subject, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_migrate_start_postcopy(Monitor *mon, const QDict *qdict)
  {
      Error *err = NULL;
      qmp_migrate_start_postcopy(&err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_x_colo_lost_heartbeat(Monitor *mon, const QDict *qdict)
@@ -1901,7 +1900,7 @@ void hmp_x_colo_lost_heartbeat(Monitor *mon, const QDict 
*qdict)
      Error *err = NULL;
qmp_x_colo_lost_heartbeat(&err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_set_password(Monitor *mon, const QDict *qdict)
@@ -1912,7 +1911,7 @@ void hmp_set_password(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_set_password(protocol, password, !!connected, connected, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_expire_password(Monitor *mon, const QDict *qdict)
@@ -1922,7 +1921,7 @@ void hmp_expire_password(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_expire_password(protocol, whenstr, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_eject(Monitor *mon, const QDict *qdict)
@@ -1932,7 +1931,7 @@ void hmp_eject(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_eject(true, device, false, NULL, true, force, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
#ifdef CONFIG_VNC
@@ -1978,7 +1977,7 @@ void hmp_change(Monitor *mon, const QDict *qdict)
                                  read_only,
                                  BLOCKDEV_CHANGE_READ_ONLY_MODE_RETAIN, &err);
              if (err) {
-                hmp_handle_error(mon, &err);
+                hmp_handle_error(mon, err);
                  return;
              }
          }
@@ -1988,7 +1987,7 @@ void hmp_change(Monitor *mon, const QDict *qdict)
                                     &err);
      }
- hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_block_set_io_throttle(Monitor *mon, const QDict *qdict)
@@ -2016,7 +2015,7 @@ void hmp_block_set_io_throttle(Monitor *mon, const QDict 
*qdict)
      }
qmp_block_set_io_throttle(&throttle, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_block_stream(Monitor *mon, const QDict *qdict)
@@ -2031,7 +2030,7 @@ void hmp_block_stream(Monitor *mon, const QDict *qdict)
                       BLOCKDEV_ON_ERROR_REPORT, false, false, false, false,
                       &error);
- hmp_handle_error(mon, &error);
+    hmp_handle_error(mon, error);
  }
void hmp_block_job_set_speed(Monitor *mon, const QDict *qdict)
@@ -2042,7 +2041,7 @@ void hmp_block_job_set_speed(Monitor *mon, const QDict 
*qdict)
qmp_block_job_set_speed(device, value, &error);- hmp_handle_error(mon, &error);
+    hmp_handle_error(mon, error);
  }
void hmp_block_job_cancel(Monitor *mon, const QDict *qdict)
@@ -2053,7 +2052,7 @@ void hmp_block_job_cancel(Monitor *mon, const QDict 
*qdict)
qmp_block_job_cancel(device, true, force, &error);- hmp_handle_error(mon, &error);
+    hmp_handle_error(mon, error);
  }
void hmp_block_job_pause(Monitor *mon, const QDict *qdict)
@@ -2063,7 +2062,7 @@ void hmp_block_job_pause(Monitor *mon, const QDict *qdict)
qmp_block_job_pause(device, &error);- hmp_handle_error(mon, &error);
+    hmp_handle_error(mon, error);
  }
void hmp_block_job_resume(Monitor *mon, const QDict *qdict)
@@ -2073,7 +2072,7 @@ void hmp_block_job_resume(Monitor *mon, const QDict 
*qdict)
qmp_block_job_resume(device, &error);- hmp_handle_error(mon, &error);
+    hmp_handle_error(mon, error);
  }
void hmp_block_job_complete(Monitor *mon, const QDict *qdict)
@@ -2083,7 +2082,7 @@ void hmp_block_job_complete(Monitor *mon, const QDict 
*qdict)
qmp_block_job_complete(device, &error);- hmp_handle_error(mon, &error);
+    hmp_handle_error(mon, error);
  }
typedef struct HMPMigrationStatus
@@ -2143,7 +2142,7 @@ void hmp_migrate(Monitor *mon, const QDict *qdict)
      qmp_migrate(uri, !!blk, blk, !!inc, inc,
                  false, false, true, resume, &err);
      if (err) {
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
@@ -2181,7 +2180,7 @@ void hmp_netdev_add(Monitor *mon, const QDict *qdict)
      }
out:
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_netdev_del(Monitor *mon, const QDict *qdict)
@@ -2190,7 +2189,7 @@ void hmp_netdev_del(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_netdev_del(id, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_object_add(Monitor *mon, const QDict *qdict)
@@ -2201,7 +2200,7 @@ void hmp_object_add(Monitor *mon, const QDict *qdict)
opts = qemu_opts_from_qdict(qemu_find_opts("object"), qdict, &err);
      if (err) {
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
@@ -2209,7 +2208,7 @@ void hmp_object_add(Monitor *mon, const QDict *qdict)
      qemu_opts_del(opts);
if (err) {
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
      }
      if (obj) {
          object_unref(obj);
@@ -2222,7 +2221,7 @@ void hmp_getfd(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_getfd(fdname, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_closefd(Monitor *mon, const QDict *qdict)
@@ -2231,7 +2230,7 @@ void hmp_closefd(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_closefd(fdname, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_sendkey(Monitor *mon, const QDict *qdict)
@@ -2290,7 +2289,7 @@ void hmp_sendkey(Monitor *mon, const QDict *qdict)
      }
qmp_send_key(head, has_hold_time, hold_time, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
out:
      qapi_free_KeyValueList(head);
@@ -2309,7 +2308,7 @@ void hmp_screendump(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_screendump(filename, id != NULL, id, id != NULL, head, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_nbd_server_start(Monitor *mon, const QDict *qdict)
@@ -2364,7 +2363,7 @@ void hmp_nbd_server_start(Monitor *mon, const QDict 
*qdict)
      qapi_free_BlockInfoList(block_list);
exit:
-    hmp_handle_error(mon, &local_err);
+    hmp_handle_error(mon, local_err);
  }
void hmp_nbd_server_add(Monitor *mon, const QDict *qdict)
@@ -2376,7 +2375,7 @@ void hmp_nbd_server_add(Monitor *mon, const QDict *qdict)
qmp_nbd_server_add(device, !!name, name, true, writable,
                         false, NULL, &local_err);
-    hmp_handle_error(mon, &local_err);
+    hmp_handle_error(mon, local_err);
  }
void hmp_nbd_server_remove(Monitor *mon, const QDict *qdict)
@@ -2387,7 +2386,7 @@ void hmp_nbd_server_remove(Monitor *mon, const QDict 
*qdict)
/* Rely on NBD_SERVER_REMOVE_MODE_SAFE being the default */
      qmp_nbd_server_remove(name, force, NBD_SERVER_REMOVE_MODE_HARD, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_nbd_server_stop(Monitor *mon, const QDict *qdict)
@@ -2395,7 +2394,7 @@ void hmp_nbd_server_stop(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_nbd_server_stop(&err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_chardev_add(Monitor *mon, const QDict *qdict)
@@ -2411,7 +2410,7 @@ void hmp_chardev_add(Monitor *mon, const QDict *qdict)
          qemu_chr_new_from_opts(opts, NULL, &err);
          qemu_opts_del(opts);
      }
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_chardev_change(Monitor *mon, const QDict *qdict)
@@ -2445,7 +2444,7 @@ end:
      qapi_free_ChardevReturn(ret);
      qapi_free_ChardevBackend(backend);
      qemu_opts_del(opts);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_chardev_remove(Monitor *mon, const QDict *qdict)
@@ -2453,7 +2452,7 @@ void hmp_chardev_remove(Monitor *mon, const QDict *qdict)
      Error *local_err = NULL;
qmp_chardev_remove(qdict_get_str(qdict, "id"), &local_err);
-    hmp_handle_error(mon, &local_err);
+    hmp_handle_error(mon, local_err);
  }
void hmp_chardev_send_break(Monitor *mon, const QDict *qdict)
@@ -2461,7 +2460,7 @@ void hmp_chardev_send_break(Monitor *mon, const QDict 
*qdict)
      Error *local_err = NULL;
qmp_chardev_send_break(qdict_get_str(qdict, "id"), &local_err);
-    hmp_handle_error(mon, &local_err);
+    hmp_handle_error(mon, local_err);
  }
void hmp_qemu_io(Monitor *mon, const QDict *qdict)
@@ -2517,7 +2516,7 @@ void hmp_qemu_io(Monitor *mon, const QDict *qdict)
fail:
      blk_unref(local_blk);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_object_del(Monitor *mon, const QDict *qdict)
@@ -2526,7 +2525,7 @@ void hmp_object_del(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
user_creatable_del(id, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_info_memory_devices(Monitor *mon, const QDict *qdict)
@@ -2576,7 +2575,7 @@ void hmp_info_memory_devices(Monitor *mon, const QDict 
*qdict)
      }
qapi_free_MemoryDeviceInfoList(info_list);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_info_iothreads(Monitor *mon, const QDict *qdict)
@@ -2605,7 +2604,7 @@ void hmp_rocker(Monitor *mon, const QDict *qdict)
rocker = qmp_query_rocker(name, &err);
      if (err != NULL) {
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
@@ -2624,7 +2623,7 @@ void hmp_rocker_ports(Monitor *mon, const QDict *qdict)list = qmp_query_rocker_ports(name, &err);
      if (err != NULL) {
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
@@ -2653,7 +2652,7 @@ void hmp_rocker_of_dpa_flows(Monitor *mon, const QDict *qdict)list = qmp_query_rocker_of_dpa_flows(name, tbl_id != -1, tbl_id, &err);
      if (err != NULL) {
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
@@ -2804,7 +2803,7 @@ void hmp_rocker_of_dpa_groups(Monitor *mon, const QDict *qdict)list = qmp_query_rocker_of_dpa_groups(name, type != 9, type, &err);
      if (err != NULL) {
-        hmp_handle_error(mon, &err);
+        hmp_handle_error(mon, err);
          return;
      }
@@ -2911,7 +2910,7 @@ void hmp_info_vm_generation_id(Monitor *mon, const QDict *qdict)
      if (info) {
          monitor_printf(mon, "%s\n", info->guid);
      }
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
      qapi_free_GuidInfo(info);
  }
@@ -2930,5 +2929,5 @@ void hmp_info_memory_size_summary(Monitor *mon, const QDict *qdict)qapi_free_MemoryInfo(info);
      }
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
diff --git a/qdev-monitor.c b/qdev-monitor.c
index e6b112eb0a..29ed73e56a 100644
--- a/qdev-monitor.c
+++ b/qdev-monitor.c
@@ -890,7 +890,7 @@ void hmp_device_add(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_device_add((QDict *)qdict, NULL, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_device_del(Monitor *mon, const QDict *qdict)
@@ -899,7 +899,7 @@ void hmp_device_del(Monitor *mon, const QDict *qdict)
      Error *err = NULL;
qmp_device_del(id, &err);
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
BlockBackend *blk_by_qdev_id(const char *id, Error **errp)
diff --git a/qom/qom-hmp-cmds.c b/qom/qom-hmp-cmds.c
index a268e01eb4..cd08233a4c 100644
--- a/qom/qom-hmp-cmds.c
+++ b/qom/qom-hmp-cmds.c
@@ -37,7 +37,7 @@ void hmp_qom_list(Monitor *mon, const QDict *qdict)
          }
          qapi_free_ObjectPropertyInfoList(start);
      }
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
void hmp_qom_set(Monitor *mon, const QDict *qdict)
@@ -59,7 +59,7 @@ void hmp_qom_set(Monitor *mon, const QDict *qdict)
          }
          object_property_parse(obj, value, property, &err);
      }
-    hmp_handle_error(mon, &err);
+    hmp_handle_error(mon, err);
  }
typedef struct QOMCompositionState {

[Prev in Thread]

Current Thread

[Next in Thread]

[PATCH v8 19/21] backends/cryptodev: drop local_err from cryptodev_backend_complete(), (continued)
- [PATCH v8 19/21] backends/cryptodev: drop local_err from cryptodev_backend_complete(), Vladimir Sementsov-Ogievskiy, 2019/12/05
- [PATCH v8 16/21] hw/tpm: rename Error ** parameter to more common errp, Vladimir Sementsov-Ogievskiy, 2019/12/05
  - Re: [PATCH v8 16/21] hw/tpm: rename Error ** parameter to more common errp, Philippe Mathieu-Daudé, 2019/12/06
- [PATCH v8 03/21] error: make Error **errp const where it is appropriate, Vladimir Sementsov-Ogievskiy, 2019/12/05
  - Re: [PATCH v8 03/21] error: make Error **errp const where it is appropriate, Philippe Mathieu-Daudé, 2019/12/06
  - Re: [PATCH v8 03/21] error: make Error **errp const where it is appropriate, Markus Armbruster, 2019/12/06
- [PATCH v8 12/21] qga: rename Error ** parameter to more common errp, Vladimir Sementsov-Ogievskiy, 2019/12/05
  - Re: [PATCH v8 12/21] qga: rename Error ** parameter to more common errp, Philippe Mathieu-Daudé, 2019/12/06
  - Re: [PATCH v8 12/21] qga: rename Error ** parameter to more common errp, Markus Armbruster, 2019/12/06
- [PATCH v8 04/21] hmp: drop Error pointer indirection in hmp_handle_error, Vladimir Sementsov-Ogievskiy, 2019/12/05
  - Re: [PATCH v8 04/21] hmp: drop Error pointer indirection in hmp_handle_error, Philippe Mathieu-Daudé <=
- [PATCH v8 11/21] hw/i386/amd_iommu: rename Error ** parameter to more common errp, Vladimir Sementsov-Ogievskiy, 2019/12/05
  - Re: [PATCH v8 11/21] hw/i386/amd_iommu: rename Error ** parameter to more common errp, Philippe Mathieu-Daudé, 2019/12/06
- [PATCH v8 07/21] ppc: well form kvmppc_hint_smt_possible error hint helper, Vladimir Sementsov-Ogievskiy, 2019/12/05
  - Re: [PATCH v8 07/21] ppc: well form kvmppc_hint_smt_possible error hint helper, David Gibson, 2019/12/05
  - Re: [PATCH v8 07/21] ppc: well form kvmppc_hint_smt_possible error hint helper, Markus Armbruster, 2019/12/10
- [PATCH v8 14/21] hw/s390x: rename Error ** parameter to more common errp, Vladimir Sementsov-Ogievskiy, 2019/12/05
  - Re: [PATCH v8 14/21] hw/s390x: rename Error ** parameter to more common errp, David Hildenbrand, 2019/12/06
  - Re: [PATCH v8 14/21] hw/s390x: rename Error ** parameter to more common errp, Philippe Mathieu-Daudé, 2019/12/06
- [PATCH v8 20/21] hw/vfio/ap: drop local_err from vfio_ap_realize, Vladimir Sementsov-Ogievskiy, 2019/12/05

Prev by Date: Re: [PATCH] target/i386: skip kvm_msr_entry_add when kvm_vmx_basic is 0
Next by Date: Re: [PATCH 0/3] Remove deprecated pc-0.x machine types and related hacks
Previous by thread: [PATCH v8 04/21] hmp: drop Error pointer indirection in hmp_handle_error
Next by thread: [PATCH v8 11/21] hw/i386/amd_iommu: rename Error ** parameter to more common errp
Index(es):
- Date
- Thread