Google Git
Sign in
qemu / qemu / d14c8cc69dcd64031019da857f93ee5f6d64bb75
commitd14c8cc69dcd64031019da857f93ee5f6d64bb75[log] [tgz]
authorVladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>Tue Sep 16 16:13:57 2025 +0300
committerDaniel P. Berrangé <berrange@redhat.com>Fri Sep 19 12:46:07 2025 +0100
tree6d5ca843c8715b93dcab0cd46028ba00c335c859
parent09759245cf762d2fed4259494ec31198ac1ec0f4 [diff]
io/channel-socket: rework qio_channel_socket_copy_fds()

We want to switch from qemu_socket_set_block() to newer
qemu_set_blocking(), which provides return status of operation,
to handle errors.

Still, we want to keep qio_channel_socket_readv() interface clean,
as currently it allocate @fds only on success.

So, in case of error, we should close all incoming fds and keep
user's @fds untouched or zero.

Let's make separate functions qio_channel_handle_fds() and
qio_channel_cleanup_fds(), to achieve what we want.

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