Google Git
Sign in
qemu / qemu / 526e31de993431569ad3b8bdf461ef26d03c404d
commit526e31de993431569ad3b8bdf461ef26d03c404d[log] [tgz]
authorVladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>Tue Feb 02 15:49:52 2021 +0300
committerEric Blake <eblake@redhat.com>Mon Mar 08 16:03:21 2021 -0600
tree358d7c191a3ef9e067e65812ffa2ab27d2b7e432
parent0c1e9d2a9a3b6dd5ce74092019882fbce691d081 [diff]
block/qcow2-bitmap: return status from qcow2_store_persistent_dirty_bitmaps

It's better to return status together with setting errp. It makes
possible to avoid error propagation.

While being here, put ERRP_GUARD() to fix error_prepend(errp, ...)
usage inside qcow2_store_persistent_dirty_bitmaps() (see the comment
above ERRP_GUARD() definition in include/qapi/error.h)

Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Greg Kurz <groug@kaod.org>
Reviewed-by: Alberto Garcia <berto@igalia.com>
Message-Id: <20210202124956.63146-11-vsementsov@virtuozzo.com>
Signed-off-by: Eric Blake <eblake@redhat.com>
2 files changed
tree: 358d7c191a3ef9e067e65812ffa2ab27d2b7e432
  1. .github/
  2. .gitlab-ci.d/
  3. accel/
  4. audio/
  5. authz/
  6. backends/
  7. block/
  8. bsd-user/
  9. chardev/
  10. contrib/
  11. crypto/
  12. default-configs/
  13. disas/
  14. docs/
  15. dump/
  16. fpu/
  17. fsdev/
  18. gdb-xml/
  19. hw/
  20. include/
  21. io/
  22. libdecnumber/
  23. linux-headers/
  24. linux-user/
  25. migration/
  26. monitor/
  27. nbd/
  28. net/
  29. pc-bios/
  30. plugins/
  31. po/
  32. python/
  33. qapi/
  34. qga/
  35. qobject/
  36. qom/
  37. replay/
  38. roms/
  39. scripts/
  40. scsi/
  41. softmmu/
  42. storage-daemon/
  43. stubs/
  44. subprojects/
  45. target/
  46. tcg/
  47. tests/
  48. tools/
  49. trace/
  50. ui/
  51. util/
  52. capstone
  53. dtc
  54. meson
  55. slirp
  56. .cirrus.yml
  57. .dir-locals.el
  58. .editorconfig
  59. .exrc
  60. .gdbinit
  61. .gitattributes
  62. .gitignore
  63. .gitlab-ci.yml
  64. .gitmodules
  65. .gitpublish
  66. .mailmap
  67. .patchew.yml
  68. .readthedocs.yml
  69. .travis.yml
  70. block.c
  71. blockdev-nbd.c
  72. blockdev.c
  73. blockjob.c
  74. configure
  75. COPYING
  76. COPYING.LIB
  77. cpu.c
  78. cpus-common.c
  79. disas.c
  80. exec-vary.c
  81. gdbstub.c
  82. gitdm.config
  83. hmp-commands-info.hx
  84. hmp-commands.hx
  85. iothread.c
  86. job-qmp.c
  87. job.c
  88. Kconfig
  89. Kconfig.host
  90. LICENSE
  91. MAINTAINERS
  92. Makefile
  93. memory_ldst.c.inc
  94. meson.build
  95. meson_options.txt
  96. module-common.c
  97. os-posix.c
  98. os-win32.c
  99. qemu-bridge-helper.c
  100. qemu-edid.c
  101. qemu-img-cmds.hx
  102. qemu-img.c
  103. qemu-io-cmds.c
  104. qemu-io.c
  105. qemu-keymap.c
  106. qemu-nbd.c
  107. qemu-options-wrapper.h
  108. qemu-options.h
  109. qemu-options.hx
  110. qemu.nsi
  111. qemu.sasl
  112. README.rst
  113. replication.c
  114. replication.h
  115. thunk.c
  116. trace-events
  117. VERSION
  118. version.rc